Fix to make sure NotEmpty validator is not already set #3683

Closed
wants to merge 1 commit into from

3 participants

@grizzm0

The InArray validator on select/radio elements are added before the NotEmpty validator and therefore \Zend\InputFilter:input::injectNotEmptyValidator() fails and overrides any custom NotEmpty validator.

Fixes #3678

@mwillbanks

Please supply a unit test for this behavior 👍

@grizzm0

I'm not that into tests myself, could you perhaps help me out with this? ;)

@weierophinney weierophinney added a commit that referenced this pull request Feb 8, 2013
@weierophinney weierophinney [#3683] Added unit test for behavior
- Allows NotEmpty validator anywhere in chain (not just as first
  validator in chain)
aae8fcd
@weierophinney weierophinney added a commit that referenced this pull request Feb 8, 2013
@weierophinney weierophinney Merge branch 'hotfix/3683' into develop
Forward port #3683
21e9e2c
@weierophinney weierophinney added a commit that closed this pull request Feb 8, 2013
@weierophinney weierophinney Merge branch 'hotfix/3683'
Close #3683
Fixes #3678
fe56fe7
@weierophinney weierophinney was assigned Feb 8, 2013
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney [#3683] Added unit test for behavior
- Allows NotEmpty validator anywhere in chain (not just as first
  validator in chain)
7cd8310
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/3683'
Close #3683
Fixes #3678
0560b18
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/3683' into develop
Forward port #3683
7868119
@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney [zendframework/zendframework#3683] Added unit test for behavior
- Allows NotEmpty validator anywhere in chain (not just as first
  validator in chain)
6e0df7a
@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3683' 0337874
@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3683' into develop 9dccbe3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment