-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Update groups.rst #11223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update groups.rst #11223
Conversation
Integrating the findings of EasyCorp/EasyAdminBundle#2673 (comment) Please fix the link to [guess your form types] - I don't know how to link to an anchor in the docs!
Thanks! Personally I find this article a bit complex ... and we keep adding more and more complexity to it ... but maybe it's the way it should be because the underlying is confusing/complex/full of edge cases. |
Well, I've seen many Symfony pages that are worse than this one ;-) It took me hours to figure the details out yesterday - just wanted to share that. And whenever anything doesn't work like one could expect, it's always better to explain it... |
First of all: Thank you for your pull request! It's definitely appreciated that you put time into finding out internal logic and make it available to the rest of the community through documentation! However, I can also see Javier's point. Can't we reduce the note to "Validation groups are ignored while guessing form types and options"? That would be a short warning and will give enough information I think to understand the "weird" behaviour. What do you think? |
I agree with @wouterj. |
First of all: #11222 is by far more important! This one here is only a side product. Sorry guys, I can't see why you're always so hesitant to include more details. Do you really think that it's easier for anybody to figure it out themselves?? Compared to spending another minute to read it?
"ignored" is ambiguous here: Will it apply all constraints (i.e. ignore the
|
Yes, less verbose docs is better. Nobody likes to (or will actually) read very verbose documentation. People will skip sentences, meaning they'll almost certainly miss the important bit. If you only type the important information and leave out any verbose details that aren't needed, you "force" the reader to read the important bits. Anyway, I'm going to close this PR as there hasn't been any activity for over a year. Besides, I just discovered that this information is already inside the documentation (for the past 9 years) on the form type guessing section, which also makes more sense to me: 7a82a89 |
Integrating the findings of EasyCorp/EasyAdminBundle#2673 (comment)
Please fix the link to [guess your form types] - I don't know how to link to an anchor in the docs!