this.$validator.validateAll() is not working correctly anymore

[Mouvedia]

Versions:

• VueJs: 2.2.2
• Vee-Validate: 2.0.0-rc.7

Description:

In short if I switch to rc.5, this.$validator.validateAll() correctly checks all fields' validity states.
In rc.7 errors.has gets triggered while this.$validator.validateAll() passes, which is obviously wrong.

Steps To Reproduce:

Pick a page with multiple validated fields (all with names, v-validate and v-model attributes) that uses this.$validator.validateAll() in rc.5 and try it out in rc.7.

[logaretm]

I can confirm there are situations where it does not work properly any more, I will look into it when PR #616 is merged since it introduces plenty of fixes for the pipeline.

[Mouvedia]

@logaretm now that it has been merged can you take a look at it?
I would like to upgrade because of #609 and #519.

PS: thanks for 54d00c1

[logaretm]

@Mouvedia So far the docs are working fine with validateAll locally, can you provide me with a fiddle? or you can test the latest built file and report back.

[Mouvedia]

can you provide me with a fiddle?

For my peculiar case, not until Monday.
Anyhow here's a boilerplate https://www.webpackbin.com/bins/-Kq-UgP_l-mZgJmGaSZI so that others that are affected will be able to chip in.

[ThomHurks]

@Mouvedia I expanded on your webpackbin some more, but couldn't get it to break:
https://www.webpackbin.com/bins/-Kq17kqOtMtGtP5RtNZ0
I don't know the specifics of your bug, so you'd have to add more details.

What you can also do is test your code with the master branch and check if it works now:
https://rawgit.com/logaretm/vee-validate/master/dist/vee-validate.js

[Mouvedia]

@logaretm I am 80% sure that it's an error on my part. I didn't read the rc6 CHANGELOG when I upgraded from rc5 to rc7 and didn't see the breaking changes. The reason behind this oversight: you didn't respect semver.

Ill close this issue Monday once Ill confirm my hunch.