Skip to content

Commit 16e8c4e

Browse files
committed
fix an unnecessary nested if
1 parent 628b2be commit 16e8c4e

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

ninja/orm/factory.py

+2-3
Original file line numberDiff line numberDiff line change
@@ -117,9 +117,8 @@ def convert_django_fields(
117117
raise ConfigError("Only one of 'fields' or 'exclude' should be set.")
118118

119119
model_fields_list = list(self._selected_model_fields(model, fields, exclude))
120-
if optional_fields:
121-
if optional_fields == "__all__":
122-
optional_fields = [f.name for f in model_fields_list]
120+
if optional_fields and optional_fields == "__all__":
121+
optional_fields = [f.name for f in model_fields_list]
123122

124123
definitions = {}
125124
for fld in model_fields_list:

0 commit comments

Comments
 (0)