Merge lp:~widelands-dev/widelands/bug-1795570-chat-br-tags into lp:widelands

[GunChleoc]

Fix br tag in chat server help message

[bunnybot (widelandsofficial)]

Continuous integration builds have changed state:

Travis build 4118. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/440588155.
Appveyor build 3915. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1795570_chat_br_tags-3915.

[Notabilis (notabilis27)]

Diff is looking good and /help is printed correctly.
Just as a side note since your commit messages indicate otherwise: This also affects all chat messages of users, e.g., I can send "line1<br>line2" and it is printed in two lines. If that is intentionally (or at least acceptable), then this can be merged.

review: Approve (diff, testing)

[GunChleoc (gunchleoc)]

I didn't think of that, but I also see no harm in it. If somebody wants to get some lolz out of it, it will be pretty harmless.

@bunnybot merge

=== modified file 'src/wui/chat_msg_layout.cc'
--- src/wui/chat_msg_layout.cc	2018-08-20 19:00:20 +0000
+++ src/wui/chat_msg_layout.cc	2018-10-12 10:30:39 +0000
@@ -19,6 +19,7 @@
 
 #include "wui/chat_msg_layout.h"
 
+#include <boost/algorithm/string.hpp>
 #include <boost/format.hpp>
 
 #include "chat/chat.h"
@@ -39,6 +40,13 @@
 	return "999999";
 }
 
+std::string sanitize_message(const std::string& given_text) {
+	std::string result = richtext_escape(given_text);
+	// Preserve br tag
+	boost::replace_all(result, "&lt;br&gt;", "<br>");
+	return result;
+}
+
 }  // namespace
 
 // Returns a richtext string that can be displayed to the user.
@@ -46,7 +54,7 @@
 	const std::string& font_face = "serif";
 	std::string message = "<p><font color=33ff33 size=9>";
 
-	std::string sanitized = richtext_escape(chat_message.msg);
+	const std::string sanitized = sanitize_message(chat_message.msg);
 
 	// time calculation
 	char ts[13];
@@ -57,8 +65,8 @@
 	           color(chat_message.playern))
 	             .str();
 
-	std::string sender_escaped = richtext_escape(chat_message.sender);
-	std::string recipient_escaped = richtext_escape(chat_message.recipient);
+	const std::string sender_escaped = richtext_escape(chat_message.sender);
+	const std::string recipient_escaped = richtext_escape(chat_message.recipient);
 
 	if (chat_message.recipient.size() && chat_message.sender.size()) {
 		// Personal message handling