[ 
https://issues.apache.org/jira/browse/VALIDATOR-277?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Kunal Chowdhary updated VALIDATOR-277:
--------------------------------------

    Description: 
It seems, inside jcv_isFieldPresent() method we are doing a wrong check. 

When the field is checked as "disabled" at that time we are marking field 
present as "false". Now, if you see to the validateRequired method, when 
jcv_isFieldPresent() is called at that time, for disabled field, it negates the 
condition and mark the disabled field as required, which is wrong. 

Ideally, there shouldn't be any validation for the disabled field.

I guess, we need a fix.

Please let me know for any confusion.






  was:
It seems, inside jcv_isFieldPresent() method we are doing a wrong check. 

When the field is checked as "disabled" at that time we are marking field 
present as "false". Now, if you see to the validateRequired method, here when 
jcv_isFieldPresent() is called, at that time for disabled field, it negates the 
condition and mark the disbaled field as required, which is wrong. 

Ideally, there shouldn't be any validation for the disabled field.

I guess, we need a fix.

Please let me know for any confusion.







> Validating disabled field
> -------------------------
>
>                 Key: VALIDATOR-277
>                 URL: https://issues.apache.org/jira/browse/VALIDATOR-277
>             Project: Commons Validator
>          Issue Type: Bug
>          Components: JavaScript
>    Affects Versions: 1.3.1 Release
>         Environment: IE 7
>            Reporter: Kunal Chowdhary
>
> It seems, inside jcv_isFieldPresent() method we are doing a wrong check. 
> When the field is checked as "disabled" at that time we are marking field 
> present as "false". Now, if you see to the validateRequired method, when 
> jcv_isFieldPresent() is called at that time, for disabled field, it negates 
> the condition and mark the disabled field as required, which is wrong. 
> Ideally, there shouldn't be any validation for the disabled field.
> I guess, we need a fix.
> Please let me know for any confusion.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to