Do not remove user-data on connection reset

AgustinSRG · Aug 24, 2016 · 94cd59a · 94cd59a
1 parent e576c40
commit 94cd59a
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 3 deletions.
diff --git a/src/lib/showdown.js b/src/lib/showdown.js
@@ -209,10 +209,15 @@ class Bot {
 		}
 		this.nextSend = 0;
 		this.rooms = {};
-		this.users = {};
 		this.status.onDisconnect();
 	}
 
+	clearUserData() {
+		for (let user in this.users) {
+			delete this.users[user];
+		}
+	}
+
 	/* Events */
 
 	/**

diff --git a/src/server/handlers/tools.js b/src/server/handlers/tools.js
@@ -35,11 +35,13 @@ App.server.setHandler('tools', (context, parts) => {
 	html += ' | ';
 	html += '<a class="submenu-option' + (opt in {'seen': 1} ? '-selected' : '') + '" href="/tools/seen/">Seen</a>';
 	html += ' | ';
+	html += '<a class="submenu-option' + (opt in {'clearusers': 1} ? '-selected' : '') + '" href="/tools/clearusers/">Clear&nbsp;User-Data</a>';
+	html += ' | ';
 	html += '<a class="submenu-option' + (opt in {'hotpatch': 1} ? '-selected' : '') + '" href="/tools/hotpatch/">Hotpatch</a>';
 	html += ' | ';
-	html += '<a class="submenu-option' + (opt in {'ddata': 1} ? '-selected' : '') + '" href="/tools/ddata/">Reload Data</a>';
+	html += '<a class="submenu-option' + (opt in {'ddata': 1} ? '-selected' : '') + '" href="/tools/ddata/">Reload&nbsp;Data</a>';
 	html += ' | ';
-	html += '<a class="submenu-option' + (opt in {'cache': 1} ? '-selected' : '') + '" href="/tools/cache/">Clear Cache</a>';
+	html += '<a class="submenu-option' + (opt in {'cache': 1} ? '-selected' : '') + '" href="/tools/cache/">Clear&nbsp;Cache</a>';
 	html += ' | ';
 	html += '<a class="submenu-option' + (opt in {'eval': 1} ? '-selected' : '') + '" href="/tools/eval/">Eval&nbsp;(JavaScript)</a>';
 	html += '</div>';
@@ -57,6 +59,9 @@ App.server.setHandler('tools', (context, parts) => {
 	case 'seen':
 		toolSeen(context, html, parts);
 		break;
+	case 'clearusers':
+		toolClearUsers(context, html, parts);
+		break;
 	case 'hotpatch':
 		toolHotpatch(context, html, parts);
 		break;
@@ -336,6 +341,30 @@ function toolClearCache(context, html, parts) {
 	context.endWithWebPage(html, {title: "Develoment Tools - Showdown ChatBot"});
 }
 
+function toolClearUsers(context, html, parts) {
+	let ok = null, error = null;
+	if (context.post.clearusers) {
+		App.bot.clearUserData();
+		App.logServerAction(context.user.id, 'Clear User-Data');
+		ok = "User-Data cleared sucessfully.";
+	}
+
+	html += '<h2>Clear User-Data</h2>';
+	html += '<p>Clears temporal data stored about pokemon showdown users. ' +
+		'This includes name, last seen date, last action and alts.</p>';
+	html += '<form method="post" action="">';
+	html += '<p><input type="submit" name="clearusers" value="Clear User-Data" /></p>';
+	html += '</form>';
+
+	if (error) {
+		html += '<p style="padding:5px;"><span class="error-msg">' + error + '</span></p>';
+	} else if (ok) {
+		html += '<p style="padding:5px;"><span class="ok-msg">' + ok + '</span></p>';
+	}
+
+	context.endWithWebPage(html, {title: "Develoment Tools - Showdown ChatBot"});
+}
+
 /* Auxiliar Functions */
 
 function tryGetRoomTitle(room) {