[richfaces-issues] [JBoss JIRA] (RF-4582) RichFaces JSMin compressor can create invalid Javascript

[Nicholas Oxhøj (Issue Comment Edited) (JIRA)]

    [ https://issues.jboss.org/browse/RF-4582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12638836#comment-12638836 ] 

Nicholas Oxhøj edited comment on RF-4582 at 11/1/11 5:49 PM:
-------------------------------------------------------------

I just ran into exactly the same issue with the "UI Layout" jquery plugin.

How can this be "Optional" priority? This bug has existed for 3(three!) years and I would think, that destroying perfectly fine Javascript code ought to be higher priority...? :-)
                
      was (Author: noxhoej):
    I just into exactly the same issue with the "UI Layout" jquery plugin.

How can this be "Optional" priority? This bug has existed for 3(three!) years and I would think, that destroying perfectly fine Javascript code ought to be higher priority...?
                  
> RichFaces JSMin compressor can create invalid Javascript
> --------------------------------------------------------
>
>                 Key: RF-4582
>                 URL: https://issues.jboss.org/browse/RF-4582
>             Project: RichFaces
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: core
>    Affects Versions: 3.2.2
>         Environment: Windows XP
>            Reporter: Matt Wheeler
>            Priority: Optional
>             Fix For: 3.Future
>
>
> If a Javascript file has something like:
> var something = 'abc' + ++this.increment;
> then, after compression, it will look something like var a='abc'+++this.increment.  This may not be the exact translation, but, the important part is the three plus signs together which make the Javascript invalid.  It can of course be fixed by changing the original script to var something = 'abc' + (++this.increment);, but everyone may not have that liberty.  I tried this with the YUI Compressor, and it seemed to handle it correctly, but we could not use that for other reasons.

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