Validation crashes for mutiple select.

[dkiestra]

Validation crashes when it is possible to make multiple selections using a 'select' element. I have to add the following code in the validate_required() function to make it work.

if ($("[name="" + attribute + ""]").attr('multiple') === 'multiple') {
return $("[name="" + attribute + ""] option:selected").size() > 0;
}


[joecwallace]

I'm unable to reproduce this. What jQuery and browser versions are you using?

[dkiestra]

Hi Joe,

I'm using Mozilla Firefox v20.0 and jQuery-1.9.1.
I did some more testing and I made a test file.
Please see multi-select.html file in attachment.
Validation fails when no selection is made.

Also, when one or more selections are made,
it does not show the alert message with the selected options.

However, when I add the following code to the validate_required()
function it works.

if ($("[name="" + attribute + ""]").attr('multiple') === 'multiple') {
return $("[name="" + attribute + ""] option:selected").size() > 0;
}

Thanks for the excellent support, Douwe.

On 2013-04-09 08:40, Joe Wallace wrote:

I'm unable to reproduce this. What jQuery and browser versions are you
using?

—
Reply to this email directly or view it on GitHub
#18 (comment).

[joecwallace]

Thanks for bringing this issue to my attention. I just pushed a fix. It's a little bit different than what you proposed, but you definitely set me down the path.

[dkiestra]

Hi Joe,

I like your programming style. Elegant and compact.

There's one minor issue with your latest code.
When no selection is made I get a JavaScript error
due to value being null/undefined and the validation fails.

When I add the following code at the start of the validate_match()
function it works.

if (value === null) {
return false;
}

Thanks, Douwe.

On 2013-04-11 08:52, Joe Wallace wrote:

Thanks for bringing this issue to my attention. I just pushed a fix.
It's a little bit different than what you proposed, but you definitely
set me down the path.

—
Reply to this email directly or view it on GitHub
#18 (comment).