Skip to content

Commit 7801309

Browse files
authored
Merge pull request #446 from chantera/fix-parameter-field-name
Fix parameter field name
2 parents f0a94d6 + fbc3c08 commit 7801309

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

fastapi_code_generator/parser.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -262,11 +262,12 @@ def get_parameter_type(
262262
path: List[str],
263263
) -> Optional[Argument]:
264264
parameters = self.resolve_object(parameters, ParameterObject)
265+
if parameters.name is None:
266+
raise RuntimeError("parameters.name is None") # pragma: no cover
265267
orig_name = parameters.name
268+
name = self.model_resolver.get_valid_field_name(parameters.name)
266269
if snake_case:
267-
name = stringcase.snakecase(parameters.name)
268-
else:
269-
name = parameters.name
270+
name = stringcase.snakecase(name)
270271

271272
schema: Optional[JsonSchemaObject] = None
272273
data_type: Optional[DataType] = None
@@ -307,8 +308,6 @@ def get_parameter_type(
307308
default = repr(schema.default) if schema.has_default else None
308309
self.imports_for_fastapi.append(field.imports)
309310
self.data_types.append(field.data_type)
310-
if field.name is None:
311-
raise RuntimeError("field.name is None") # pragma: no cover
312311
return Argument(
313312
name=UsefulStr(field.name),
314313
type_hint=UsefulStr(field.type_hint),

0 commit comments

Comments
 (0)