Merge pull request #2022 from thelounge/xpaw/not-loggable-types

Define which message types should not be logged
thelounge · Feb 2, 2018 · ec70bd9 · ec70bd9
2 parents 6b02419 + 1fdd1ff
commit ec70bd9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/src/models/chan.js b/src/models/chan.js
@@ -174,6 +174,10 @@ Chan.prototype.getFilteredClone = function(lastActiveChannel, lastMessage) {
 };
 
 function writeUserLog(client, msg) {
+	if (!msg.isLoggable()) {
+		return false;
+	}
+
 	const target = client.find(this.id);
 
 	if (!target) {

diff --git a/src/models/msg.js b/src/models/msg.js
@@ -37,6 +37,12 @@ class Msg {
 	findPreview(link) {
 		return this.previews.find((preview) => preview.link === link);
 	}
+
+	isLoggable() {
+		return this.type !== Msg.Type.MOTD &&
+			this.type !== Msg.Type.BANLIST &&
+			this.type !== Msg.Type.WHOIS;
+	}
 }
 
 Msg.Type = {