Opened 13 years ago
Closed 13 years ago
#5402 closed bug (invalid)
jquery-1.3.2-min.js add space between return and (E.value||"").replace(/\r/g,"")
Reported by: | pottermm | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.4 |
Component: | core | Version: | 1.3.2 |
Keywords: | space juniper min file | Cc: | |
Blocked by: | Blocking: |
Description (last modified by )
jquery-1.3.2-min.js add space between return
and (E.value||"").replace(/\r/g,"")
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.
it becomes returnDanaReplaceFunction(E.value||"").replace(/\r/g,"")
which is invalid.
it should be return DanaReplaceFunction(E.value||"").replace(/\r/g,"")
Change History (3)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
Description: | modified (diff) |
---|
(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.
comment:3 Changed 13 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
Closed for inactivity; try it with jQuery 1.4.
That's Juniper's problem and not a problem with jquery.