From eb7f9ab2985f303b051a491c54e362a813f2d997 Mon Sep 17 00:00:00 2001
From: Pavel Djundik <xPaw@users.noreply.github.com>
Date: Thu, 30 Jan 2020 10:52:29 +0200
Subject: [PATCH 1/4] Implement channel history clearing on the server

---
 src/client.js                        | 22 +++++++++++++++++
 src/plugins/messageStorage/sqlite.js | 14 +++++++++++
 src/plugins/messageStorage/text.js   | 36 ++++++++++++++++++++++++----
 src/server.js                        |  6 +++++
 4 files changed, 74 insertions(+), 4 deletions(-)

diff --git a/src/client.js b/src/client.js
index b188fafa..ee5f8de5 100644
--- a/src/client.js
+++ b/src/client.js
@@ -501,6 +501,28 @@ Client.prototype.more = function(data) {
 	};
 };
 
+Client.prototype.clearHistory = function(data) {
+	const client = this;
+	const target = client.find(data.target);
+
+	if (!target) {
+		return;
+	}
+
+	target.chan.messages = [];
+	target.chan.unread = 0;
+	target.chan.highlight = 0;
+	target.chan.firstUnread = 0;
+
+	if (!target.chan.isLoggable()) {
+		return;
+	}
+
+	for (const messageStorage of this.messageStorage) {
+		messageStorage.deleteChannel(target.network, target.chan);
+	}
+};
+
 Client.prototype.open = function(socketId, target) {
 	// Due to how socket.io works internally, normal events may arrive later than
 	// the disconnect event, and because we can't control this timing precisely,
diff --git a/src/plugins/messageStorage/sqlite.js b/src/plugins/messageStorage/sqlite.js
index a1c56cd9..f54dba93 100644
--- a/src/plugins/messageStorage/sqlite.js
+++ b/src/plugins/messageStorage/sqlite.js
@@ -144,6 +144,20 @@ class MessageStorage {
 		);
 	}
 
+	deleteChannel(network, channel) {
+		if (!this.isEnabled) {
+			return;
+		}
+
+		this.database.serialize(() =>
+			this.database.run(
+				"DELETE FROM messages WHERE network = ? AND channel = ?",
+				network.uuid,
+				channel.name.toLowerCase()
+			)
+		);
+	}
+
 	/**
 	 * Load messages for given channel on a given network and resolve a promise with loaded messages.
 	 *
diff --git a/src/plugins/messageStorage/text.js b/src/plugins/messageStorage/text.js
index 0feebe8a..90044468 100644
--- a/src/plugins/messageStorage/text.js
+++ b/src/plugins/messageStorage/text.js
@@ -100,11 +100,35 @@ class TextFileMessageStorage {
 
 		line += "\n";
 
-		fs.appendFile(path.join(logPath, `${cleanFilename(channel.name)}.log`), line, (e) => {
-			if (e) {
-				log.error("Failed to write user log", e);
+		fs.appendFile(
+			path.join(logPath, TextFileMessageStorage.getChannelFileName(channel)),
+			line,
+			(e) => {
+				if (e) {
+					log.error("Failed to write user log", e);
+				}
 			}
-		});
+		);
+	}
+
+	deleteChannel() {
+		/* TODO: Truncating text logs is disabled, until we figure out some UI for it
+		if (!this.isEnabled) {
+			return;
+		}
+
+		const logPath = path.join(
+			Helper.getUserLogsPath(),
+			this.client.name,
+			TextFileMessageStorage.getNetworkFolderName(network),
+			TextFileMessageStorage.getChannelFileName(channel)
+		);
+
+		fs.truncate(logPath, 0, (e) => {
+			if (e) {
+				log.error("Failed to truncate user log", e);
+			}
+		});*/
 	}
 
 	getMessages() {
@@ -125,6 +149,10 @@ class TextFileMessageStorage {
 
 		return `${networkName}-${network.uuid.substring(networkName.length + 1)}`;
 	}
+
+	static getChannelFileName(channel) {
+		return `${cleanFilename(channel.name)}.log`;
+	}
 }
 
 module.exports = TextFileMessageStorage;
