Letractively / wro4j

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

CssUrlRewritingProcessor does not handle properly empty url reference [url("")] #877

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.include this definition in one of the css files:
background-image: url("");  
2. use CssUrlRewritingProcessor

What is the expected output? What do you see instead?
The expected output is:
background-image: url("");

The current output is like:
background-image: ../cssu../cssr../cssl../css(../css"../css"../css)../css;

What version of the product are you using? On what operating system?
Im using 1.7.5 version on Windows7 (tomcat server)

Please provide any additional information below.

Original issue reported on code.google.com by itc...@gmail.com on 1 Jun 2014 at 9:35

GoogleCodeExporter commented 9 years ago

Original comment by alex.obj...@gmail.com on 2 Jun 2014 at 7:43

GoogleCodeExporter commented 9 years ago
Just curious, what the use-case for having an empty url?

Original comment by alex.obj...@gmail.com on 2 Jun 2014 at 7:48

GoogleCodeExporter commented 9 years ago
Fixed in branch 1.7.x

Original comment by alex.obj...@gmail.com on 2 Jun 2014 at 8:31

GoogleCodeExporter commented 9 years ago
Thank you very much Alex!

The use case is irresponsible customers ):

Original comment by itc...@gmail.com on 2 Jun 2014 at 9:31

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
This is one of the most funniest use-case I've ever heard of :D

Original comment by alex.obj...@gmail.com on 2 Jun 2014 at 9:40

GoogleCodeExporter commented 9 years ago
Hi Alex
was the fix applied for 1.7.5 version in maven mvnrepository?
http://mvnrepository.com/artifact/ro.isdc.wro4j/wro4j-core/1.7.5

Original comment by itc...@gmail.com on 8 Jun 2014 at 8:16

GoogleCodeExporter commented 9 years ago
No, it will be released with 1.7.6.

Original comment by alex.obj...@gmail.com on 8 Jun 2014 at 3:41