Thanks for that fix @func0der ! Could you fix the code styling please ?
Also I would prefer you to add a test on that point, to make it simple you'll need to:

• Add a model that does not have the nullable attribute and use validation on it to validate that if we pass null, it throw an error ;
• Add a model with the nullable attribute and use validation to validate that everything is fine.

All theses tests are done in https://github.com/janephp/janephp/blob/next/src/Component/JsonSchema/Tests/Validation/ValidationTest.php this file. And you can add models by updating the schema in the same directory.

Hey @func0der, will it be possible to fix coding style and add a test as I described to you on January ?

@Korbeil All done :)
Sorry for the long wait.

Thanks for the pull request ! Could you rebase and run the cs-fixer ? Also, could add a note about your fix in the CHANGELOG file ?
Thanks ! 🙏

The cs issues are not related to my branch and I'd rather not change any of them, because it is not at all related to the issue at hand. This would retrospectively just confuse people.

Please refer to my comment(s) in the changes above.

I'll add the changelog note real quick.

Hello @Korbeil ,

we would like to integrate this pull request because without these changes, we cannot generate the correct clients. Could you please review the PR and, if necessary, tag and integrate it?

Best regards