slashes and html in messages

[Sub_Zero]

Hey

You have a bug in your message system.....

The recieved message shows slashes and screwed up html codes....

The solution is to add this to the code:

Code: Select all

$message = htmlentities(stripslashes($message));

That's all...

[vn70072]

Interesting. Still, it's been funny like that since I started. I wonder if it was just considered too minor to bother with or something.

[Sub_Zero]

Yeah, it's not that it's very important, but can be annoying sometimes, and it is easy to fix.

For example if you are posting an ingame nick or alliance name with a special character like ™ or Ω, not to mention the slashes infront of an aphostrophe or quotation mark

[RobinInDaHood]

Hey

You have a bug in your message system.....

The recieved message shows slashes and screwed up html codes....

The solution is to add this to the code:

Code: Select all

$message = htmlentities(stripslashes($message));

That's all...

It would be nice if you'd also do the following:

Code: Select all

$message = str_replace("\n", "<br>", $message);

That would maintain newlines in the messages and not run everything together into one contiguous line. This line would have to be inserted after the htmlentities() call above otherwise the <br> tags would be rendered as text.

[Legoless]

Or just use phpBB BBCode class and everything is solved.

[Inferno™]

Hey

You have a bug in your message system.....

The recieved message shows slashes and screwed up html codes....

The solution is to add this to the code:

Code: Select all

$message = htmlentities(stripslashes($message));

That's all...

It would be nice if you'd also do the following:

Code: Select all

$message = str_replace("\n", "<br>", $message);

That would maintain newlines in the messages and not run everything together into one contiguous line. This line would have to be inserted after the htmlentities() call above otherwise the <br> tags would be rendered as text.

Its fixed so paragraphs are now possible. Dont you use the message system?

[RobinInDaHood]

Its fixed so paragraphs are now possible. Dont you use the message system?

Actually, not much because of its limitations. If what you're saying is correct, it's a step in the right direction of making a more usable tool.

[Sub_Zero]

It would be nice if you'd also do the following:

Code: Select all

$message = str_replace("\n", "<br>", $message);

That would maintain newlines in the messages and not run everything together into one contiguous line. This line would have to be inserted after the htmlentities() call above otherwise the <br> tags would be rendered as text.

Going into details, I obviously forgot to mention that the htmlentities function also replaces the "<" and ">" in tags, so the line:

Code: Select all

$message = htmlentities(stripslashes($message));

has to be placed before any nl2br() function call, eg. remove any existing nl2br() call and do this:

Code: Select all

$message = str_replace("\n", "<br />\n",htmlentities(stripslashes($message)));

Which would maintain the newlines and not corrupt the tags into html text.