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

Little fix in InputFilter/Factory #4838

Closed
wants to merge 2 commits into from
Closed

Conversation

Kill0p
Copy link
Contributor

@Kill0p Kill0p commented Jul 16, 2013

Hi there!

We can now add error_message option to Input without fallback which caused problems with validation final result.

Maks3w and others added 2 commits July 15, 2013 08:24
Hi there!

We can now add error_message option to Input without fallback which caused problems with validation final result.
@Maks3w
Copy link
Member

Maks3w commented Jul 16, 2013

Please add a test case

@bakura10
Copy link
Contributor

Not sure a test is needed fr adding a break.

@Maks3w
Copy link
Member

Maks3w commented Jul 17, 2013

@bakura10 Of course! because with a test we can avoid a regresion

@ghost ghost assigned weierophinney Jul 19, 2013
weierophinney added a commit that referenced this pull request Jul 19, 2013
- Added test to ensure that change works
weierophinney added a commit that referenced this pull request Jul 19, 2013
@weierophinney
Copy link
Member

I cherry-picked into master, added a unit test, and forward-ported to develop.

weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
- Added test to ensure that change works
weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants