Tokenize2 / ordered lists mangled when setting the same content twice, …

…closes opnsense/plugins#1494
opnsense · Sep 25, 2019 · a4217d8 · a4217d8 · fichtner · Sep 25, 2019
1 parent 680bd22
commit a4217d8
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/opnsense/www/js/opnsense.js b/src/opnsense/www/js/opnsense.js
@@ -136,6 +136,12 @@ function setFormData(parent,data) {
                     if (targetNode.is("select")) {
                         // handle select boxes
                         targetNode.empty(); // flush
+                        if (targetNode.hasClass("tokenize")) {
+                            // when setting the same content twice to a widget, tokenize2 sorting mixes up.
+                            // Ideally formatTokenizersUI() or tokenize2 should handle this better, but for now
+                            // this seems like the only fix that actually works.
+                            targetNode.tokenize2().trigger('tokenize:clear');
+                        }
                         $.each(node[keypart],function(indxItem, keyItem){
                             var opt = $("<option>").val(htmlDecode(indxItem)).text(keyItem["value"]);
                             if (keyItem["selected"] != "0") {