Re-configured routing of endorse so that 'delete' request should be h…

…andled correctly.
CMU-313 · Oct 9, 2024 · 17c67f9 · 17c67f9
1 parent 755fcdf
commit 17c67f9
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/dump.rdb b/dump.rdb
diff --git a/public/src/client/topic/threadTools.js b/public/src/client/topic/threadTools.js
@@ -59,7 +59,7 @@ define('forum/topic/threadTools', [
 		});
 
 		topicContainer.on('click', '[component="topic/unendorse"]', function () {
-			topicCommand('del', '/unendorse', 'unendorse');
+			topicCommand('del', '/endorse', 'unendorse');
 			return false;
 		});
 
@@ -347,6 +347,7 @@ define('forum/topic/threadTools', [
 		components.get('topic/endorse').toggleClass('hidden', data.isEndorsed).parent().attr('hidden', data.isEndorsed ? '' : null);
 		components.get('topic/unendorse').toggleClass('hidden', !data.isEndorsed).parent().attr('hidden', !data.isEndorsed ? '' : null);
 
+		$('[component="topic/labels"] [component="topic/locked"]').toggleClass('hidden', !data.isLocked);
 		$('[component="topic/endorsed"]').toggleClass('hidden', !data.isEndorsed);
 
 		ajaxify.data.endorsed = data.isEndorsed;

diff --git a/src/routes/write/topics.js b/src/routes/write/topics.js
@@ -30,7 +30,7 @@ module.exports = function () {
 	// Comment @YG
 	// Configured routes for 'endorse' button on the topic drop-down menu.
 	setupApiRoute(router, 'put', '/:tid/endorse', [...middlewares], controllers.write.topics.endorse);
-	setupApiRoute(router, 'delete', '/:tid/unendorse', [...middlewares], controllers.write.topics.unendorse);
+	setupApiRoute(router, 'delete', '/:tid/endorse', [...middlewares], controllers.write.topics.unendorse);
 
 	setupApiRoute(router, 'put', '/:tid/follow', [...middlewares, middleware.assert.topic], controllers.write.topics.follow);
 	setupApiRoute(router, 'delete', '/:tid/follow', [...middlewares, middleware.assert.topic], controllers.write.topics.unfollow);