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

Wes Wannemacher updated WW-3154:
--------------------------------

    Fix Version/s: 2.1.8

bumping to 2.1.8, I don't have time to do this right now, will do it later.

> RepopulateConversionErrorFieldValidatorSupport logs a warning when 
> repopulating fields without conversion errors
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: WW-3154
>                 URL: https://issues.apache.org/struts/browse/WW-3154
>             Project: Struts 2
>          Issue Type: Bug
>    Affects Versions: 2.1.2
>         Environment: Tomcat 6.0.18 on CentOS 4
>            Reporter: Lucas Nelson
>             Fix For: 2.1.8
>
>
> In RepopulateConversionErrorFieldValidatorSupport at line 176 (in the 2.1.2 
> version at least), a warning is logged on any field that does not have a 
> conversion error.
> The call to "conversionErrors.get(fullFieldName)" returns null when there was 
> no errors, and therefore the result of that call fails to match the 
> instanceof checks, falling through to the else clause and logging the error.
> There should be no log message when there is no conversion error as the 
> repopulate function has nothing it needs to do in that case. I would suggest 
> a "conversionErrors.contains(fullFieldName)" prior to doing anything in that 
> method. Or, change the validate() method to test for that before calling the 
> repopulateField() method.

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