luci-base: ui.js: resolve parent ul early in UIDropdown.toggleItem()

The parent node of the current li might be null after collapsing the
dropdown, so resolve the parent ul early to avoid passing null to
subsequent calls.

Fixes clearing custom input values in DynamicList dropdowns.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
(cherry picked from commit b8d2bcd432ab23a883de1dd4fbce29f9751a6e84)

diff --git a/modules/luci-base/htdocs/luci-static/resources/ui.js b/modules/luci-base/htdocs/luci-static/resources/ui.js
index 5d6b2e43f71b59bb868d516c65ff55fa40d39d45..95419c4991dd359b8ea961329b503875531128a9 100644 (file)
--- a/modules/luci-base/htdocs/luci-static/resources/ui.js
+++ b/modules/luci-base/htdocs/luci-static/resources/ui.js
@@ -1306,6 +1306,8 @@ var UIDropdown = UIElement.extend(/** @lends LuCI.ui.Dropdown.prototype */ {
 
        /** @private */
        toggleItem: function(sb, li, force_state) {
+               var ul = li.parentNode;
+
                if (li.hasAttribute('unselectable'))
                        return;
 
@@ -1382,7 +1384,7 @@ var UIDropdown = UIElement.extend(/** @lends LuCI.ui.Dropdown.prototype */ {
                        this.closeDropdown(sb, true);
                }
 
-               this.saveValues(sb, li.parentNode);
+               this.saveValues(sb, ul);
        },
 
        /** @private */