diff --git a/src/server.js b/src/server.js
index 6827daa7..6b5281bd 100644
--- a/src/server.js
+++ b/src/server.js
@@ -420,6 +420,12 @@ function initializeClient(socket, client, token, lastMessage, openChannel) {
 		network.edit(client, data);
 	});
 
+	socket.on("history:clear", (data) => {
+		if (typeof data === "object") {
+			client.clearHistory(data);
+		}
+	});
+
 	if (!Helper.config.public && !Helper.config.ldap.enable) {
 		socket.on("change-password", (data) => {
 			if (typeof data === "object") {

From 7216b8124b9f558afb36168ebb6d4b25932a8bf1 Mon Sep 17 00:00:00 2001
From: Pavel Djundik <xPaw@users.noreply.github.com>
Date: Thu, 30 Jan 2020 10:57:38 +0200
Subject: [PATCH 2/4] Add context menu to clear channel history

---
 client/css/style.css             |  1 +
 client/js/helpers/contextMenu.js | 21 +++++++++++++++++++++
 2 files changed, 22 insertions(+)

diff --git a/client/css/style.css b/client/css/style.css
index c79224e8..73cae3ff 100644
--- a/client/css/style.css
+++ b/client/css/style.css
@@ -349,6 +349,7 @@ p {
 .context-menu-action-voice::before { content: "\f067"; /* http://fontawesome.io/icon/plus/ */ }
 .context-menu-network::before { content: "\f233"; /* https://fontawesome.com/icons/server?style=solid */ }
 .context-menu-edit::before { content: "\f303"; /* https://fontawesome.com/icons/pencil-alt?style=solid */ }
+.context-menu-clear-history::before { content: "\f1f8"; /* https://fontawesome.com/icons/trash?style=solid */ }
 
 .channel-list-item .not-secure-icon::before {
 	content: "\f071"; /* https://fontawesome.com/icons/exclamation-triangle?style=solid */
diff --git a/client/js/helpers/contextMenu.js b/client/js/helpers/contextMenu.js
index 5c7d9066..0ca7b493 100644
--- a/client/js/helpers/contextMenu.js
+++ b/client/js/helpers/contextMenu.js
@@ -129,6 +129,27 @@ export function generateChannelContextMenu($root, channel, network) {
 		});
 	}
 
+	if (channel.type === "channel" || channel.type === "query") {
+		items.push({
+			label: "Clear history",
+			type: "item",
+			class: "clear-history",
+			action() {
+				// TODO: Confirmation window
+
+				channel.messages = [];
+				channel.unread = 0;
+				channel.highlight = 0;
+				channel.firstUnread = 0;
+				channel.moreHistoryAvailable = false;
+
+				socket.emit("history:clear", {
+					target: channel.id,
+				});
+			},
+		});
+	}
+
 	// Add close menu item
 	items.push({
 		label: closeMap[channel.type],

From 44a8925b8c43b263944533a485ea76c0f35e20e1 Mon Sep 17 00:00:00 2001
From: Pavel Djundik <xPaw@users.noreply.github.com>
Date: Tue, 25 Feb 2020 11:16:05 +0200
Subject: [PATCH 3/4] Create a generic confirmation dialog

---
 client/components/App.vue           |  3 ++
 client/components/ConfirmDialog.vue | 84 +++++++++++++++++++++++++++++
 client/components/ContextMenu.vue   | 12 +----
 client/css/style.css                |  4 ++
 client/js/helpers/contextMenu.js    | 58 ++++++++------------
 client/js/vue.js                    | 31 ++++++-----
 6 files changed, 131 insertions(+), 61 deletions(-)
 create mode 100644 client/components/ConfirmDialog.vue

diff --git a/client/components/App.vue b/client/components/App.vue
index 9cab17f3..fbbe9ccf 100644
--- a/client/components/App.vue
+++ b/client/components/App.vue
@@ -5,6 +5,7 @@
 		<router-view ref="window"></router-view>
 		<ImageViewer ref="imageViewer" />
 		<ContextMenu ref="contextMenu" />
+		<ConfirmDialog ref="confirmDialog" />
 		<div id="upload-overlay"></div>
 	</div>
 </template>
@@ -18,6 +19,7 @@ import storage from "../js/localStorage";
 import Sidebar from "./Sidebar.vue";
 import ImageViewer from "./ImageViewer.vue";
 import ContextMenu from "./ContextMenu.vue";
+import ConfirmDialog from "./ConfirmDialog.vue";
 
 export default {
 	name: "App",
@@ -25,6 +27,7 @@ export default {
 		Sidebar,
 		ImageViewer,
 		ContextMenu,
+		ConfirmDialog,
 	},
 	computed: {
 		viewportClasses() {
diff --git a/client/components/ConfirmDialog.vue b/client/components/ConfirmDialog.vue
new file mode 100644
index 00000000..c7ca473c
--- /dev/null
+++ b/client/components/ConfirmDialog.vue
@@ -0,0 +1,84 @@
+<template>
+	<div id="confirm-dialog-overlay" :class="{opened: data !== null}">
+		<div v-if="data !== null" id="confirm-dialog">
+			<div class="confirm-text">
+				<div class="confirm-text-title">{{ data.title }}</div>
+				<p>{{ data.text }}</p>
+			</div>
+			<div class="confirm-buttons">
+				<button class="btn btn-cancel" @click="close(false)">Cancel</button>
+				<button class="btn btn-danger" @click="close(true)">{{ data.button }}</button>
+			</div>
+		</div>
+	</div>
+</template>
+
+<style>
+#confirm-dialog {
+	background: var(--body-bg-color);
+	color: #fff;
+	margin: 10px;
+	border-radius: 5px;
+	max-width: 500px;
+}
+
+#confirm-dialog .confirm-text {
+	padding: 15px;
+	user-select: text;
+}
+
+#confirm-dialog .confirm-text-title {
+	font-size: 20px;
+	font-weight: 700;
+	margin-bottom: 10px;
+}
+
+#confirm-dialog .confirm-buttons {
+	display: flex;
+	justify-content: flex-end;
+	padding: 15px;
+	background: rgba(0, 0, 0, 0.3);
+}
+
+#confirm-dialog .confirm-buttons .btn {
+	margin-bottom: 0;
+	margin-left: 10px;
+}
+
+#confirm-dialog .confirm-buttons .btn-cancel {
+	border-color: transparent;
+}
+</style>
+
+<script>
+export default {
+	name: "ConfirmDialog",
+	data() {
+		return {
+			data: null,
+			callback: null,
+		};
+	},
+	mounted() {
+		this.$root.$on("escapekey", this.close);
+		this.$root.$on("confirm-dialog", this.open);
+	},
+	destroyed() {
+		this.$root.$off("escapekey", this.close);
+		this.$root.$off("confirm-dialog", this.open);
+	},
+	methods: {
+		open(data, callback) {
+			this.data = data;
+			this.callback = callback;
+		},
+		close(result) {
+			this.data = null;
+
+			if (this.callback) {
+				this.callback(!!result);
+			}
+		},
+	},
+};
+</script>
diff --git a/client/components/ContextMenu.vue b/client/components/ContextMenu.vue
index 81011443..a1cfefc2 100644
--- a/client/components/ContextMenu.vue
+++ b/client/components/ContextMenu.vue
@@ -38,11 +38,7 @@
 </template>
 
 <script>
-import {
-	generateUserContextMenu,
-	generateChannelContextMenu,
-	generateRemoveNetwork,
-} from "../js/helpers/contextMenu.js";
+import {generateUserContextMenu, generateChannelContextMenu} from "../js/helpers/contextMenu.js";
 
 export default {
 	name: "ContextMenu",
@@ -65,21 +61,15 @@ export default {
 		this.$root.$on("escapekey", this.close);
 		this.$root.$on("contextmenu:user", this.openUserContextMenu);
 		this.$root.$on("contextmenu:channel", this.openChannelContextMenu);
-		this.$root.$on("contextmenu:removenetwork", this.openRemoveNetworkContextMenu);
 	},
 	destroyed() {
 		this.$root.$off("escapekey", this.close);
 		this.$root.$off("contextmenu:user", this.openUserContextMenu);
 		this.$root.$off("contextmenu:channel", this.openChannelContextMenu);
-		this.$root.$off("contextmenu:removenetwork", this.openRemoveNetworkContextMenu);
 
 		this.close();
 	},
 	methods: {
-		openRemoveNetworkContextMenu(data) {
-			const items = generateRemoveNetwork(this.$root, data.lobby);
-			this.open(data.event, items);
-		},
 		openChannelContextMenu(data) {
 			const items = generateChannelContextMenu(this.$root, data.channel, data.network);
 			this.open(data.event, items);
diff --git a/client/css/style.css b/client/css/style.css
index 73cae3ff..69a679e1 100644
--- a/client/css/style.css
+++ b/client/css/style.css
@@ -2709,6 +2709,7 @@ part/quit messages where we don't load previews (adds a blank line otherwise) */
 
 /* Image viewer and drag-and-drop overlay */
 
+#confirm-dialog-overlay,
 #upload-overlay,
 #image-viewer,
 #image-viewer .open-btn,
@@ -2720,6 +2721,7 @@ part/quit messages where we don't load previews (adds a blank line otherwise) */
 	justify-content: center;
 }
 
+#confirm-dialog-overlay,
 #upload-overlay,
 #image-viewer {
 	position: fixed;
@@ -2735,12 +2737,14 @@ part/quit messages where we don't load previews (adds a blank line otherwise) */
 	user-select: none;
 }
 
+#confirm-dialog-overlay.opened,
 #upload-overlay.is-dragover,
 #image-viewer.opened {
 	visibility: visible;
 	opacity: 1;
 }
 
+#confirm-dialog-overlay,
 #image-viewer {
 	background: rgba(0, 0, 0, 0.9);
 }
diff --git a/client/js/helpers/contextMenu.js b/client/js/helpers/contextMenu.js
index 0ca7b493..9f508ebf 100644
--- a/client/js/helpers/contextMenu.js
+++ b/client/js/helpers/contextMenu.js
@@ -135,17 +135,29 @@ export function generateChannelContextMenu($root, channel, network) {
 			type: "item",
 			class: "clear-history",
 			action() {
-				// TODO: Confirmation window
+				$root.$emit(
+					"confirm-dialog",
+					{
+						title: "Clear history",
+						text: `Are you sure you want to clear history for ${channel.name}? This cannot be undone.`,
+						button: "Clear history",
+					},
+					(result) => {
+						if (!result) {
+							return;
+						}
 
-				channel.messages = [];
-				channel.unread = 0;
-				channel.highlight = 0;
-				channel.firstUnread = 0;
-				channel.moreHistoryAvailable = false;
+						channel.messages = [];
+						channel.unread = 0;
+						channel.highlight = 0;
+						channel.firstUnread = 0;
+						channel.moreHistoryAvailable = false;
 
-				socket.emit("history:clear", {
-					target: channel.id,
-				});
+						socket.emit("history:clear", {
+							target: channel.id,
+						});
+					}
+				);
 			},
 		});
 	}
