Merge pull request #21 from CMU-313/eric-p1-refactor

Added refactored code from p1
CMU-313 · Sep 22, 2024 · 56862d5 · 56862d5
2 parents 3d490ca + 159603d
commit 56862d5
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/topics/tags.js b/src/topics/tags.js
@@ -131,7 +131,7 @@ module.exports = function (Topics) {
 			await renameTag(tagData.value, tagData.newName);
 		}
 	};
-
+	const updateTagValues = tags => tags.map(tagItem => tagItem.value);
 	async function renameTag(tag, newTagName) {
 		if (!newTagName || tag === newTagName) {
 			return;
@@ -158,7 +158,7 @@ module.exports = function (Topics) {
 
 			// update 'tags' field in topic hash
 			topicData.forEach((topic) => {
-				topic.tags = topic.tags.map(tagItem => tagItem.value);
+				topic.tags = updateTagValues(topic.tags);
 				const index = topic.tags.indexOf(tag);
 				if (index !== -1) {
 					topic.tags.splice(index, 1, newTagName);