[jira] [Commented] (WW-3736) Client Validation is broken for Struts2 themes

["Dave Newton (Commented) (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/WW-3736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200003#comment-13200003 ] 

Dave Newton commented on WW-3736:
---------------------------------

Any objection to changing the code to:

{code:javascript}
StrutsUtils.getValidationErrors = function(data) {
   if (typeof data === "object") {
       return data;
   }

   if (data.indexOf("/* {") === 0) {
       return eval("( " + data.substring(2, data.length - 2) + " )");
   }
   
   return null;
};
{code}

                
> Client Validation is broken for Struts2 themes
> ----------------------------------------------
>
>                 Key: WW-3736
>                 URL: https://issues.apache.org/jira/browse/WW-3736
>             Project: Struts 2
>          Issue Type: Bug
>          Components: XML Validators
>    Affects Versions: 2.3.1, 2.3.1.1
>            Reporter: Johannes Geppert
>            Assignee: Johannes Geppert
>            Priority: Critical
>              Labels: client, json, validation
>             Fix For: 2.3.2
>
>
> Since Version 2.3.1 the Client Validation for Struts2 Themes is broken.
> Because the StrutsUtils.getValidationErrors Methos in utils.js expected an commented JSON String.
> StrutsUtils.getValidationErrors = function(data) {
>   if(data.indexOf("/* {") === 0) {
>     return eval("( " + data.substring(2, data.length - 2) + " )");
>   } else {
>     return null;
>   }  
> };

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira