commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kunal Chowdhary (JIRA)" <j...@apache.org>
Subject [jira] Updated: (VALIDATOR-277) Validating disabled field
Date Mon, 27 Jul 2009 08:13:14 GMT

     [ 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.


Mime
View raw message