Fix #565

[rodrigm]

Fix #565

---

Required fields will be false for PATCH routes instead of PUT

[Padam87]

👍

[rodrigm]

Sorry, can you give feedback to this?

[willdurand]

Could you please squash your commits? thank you

[rodrigm]

Done!

[willdurand]

thank you!

[rodrigm]

👍

[joserobleda]

😍

[devantoine]

Doesn't seem to work as expected:

[rodrigm]

Are you in the correct version?

I've tested and worked fine

[devantoine]

I'm in 2.11 and I can see your edit in my vendors.

[rodrigm]

It's strange, be sure to delete cache, etc.

[devantoine]

Already did.

The input parameter is set to a form type, is it working with form types too?

[rodrigm]

Maybe, the form fields have priority

[devantoine]

That's it. If I use the entity as the input parameter all the fields requirement are set to false. But it's not possible with a FormType then?

[rodrigm]

I think also it should be well.

[devantoine]

So what am I missing here? Do you have to set all my form's fields to required => false?

[rodrigm]

No, I think being a PATCH should ignore the form and put all required => false

[devantoine]

So a new PR is needed right?

[rodrigm]

Yeah, it's a new issue/PR