Skip to content

Fix the validation of form resources to register the default theme #15428

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

Merged
merged 1 commit into from
Aug 26, 2015

Conversation

stof
Copy link
Member

@stof stof commented Aug 1, 2015

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets n/a
License MIT
Doc PR n/a

#14173 made some wrong changes, changing the behavior of the code. This reverts to the right behavior in affected places

@Tobion
Copy link
Contributor

Tobion commented Aug 2, 2015

👍 but a test would help to prevent further regressions

@xabbuh
Copy link
Member

xabbuh commented Aug 2, 2015

👍

@javiereguiluz
Copy link
Member

@stof thanks for fixing this error. However, I agree with @Tobion: we should provide a test to avoid more regressions.

@stof
Copy link
Member Author

stof commented Aug 3, 2015

@javiereguiluz will do this evening

@stof stof force-pushed the fix_config_validation branch from 2763fab to ea92610 Compare August 26, 2015 16:52
@stof
Copy link
Member Author

stof commented Aug 26, 2015

Updated with tests

@Tobion
Copy link
Contributor

Tobion commented Aug 26, 2015

Good catch, thanks @stof.

@Tobion Tobion merged commit ea92610 into symfony:2.3 Aug 26, 2015
Tobion added a commit that referenced this pull request Aug 26, 2015
…lt theme (stof)

This PR was merged into the 2.3 branch.

Discussion
----------

Fix the validation of form resources to register the default theme

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

#14173 made some wrong changes, changing the behavior of the code. This reverts to the right behavior in affected places

Commits
-------

ea92610 Fix the validation of form resources to register the default theme
@stof stof deleted the fix_config_validation branch August 27, 2015 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants