wiki [[Image(URL)]] breaks when the URL contains commas

[fermaf@…]

I'm trying to publish a wiki page in my trac using a the Image Macro.
When I post an image like this:

[[Image(http://www.lacasadelaposada.com/casa91.jpg)]]

There is no problem and the image is displayed correctly in the browser and when I focus on the image the URL context information (left botton corner is shown correctly).

But when I use a longest URL address like this:

[[Image(http://chart.apis.google.com/chart?cht=bvs&chs=500x200&chco=00FF00,FF0000&chdl=OnTime|TimeOut&chd=t:0,0,0,1,0,1,0,0,0,1,0,0|0,0,0,3,0,0,0,0,1,0,4,2&chds=0,5&chbh=25&chtt=Evoluci%C3%B3n+de+Tickets+Cerrados(Fixed)|Prioridad:%22Normal%22|Semanalmente+2009&chxt=x,y,x,y&chxl=0:|1|2|3|4|5|6|7|8|9|10|11|12|1:||1|2|3|4|5|2:|||Semanas+del+2009||||||(%C3%BAltima%20es%20del16+al+22+de+marzo)|3:|||||Cantidad)]]

The image is not displayed and the context information in my browser (left botton corner) only shows a truncated URL when I am on the place where should be the image. The url displayed has a length 74 characters. like this:

http://chart.apis.google.com/chart?cht=bvs&chs=500x200&chco=00FF00,FF0000

I have test it using Firefox, Explorer and Opera and the same problem.

[Christian Boos]

Yep, reproduced.

[François Terrier <fterrier@…>]

The syntax of the image macro is [[Image(path, [args,] arg)]. Take for instance for the url http://www.google.com/?image=1,2,3,4,5 then it will be parsed as the following:

• path=http://www.google.com/?image=1
• arg1=2
• arg2=3
• …

The path is then wrong and since the arguments do not match the predefined argument list, they will simply be ignored.

One solution would be to have the path be everything that is before the first recognized argument. It would fix the issue for this ticket but what if the URL contains a valid argument, eg. : http://www.google.com/?image=1,title=test then title=test will be considered as an argument although it is part of the URL.

Another solution would be to introduce an escape character for the commas. The macro to insert the URL above would the be [[Image(http://www.google.com/?image=1\,2\,3\,4\,5)]]

What do you think ?

[mrelbe <mikael@…>]

I think quoting the URL should be the correct syntax, inline with ordinary links: [[Image("http://www.google.com/?image=1,2,3,4,5", arg=...)]]

[Thijs Triemstra <lists@…>]

Replying to François Terrier <fterrier@…>:

What do you think ?

Sounds like the bigger problem here but hopefully this particular bug can be patched in the meantime?

[Thijs Triemstra <lists@…>]

So yea, it's not related to 74 chars, it's the parsing of comma's like Francois describes in comment:2. Here's a 94 char url that works:

[[Image(http://www.arunalabs.com/blogs/wp-content/uploads/2007/05/aruna_url_yahoo_widget_short_url.png)]]

Renaming the ticket.

[Dirk Stöcker]

Alternatively a Quoting could be useful. Something like

  [[Image('pict,ure.png')]]

This should be easy to implement and also would work for other macros which use same argument parsing.

[iargent@…]

I found a solution - URL-encode each comma as %2C, like this:

Took me too long to figure this out, so I hope it helps someone else.

[iargent@…]

I found a solution - URL-encode each comma as %2C, like this:

[[Image(http://apps.vanpuffelen.net/charts/burndown.jsp?days=9%2C10%2C11%2C14%2C15%2C16%2C17%2C18%2C21%2C22%2C23%2C24%2C25%2C28%2C29%2C30%2C31&work=236%2C230)]]

Took me too long to figure this out, so I hope it helps someone else.

[Dirk Stöcker]

Note: The patch from r11732 can probably be adapted to allow proper quoting for the arguments as well and thus also solves #11773 without workaround.

[Ryan J Ollos]

As mentioned in comment:10, this will be resolved with #11773. IMAGE_MACRO_TEST_CASES will be extended to cover the URLs containing commas: tags/trac-1.0.4/trac/wiki/tests/macros.py@:56#L52​.

I'm considering a function for trac.util.text that will split a string at the specified token(s) provided they aren't surrounded in quotes: ​SO:2787064/121694.

[Ryan J Ollos]

Committed to 1.0-stable in [13787], merged to trunk in [13788].