@@ -282,31 +294,3 @@ export function generateUserContextMenu($root, channel, network, user) {
 
 	return items;
 }
-
-export function generateRemoveNetwork($root, lobby) {
-	return [
-		{
-			label: lobby.name,
-			type: "item",
-			class: "network",
-		},
-		{
-			type: "divider",
-		},
-		{
-			label: "Yes, remove this",
-			type: "item",
-			action() {
-				lobby.closed = true;
-				socket.emit("input", {
-					target: Number(lobby.id),
-					text: "/quit",
-				});
-			},
-		},
-		{
-			label: "Cancel",
-			type: "item",
-		},
-	];
-}
diff --git a/client/js/vue.js b/client/js/vue.js
index 3115b3a4..c23f5567 100644
--- a/client/js/vue.js
+++ b/client/js/vue.js
@@ -30,20 +30,25 @@ const vueApp = new Vue({
 		},
 		closeChannel(channel) {
 			if (channel.type === "lobby") {
-				const el = document.querySelector(
-					`#sidebar .channel-list-item[aria-controls="#chan-${channel.id}"]`
-				);
-				const rect = el.getBoundingClientRect();
-				const event = new MouseEvent("click", {
-					view: window,
-					clientX: rect.left + 10,
-					clientY: rect.top,
-				});
+				this.$root.$emit(
+					"confirm-dialog",
+					{
+						title: "Remove network",
+						text: `Are you sure you want to quit and remove ${channel.name}? This cannot be undone.`,
+						button: "Remove network",
+					},
+					(result) => {
+						if (!result) {
+							return;
+						}
 
-				this.$root.$emit("contextmenu:removenetwork", {
-					event: event,
-					lobby: channel,
-				});
+						channel.closed = true;
+						socket.emit("input", {
+							target: Number(channel.id),
+							text: "/quit",
+						});
+					}
+				);
 
 				return;
 			}

From c463d1ddd354e9ad292f93fd61d8c76f0fde5a4d Mon Sep 17 00:00:00 2001
From: Pavel Djundik <xPaw@users.noreply.github.com>
Date: Fri, 28 Feb 2020 17:01:17 +0200
Subject: [PATCH 4/4] Emit an event to clear history on all open clients

---
 client/js/helpers/contextMenu.js         |  6 ------
 client/js/socket-events/history_clear.js | 14 ++++++++++++++
 client/js/socket-events/index.js         |  1 +
 src/client.js                            |  4 ++++
 4 files changed, 19 insertions(+), 6 deletions(-)
 create mode 100644 client/js/socket-events/history_clear.js

diff --git a/client/js/helpers/contextMenu.js b/client/js/helpers/contextMenu.js
index 9f508ebf..49b3834e 100644
--- a/client/js/helpers/contextMenu.js
+++ b/client/js/helpers/contextMenu.js
@@ -147,12 +147,6 @@ export function generateChannelContextMenu($root, channel, network) {
 							return;
 						}
 
-						channel.messages = [];
-						channel.unread = 0;
-						channel.highlight = 0;
-						channel.firstUnread = 0;
-						channel.moreHistoryAvailable = false;
-
 						socket.emit("history:clear", {
 							target: channel.id,
 						});
diff --git a/client/js/socket-events/history_clear.js b/client/js/socket-events/history_clear.js
new file mode 100644
index 00000000..e0d7b2b0
--- /dev/null
+++ b/client/js/socket-events/history_clear.js
@@ -0,0 +1,14 @@
+"use strict";
+
+import socket from "../socket";
+import store from "../store";
+
+socket.on("history:clear", function(data) {
+	const {channel} = store.getters.findChannel(data.target);
+
+	channel.messages = [];
+	channel.unread = 0;
+	channel.highlight = 0;
+	channel.firstUnread = 0;
+	channel.moreHistoryAvailable = false;
+});
diff --git a/client/js/socket-events/index.js b/client/js/socket-events/index.js
index 722ec3a7..0384acd2 100644
--- a/client/js/socket-events/index.js
+++ b/client/js/socket-events/index.js
@@ -23,3 +23,4 @@ import "./sessions_list";
 import "./configuration";
 import "./changelog";
 import "./setting";
+import "./history_clear";
diff --git a/src/client.js b/src/client.js
index ee5f8de5..637f864d 100644
--- a/src/client.js
+++ b/src/client.js
@@ -514,6 +514,10 @@ Client.prototype.clearHistory = function(data) {
 	target.chan.highlight = 0;
 	target.chan.firstUnread = 0;
 
+	client.emit("history:clear", {
+		target: target.chan.id,
+	});
+
 	if (!target.chan.isLoggable()) {
 		return;
 	}