‹‹ Back to the Bugs
Posts: 9  
Users visited: 58  
Search this topic:  


corrupt
With a lowercase c

Posts: 1543
From: Germany

  28.05.2010 at 13:51
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.
----


Schwarzmetall!
Ivan
Admin

Posts: 8756

Age: 34
From: Finland

  31.05.2010 at 20:05
Fixed. The problem was with passing the input through URL (AJAX) and "+" being decoded as space.
Ragana
Rawrcat

Posts: 4918

Age: 23
From: Germany

  17.04.2011 at 23:31
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.
corrupt
With a lowercase c

Posts: 1543
From: Germany

  17.04.2011 at 23:42
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.
----


Schwarzmetall!
Ivan
Admin

Posts: 8756

Age: 34
From: Finland

  19.04.2011 at 10:59
Fixed.
corrupt
With a lowercase c

Posts: 1543
From: Germany

  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
----


Schwarzmetall!
Ivan
Admin

Posts: 8756

Age: 34
From: Finland

  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.
corrupt
With a lowercase c

Posts: 1543
From: Germany

  20.04.2011 at 00:04
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.
----


Schwarzmetall!
Ivan
Admin

Posts: 8756

Age: 34
From: Finland

  20.04.2011 at 09:44
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.

Advertise on Metal Storm


Login or register to post here.



Similar topics

Forum Topic Similarity Started
Bugs False Links 6 15.01.2008 by Lord TJ
Bugs Links in New Window 6 21.05.2006 by
Bugs Just an error in the bands countries 6 18.05.2006 by Raiden
Bugs No points for adding band pictures 4.5 09.07.2006 by Black_Handed
Bugs Wrapped strings on band profiles 4.5 03.07.2011 by Vykintas