jcv_isFieldPresent function causes error in IE5

• Key: VALIDATOR-208
• Type: Bug
• Status: Closed
• Resolution: Fixed
• Priority: Minor
• Assignee: Niall Pemberton
• Reporter: Leo Asanov
• Component/s: JavaScript
• Affects Version/s: 1.3.0 Release
• Fix Version/s: 1.3.1 Release

Description

Hi,

In jcv_isFieldPresent function code
"if (field == null || field == undefined)"
causes Javascript error in IE5.

Should be
if (field == null || typeof field == 'undefined')

That way it should work in all browsers.

I guess it would make sense to do a search for other references to "undefined" in javascript code.

Regards,
Leo

P.S. The following link has the instructions on how to run IE5 without need to install it locally. You night find it useful.
http://labs.insert-title.com/labs/Multiple-IEs-in-Windows_article795.aspx

Comments

[Niall Pemberton]

Fixed, thanks!

http://svn.apache.org/viewvc?view=rev&revision=478566

I've tested this on IE6 and Firefox 1.5 - but if you could test on IE5 it would be appreciated. You can download the next nightly build to test it out here:

http://people.apache.org/builds/jakarta-commons/nightly/commons-validator/

Niall