google-code-export / wro4j

Automatically exported from code.google.com/p/wro4j
1 stars 1 forks source link

Aggregate Path Prefix for ImageUrlRewriter calculated incorrectly on Windows #747

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I'm using the maven plugin and found that the aggregate folder is not 
calculating the depth properly.

If you look at computeAggregatedFolderPath() in Wro4jMojo.java in the maven 
plugin you will see the cssTargetFolder.getPath() will return the path with 
backward slashes (\).

Now look at computeAggregationPathPrefix in ImageUrlRewriter.java and observe 
that the depth is calculated by splitting the aggregate folder path by a 
forward slash (/), so the depth will always appear to be one.

Original issue reported on code.google.com by Scoo...@gmail.com on 16 Jun 2013 at 10:57

GoogleCodeExporter commented 9 years ago

Original comment by alex.obj...@gmail.com on 17 Jun 2013 at 3:02

GoogleCodeExporter commented 9 years ago
A new version of webjar-locator has been released 
(https://github.com/webjars/webjars-locator/tree/webjars-locator-0.5). 
Could you update the webjar-locator dependency version to 0.5 and confirm that 
the problem is fixed?

Original comment by alex.obj...@gmail.com on 17 Jun 2013 at 6:28

GoogleCodeExporter commented 9 years ago
I updated the webjar-locator locally to version 0.5 and I can confirm that it 
resolves this issue

Original comment by thomas.e...@gmail.com on 12 Aug 2013 at 1:56

GoogleCodeExporter commented 9 years ago

Original comment by alex.obj...@gmail.com on 18 Aug 2013 at 10:33