Simple one. A link like http://www.setlist.fm/search?query=primal+fear is only parsed until the + sign. It will result in

[link] fear

instead of

[link]

I guess it's a regex thing somewhere.

Fixed. The problem was with passing the input through URL (AJAX) and "+" being decoded as space.

Okay, so the problem still exists - this link (http://webcache.googleusercontent.com/search?q=cache:N8M-C8x1SegJ:www.esn.lv/content/cafes-pubs-clubs-riga+metal+bars+riga&cd=2&hl=en&ct=clnk&source=www.google.com) also didn't look that good in the shoutbox (it appeared as: [link] bar in riga&cd=7&hl=en&ct=clnk& source=www.google.com ).


// I edited this post & added the correct address.

Actually, Ragana seems to have discovered two bugs there:

In http://citycatalogue.com/latvia/riga/bars,-pubs/6589791/, the shoutbox stops parsing at the comma, which is one of the reserved characters in the URL standard (RFC 2396).

The second one is peculiar. It seems to parse the first two '+' signs and then stop for some reason. Notice that it also cuts the 's' after 'bar'. That error is reproducible.

Fixed.

Written by Ivan on 19.04.2011 at 10:59

Fixed.

Just shouted Raganas google search as posted above and, no. Not fixed. I left the shout online for confirmation. But the 's' in 'bars' did stay this time

Written by corrupt on 19.04.2011 at 18:59

Written by Ivan on 19.04.2011 at 10:59

Fixed.

Just shouted Raganas google search as posted above and, no. Not fixed. I left the shout online for confirmation. But the 's' in 'bars' did stay this time

Are you sure your Javascript was reloaded? Because that link worked fine for me.

Written by Ivan on 19.04.2011 at 23:47

Written by corrupt on 19.04.2011 at 18:59

Written by Ivan on 19.04.2011 at 10:59

Fixed.

Just shouted Raganas google search as posted above and, no. Not fixed. I left the shout online for confirmation. But the 's' in 'bars' did stay this time

Are you sure your Javascript was reloaded? Because that link worked fine for me.

Wait, you're doing that client-side? I don't have time to check what you changed right now, as load_shoutbox() looks the same to me, but ever thought of just base64'ing the whole shit and then storing links in the backend as escaped html? That at least is what I thought you did until I started digging into the structure and js of metalstorm. Otherwise stuff like this is bound to happen again.
But it is working now, my mistake.

Written by corrupt on 20.04.2011 at 00:04

Wait, you're doing that client-side? I don't have time to check what you changed right now, as load_shoutbox() looks the same to me, but ever thought of just base64'ing the whole shit and then storing links in the backend as escaped html? That at least is what I thought you did until I started digging into the structure and js of metalstorm. Otherwise stuff like this is bound to happen again.
But it is working now, my mistake.

Shoutbox uses AJAX. To send the shout I need to pass it through javascript. Parsing the displayed shouts is done server-side.