Closed (fixed)
Project:
Clientside Validation
Version:
7.x-1.34
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Sep 2012 at 13:32 UTC
Updated:
8 Oct 2012 at 18:11 UTC
I have a couple of hidden text fields and they were not being checked for min/max values. After digging around, I found out that the following code(line 389 to 394):
if (in_array($form['#form_id'], $include_hidden)) {
$setting['clientsideValidation']['forms'][$form_id]['includeHidden'] = TRUE;
}
elseif ($definedsettings['include_hidden']['include_hidden']) {
$setting['clientsideValidation']['forms'][$form_id]['includeHidden'] = TRUE;
}
should be changed to
if (in_array($form['#form_id'], $include_hidden)) {
$settings['clientsideValidation']['forms'][$form_id]['includeHidden'] = TRUE;
}
elseif ($definedsettings['include_hidden']['include_hidden']) {
$settings['clientsideValidation']['forms'][$form_id]['includeHidden'] = TRUE;
}
Please let me know if this is the correct solution
Thanks
Comments
Comment #1
attiks commentedthey look the same to me, our my eyes are getting tired ;-)
Comment #2
jelle_s@attiks
vs
;-)
Comment #3
jelle_sFixed in latest dev
Comment #4
attiks commented@qasimzee nice catch