Correctly hydrate when using base fieldset #2716

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

Contributor
bakura10 commented Oct 9, 2012

This fix an odd bug that I discovered while writing my webinar code. I think nobody reported it because it only occurs when using base fieldset and ArraySerializable hydrator (most people use ClassMethods when the use case is more complex I suppose).

Basically, the problem is that even if a base fieldset was set, the form itself was hydrated. The logic was correctly done for extracting phase but not for hydrating phase.

Owner

Can you provide a test, please?

@weierophinney weierophinney added a commit that referenced this pull request Oct 9, 2012
@weierophinney weierophinney [#2716] Fix form image captcha test
- tearDown() was referencing an invalid property, leading to errors.
6d31aa4
@weierophinney weierophinney added a commit that referenced this pull request Oct 9, 2012
@weierophinney weierophinney Merge branch 'hotfix/2716' into develop
Forward port #2716
771b5e3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment