fix: multiple sorted-lists do not save to the correct set

NodeBB · Jan 29, 2021 · 4029ec3 · 4029ec3
1 parent 6c3b1fd
commit 4029ec3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/public/src/modules/settings.js b/public/src/modules/settings.js
@@ -531,7 +531,9 @@ define('settings', function () {
 				helper.whenReady(function () {
 					var list = formEl.find('[data-sorted-list]');
 					if (list.length) {
-						getHook(list, 'set').call(Settings, list, values);
+						list.each((idx, item) => {
+							getHook(item, 'set').call(Settings, $(item), values);
+						});
 					}
 				});
 

diff --git a/public/src/modules/settings/sorted-list.js b/public/src/modules/settings/sorted-list.js
@@ -70,7 +70,6 @@ define('settings/sorted-list', [
 	function setupRemoveButton($container, itemUUID) {
 		var removeBtn = $container.find('[data-sorted-list-uuid="' + itemUUID + '"] [data-type="remove"]');
 		removeBtn.on('click', function () {
-			console.log(itemUUID);
 			$('[data-sorted-list-uuid="' + itemUUID + '"]').remove();
 		});
 	}