ericwang14 / webutilities

Automatically exported from code.google.com/p/webutilities
0 stars 0 forks source link

JSCSSMergeServlet does not split the combined Javascript request correctly #19

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
We are using version 0.0.4.

Below is an extract of the server logs which highlights the following error:

 - /sdppp/jquery/jquery.json-2.2.min was requested in the combined server request
 - When it adds / processes the file it is incorrectly looking for the filename "/jquery/jqueryon-2.2.min.js"
 - It has incorrectly removed the ".js" from ".json"
 - It should be looking for a file "/jquery/jquery.json-2.2.min.js"

...
2011-07-25 15:32:20,453  INFO doGetCalled : 
/sdppp/jquery/jquery-1.5.2,/sdppp/jquery/jquery.gritter,/sdppp/jquery/jquery.jso
n-2.2.min,/sdppp/jquery/jquery.countdown - 
c.g.w.s.JSCSSMergeServlet[qtp20578834-25]
...
2011-07-25 15:32:20,469  INFO Adding path: /jquery/jqueryon-2.2.min.js(Path for 
next relative resource will be : \jquery) - 
c.g.w.s.JSCSSMergeServlet[qtp20578834-25]
...
2011-07-25 15:32:20,547  INFO Processing resource : /jquery/jqueryon-2.2.min.js 
- c.g.w.s.JSCSSMergeServlet[qtp20578834-25]
...

Original issue reported on code.google.com by keith.ha...@gmail.com on 25 Jul 2011 at 5:57

GoogleCodeExporter commented 9 years ago
We are looking into it and will get back ASAP.

Original comment by rr.patil...@gmail.com on 28 Jul 2011 at 3:48

GoogleCodeExporter commented 9 years ago
Bad bug. It has removed ".js" from the file name (containing .....json....) 
irrespective of whether it is on the end or not.

Original comment by rr.patil...@gmail.com on 15 Oct 2011 at 9:37

GoogleCodeExporter commented 9 years ago
Fix checked in - http://code.google.com/p/webutilities/source/detail?r=239

Original comment by rr.patil...@gmail.com on 15 Oct 2011 at 9:56

GoogleCodeExporter commented 9 years ago

Original comment by rr.patil...@gmail.com on 15 Oct 2011 at 7:08