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

add flag for fallback value #4401

Merged

Conversation

tolean
Copy link
Contributor

@tolean tolean commented May 3, 2013

can set empty string or null in fallback value

@@ -272,4 +272,30 @@ public function testDoNotInjectNotEmptyValidatorIfAnywhereInChain()
$this->assertEquals(2, count($validators));
$this->assertEquals($notEmptyMock, $validators[1]['instance']);
}

public static function dataFallbackValue()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should not be static

@prolic
Copy link
Contributor

prolic commented May 3, 2013

besides my small note, good to merge

@weierophinney
Copy link
Member

Please check the unit tests -- this introduces regressions.

weierophinney added a commit that referenced this pull request May 22, 2013
weierophinney added a commit that referenced this pull request May 22, 2013
@weierophinney weierophinney merged commit d308a5c into zendframework:develop May 22, 2013
@ghost ghost assigned weierophinney May 22, 2013
@tolean tolean deleted the feature/empty_fallback_value branch May 24, 2013 06:35
weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
…re/empty_fallback_value

add flag for fallback value
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.

3 participants