-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Form] Remove group options without data on debug:form command #25404
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
Conversation
(Travis failure unrelated) |
this will remove also options with a |
@stof no, this is a list of defined options only, i.e they are names (strings). |
I just applied your patch, and yes it works. |
@maidmaid thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Travis failure in unrelated
@yceruto Could you add a test case to prevent regressions? |
2259962
to
083c8af
Compare
Test added (still failure is unrelated) |
Thank you @yceruto. |
…mand (yceruto) This PR was merged into the 3.4 branch. Discussion ---------- [Form] Remove group options without data on debug:form command | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #25394 | License | MIT | Doc PR | - This also fix the normalization of the options column for text descriptor, regardless of changes in `collectOptions()` method (which is enough to fix the related bug). @maidmaid could you confirm if these changes solve the problem? Commits ------- 083c8af Remove group options without data and fix normalization
This also fix the normalization of the options column for text descriptor, regardless of changes in
collectOptions()
method (which is enough to fix the related bug).@maidmaid could you confirm if these changes solve the problem?