[jira] Updated: (WW-3154) RepopulateConversionErrorFieldValidatorSupport logs a warning when repopulating fields without conversion errors

["Wes Wannemacher (JIRA)" <ji...@apache.org>]

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