mirror of
				https://github.com/NodeBB/NodeBB.git
				synced 2025-10-30 18:46:01 +01:00 
			
		
		
		
	fix: stripTags for editing sorted list items as well
This commit is contained in:
		| @@ -22,9 +22,7 @@ define('settings/sorted-list', [ | |||||||
| 				var itemUUID = $(item).attr('data-sorted-list-uuid'); | 				var itemUUID = $(item).attr('data-sorted-list-uuid'); | ||||||
|  |  | ||||||
| 				var formData = Settings.helper.serializeForm($('[data-sorted-list-object="' + key + '"][data-sorted-list-uuid="' + itemUUID + '"]')); | 				var formData = Settings.helper.serializeForm($('[data-sorted-list-object="' + key + '"][data-sorted-list-uuid="' + itemUUID + '"]')); | ||||||
| 				Object.entries(formData || {}).forEach(([field, value]) => { | 				stripTags(formData); | ||||||
| 					formData[field] = utils.stripHTMLTags(value, utils.stripTags); |  | ||||||
| 				}); |  | ||||||
| 				values[key].push(formData); | 				values[key].push(formData); | ||||||
| 			}); | 			}); | ||||||
| 		}, | 		}, | ||||||
| @@ -103,6 +101,7 @@ define('settings/sorted-list', [ | |||||||
|  |  | ||||||
|  |  | ||||||
| 					var data = Settings.helper.serializeForm(form); | 					var data = Settings.helper.serializeForm(form); | ||||||
|  | 					stripTags(data); | ||||||
|  |  | ||||||
| 					app.parseAndTranslate(itemTpl, data, function (itemHtml) { | 					app.parseAndTranslate(itemTpl, data, function (itemHtml) { | ||||||
| 						itemHtml = $(itemHtml); | 						itemHtml = $(itemHtml); | ||||||
| @@ -123,9 +122,7 @@ define('settings/sorted-list', [ | |||||||
| 		var $list = $container.find('[data-type="list"]'); | 		var $list = $container.find('[data-type="list"]'); | ||||||
| 		var itemTpl = $container.attr('data-item-template'); | 		var itemTpl = $container.attr('data-item-template'); | ||||||
|  |  | ||||||
| 		Object.entries(data || {}).forEach(([field, value]) => { | 		stripTags(data); | ||||||
| 			data[field] = utils.stripHTMLTags(value, utils.stripTags); |  | ||||||
| 		}); |  | ||||||
|  |  | ||||||
| 		return new Promise((resolve) => { | 		return new Promise((resolve) => { | ||||||
| 			app.parseAndTranslate(itemTpl, data, function (itemHtml) { | 			app.parseAndTranslate(itemTpl, data, function (itemHtml) { | ||||||
| @@ -140,5 +137,11 @@ define('settings/sorted-list', [ | |||||||
| 		}); | 		}); | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
|  | 	function stripTags(data) { | ||||||
|  | 		return Object.entries(data || {}).forEach(([field, value]) => { | ||||||
|  | 			data[field] = utils.stripHTMLTags(value, utils.stripTags); | ||||||
|  | 		}); | ||||||
|  | 	} | ||||||
|  |  | ||||||
| 	return SortedList; | 	return SortedList; | ||||||
| }); | }); | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user