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

Multiple isValid calls #19

Merged
merged 3 commits into from
Feb 10, 2016
Merged

Multiple isValid calls #19

merged 3 commits into from
Feb 10, 2016

Conversation

wandersonwhcr
Copy link
Contributor

When we call validator with a forced parse error, next time we try to validate, we'll always receive false. I just fixed it, cleaning the $this->formatter and reconfiguring it directly with configured object parameters.

Please check ZendTest\Validator\DateTimeTest::testMultipleIsValidCalls test.

@wandersonwhcr wandersonwhcr changed the title Multiple is valid calls Multiple isValid calls Dec 15, 2015
@wandersonwhcr
Copy link
Contributor Author

You are with errors in PHPUnit develop branch.

@weierophinney weierophinney added this to the 2.6.0 milestone Feb 10, 2016
@weierophinney weierophinney self-assigned this Feb 10, 2016
@weierophinney weierophinney merged commit ef7d5de into zendframework:develop Feb 10, 2016
weierophinney added a commit that referenced this pull request Feb 10, 2016
weierophinney added a commit that referenced this pull request Feb 10, 2016
The test as written did not work on my environment, and any attempt I
made to seed it with what should be a known-good value and set of
options failed. In the end, I used `IntlDateFormatter::create(/* ... */)->format(new DateTime())`,
per the basic validation tests, and this allowed it to work.
weierophinney added a commit that referenced this pull request Feb 10, 2016
weierophinney added a commit that referenced this pull request Feb 10, 2016
@wandersonwhcr wandersonwhcr deleted the multiple-is-valid-calls branch February 12, 2016 14:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants