Skip to content

Commit a8dca35

Browse files
committed
[php] Fix null+required properties to accept null values in generated models. Also fix PHP annotations related to same.
1 parent 64c8711 commit a8dca35

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

modules/openapi-generator/src/main/resources/php/model_generic.mustache

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -284,7 +284,7 @@ class {{classname}} {{#parentSchema}}extends {{{parent}}}{{/parentSchema}}{{^par
284284

285285
{{#vars}}
286286
{{#required}}
287-
if ($this->container['{{name}}'] === null) {
287+
if ($this->container['{{name}}'] === null{{#isNullable}} && !$this->isNullableSetToNull('{{name}}'){{/isNullable}}) {
288288
$invalidProperties[] = "'{{name}}' can't be null";
289289
}
290290
{{/required}}
@@ -365,7 +365,7 @@ class {{classname}} {{#parentSchema}}extends {{{parent}}}{{/parentSchema}}{{^par
365365
/**
366366
* Gets {{name}}
367367
*
368-
* @return {{{dataType}}}{{^required}}|null{{/required}}
368+
* @return {{{dataType}}}{{#isNullable}}|null{{/isNullable}}
369369
{{#deprecated}}
370370
* @deprecated
371371
{{/deprecated}}
@@ -378,7 +378,7 @@ class {{classname}} {{#parentSchema}}extends {{{parent}}}{{/parentSchema}}{{^par
378378
/**
379379
* Sets {{name}}
380380
*
381-
* @param {{{dataType}}}{{^required}}|null{{/required}} ${{name}}{{#description}} {{{.}}}{{/description}}{{^description}} {{{name}}}{{/description}}
381+
* @param {{{dataType}}}{{#isNullable}}|null{{/isNullable}} ${{name}}{{#description}} {{{.}}}{{/description}}{{^description}} {{{name}}}{{/description}}
382382
*
383383
* @return self
384384
{{#deprecated}}

0 commit comments

Comments
 (0)