Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Zend\InputFilter\Input\Input->merge error #6819

Closed
mezor opened this issue Oct 29, 2014 · 2 comments
Closed

Zend\InputFilter\Input\Input->merge error #6819

mezor opened this issue Oct 29, 2014 · 2 comments

Comments

@mezor
Copy link

mezor commented Oct 29, 2014

If you merge two Zend\InputFilter\Input\Input Instances (with the same property name) into each other, the merge could be wrong. Unittest:

setRequired(true); $input->setAllowEmpty(true); ``` $input2 = new Input('foo'); $input2->setRequired(true); $input2->setAllowEmpty(true); $input2->merge($input); $this->assertTrue($input2->isRequired()); $this->assertTrue($input2->allowEmpty()); } ``` }
@mezor
Copy link
Author

mezor commented Oct 29, 2014

Seems to be fixed in 2.3.3, but the fix needs to be merged in 2.2.x

@Maks3w
Copy link
Member

Maks3w commented Aug 27, 2015

Close. 2.4 and 2.5 has been released.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants