Bug Tracker

Changes between Initial Version and Version 2 of Ticket #5402


Ignore:
Timestamp:
Oct 21, 2009, 9:39:04 PM (13 years ago)
Author:
dmethvin
Comment:

(reformatted the original post)

I think this is referring to line 449 in the uncompressed 1.3.2 file, which is inside the val() method: 

  // Everything else, we just grab the value
  return (elem.value || "").replace(/\r/g, "");

The compressed code looks like this: 

  return(E.value||"").replace(/\r/g,"")

So you are saying that adding a space between the return and the ( would satisfy Juniper's (broken) parser? We're using YUI compressor to create the minified version, and it rightly is removing that space because it's not needed. I'm not sure how it can be retained in the minified version.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #5402 – Description

    initial v2  
    1 jquery-1.3.2-min.js add space between return and (E.value||"").replace(/\r/g,"")
     1jquery-1.3.2-min.js add space between {{{return}}} and {{{(E.value||"").replace(/\r/g,"")}}}
    22
    3 When using juniper network protect it manipulates javascript files for it to work.  Juniper creates a wrapper function around (E.value||"").replace(/\r/g,"") causing it to be an invalid function.
     3When using juniper network protect it manipulates javascript files for it to work.  Juniper creates a wrapper function around {{{(E.value||"").replace(/\r/g,"")}}} causing it to be an invalid function.
    44
    5 it becomes returnDanaReplaceFunction(E.value||"").replace(/\r/g,"") which is invalid.
     5it becomes {{{returnDanaReplaceFunction(E.value||"").replace(/\r/g,"")}}} which is invalid.
    66
    7 it should be return DanaReplaceFunction(E.value||"").replace(/\r/g,"")
     7it should be {{{return DanaReplaceFunction(E.value||"").replace(/\r/g,"")}}}
    88
    99