Fixed validation for named array within form validation (works for name="xyz" and name="xyz[]") both

[saurabhiam]

Description

If a form input has a name="xyz[]" then only first element with name="xyz[]" is validated and rest are ignored.
This fix adds the flexibility for validating the named array (name="xyz[]") or a single name attribute (name="xyz") with no extra effort

Thank you!

[saurabhiam]

Build is failing because of missing spaces after ( ) and a new line before comment. rest is fine

[gimler]

@Saurabh-Sharma i run into the same issue. Your patch works for me. why did you close this PR?

[saurabhiam]

@gimler Reopened, need to update test cases

[saurabhiam]

@cmcawood Here it is #2000

[stale]

This issue/proposal has been automatically marked as idle and stale because it hasn't had any recent activity. It will be automtically closed if no further activity occurs. If you think this is wrong, or the problem still persists, just pop a reply in the comments and one of the maintainers will (try!) to follow up.
Thank you for contributing :)

[gimler]

still an issue the fix works

[saurabhiam]

Checks are failing due to test cases, which are expecting validation on single field. which is exactly not the case here.

>> validator - validating multiple checkboxes with 'required'
>> Message: null
>> Actual: 5
>> Expected: 1
>> file:///home/travis/build/jquery-validation/jquery-validation/test/test.js:1098:14

>> events - validate multiple checkbox on click
>> Message: null
>> Actual: 2
>> Expected: 1
>> errors@file:///home/travis/build/jquery-validation/jquery-validation/test/test.js:1917:15
>> file:///home/travis/build/jquery-validation/jquery-validation/test/test.js:1941:8

>> events - validate radio on click
>> Message: null
>> Actual: 2
>> Expected: 1
>> errors@file:///home/travis/build/jquery-validation/jquery-validation/test/test.js:1976:15
>> file:///home/travis/build/jquery-validation/jquery-validation/test/test.js:1994:8

[fansanelli]

@Saurabh-Sharma Hi,
I've applied you patch and it seems to workaround the issue without fixing it.
I suggest to add an "index" property in the "element" object using the jquery .index() method, then change the find-by-name function to accept it:
return $( this.currentForm ).find( "[name='" + this.escapeCssMeta( name ) + "']:eq(" + index + ")" );
I'm not familiar with the validator code, but if you cannot give it a try I could to hack around my idea.
What do you think?
Cheers,
Francesco

[gimler]

1.18.0 still in issue. The PR works fine for me.