From 0560858380a583e0ce26a99a2396d8255f138cf8 Mon Sep 17 00:00:00 2001
From: Jo-Philipp Wich <jo@mein.io>
Date: Thu, 13 Jun 2019 15:13:11 +0200
Subject: [PATCH] luci-base: uci.js: add extended section syntax support,
 improve loading

Add support for extended section syntax notation in @type[#] format and
refactor the package loading function to not load configurations twice
when a package load operation is already pending and another load request
is made for the same package.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
---
 .../htdocs/luci-static/resources/uci.js       | 55 +++++++++++++++----
 1 file changed, 45 insertions(+), 10 deletions(-)

diff --git a/modules/luci-base/htdocs/luci-static/resources/uci.js b/modules/luci-base/htdocs/luci-static/resources/uci.js
index fdb8c6ab4..f57af6ae7 100644
--- a/modules/luci-base/htdocs/luci-static/resources/uci.js
+++ b/modules/luci-base/htdocs/luci-static/resources/uci.js
@@ -11,6 +11,8 @@ return L.Class.extend({
 			deletes: { },
 			reorder: { }
 		};
+
+		this.loaded = {};
 	},
 
 	callLoad: rpc.declare({
@@ -68,6 +70,24 @@ return L.Class.extend({
 		return sid;
 	},
 
+	resolveSID: function(conf, sid) {
+		if (typeof(sid) != 'string')
+			return sid;
+
+		var m = /^@([a-zA-Z0-9_-]+)\[(-?[0-9]+)\]$/.exec(sid);
+
+		if (m) {
+			var type = m[1],
+			    pos = +m[2],
+			    sections = this.sections(conf, type),
+			    section = sections[pos >= 0 ? pos : sections.length + pos];
+
+			return section ? section['.name'] : null;
+		}
+
+		return sid;
+	},
+
 	reorderSections: function() {
 		var v = this.state.values,
 		    n = this.state.creates,
@@ -109,9 +129,15 @@ return L.Class.extend({
 		return Promise.all(tasks);
 	},
 
+	loadPackage: function(packageName) {
+		if (this.loaded[packageName] == null)
+			return (this.loaded[packageName] = this.callLoad(packageName));
+
+		return Promise.resolve(this.loaded[packageName]);
+	},
+
 	load: function(packages) {
 		var self = this,
-		    seen = { },
 		    pkgs = [ ],
 		    tasks = [];
 
@@ -119,17 +145,17 @@ return L.Class.extend({
 			packages = [ packages ];
 
 		for (var i = 0; i < packages.length; i++)
-			if (!seen[packages[i]] && !self.state.values[packages[i]]) {
+			if (!self.state.values[packages[i]]) {
 				pkgs.push(packages[i]);
-				seen[packages[i]] = true;
-				tasks.push(self.callLoad(packages[i]));
+				tasks.push(self.loadPackage(packages[i]));
 			}
 
 		return Promise.all(tasks).then(function(responses) {
 			for (var i = 0; i < responses.length; i++)
 				self.state.values[pkgs[i]] = responses[i];
 
-			document.dispatchEvent(new CustomEvent('uci-loaded'));
+			if (responses.length)
+				document.dispatchEvent(new CustomEvent('uci-loaded'));
 
 			return pkgs;
 		});
@@ -144,6 +170,8 @@ return L.Class.extend({
 			delete this.state.creates[packages[i]];
 			delete this.state.changes[packages[i]];
 			delete this.state.deletes[packages[i]];
+
+			delete this.loaded[packages[i]];
 		}
 	},
 
@@ -225,22 +253,24 @@ return L.Class.extend({
 		    c = this.state.changes,
 		    d = this.state.deletes;
 
-		if (typeof(sid) == 'undefined')
-			return undefined;
+		sid = this.resolveSID(conf, sid);
+
+		if (sid == null)
+			return null;
 
 		/* requested option in a just created section */
 		if (n[conf] && n[conf][sid]) {
 			if (!n[conf])
 				return undefined;
 
-			if (typeof(opt) == 'undefined')
+			if (opt == null)
 				return n[conf][sid];
 
 			return n[conf][sid][opt];
 		}
 
 		/* requested an option value */
-		if (typeof(opt) != 'undefined') {
+		if (opt != null) {
 			/* check whether option was deleted */
 			if (d[conf] && d[conf][sid]) {
 				if (d[conf][sid] === true)
@@ -252,7 +282,7 @@ return L.Class.extend({
 			}
 
 			/* check whether option was changed */
-			if (c[conf] && c[conf][sid] && typeof(c[conf][sid][opt]) != 'undefined')
+			if (c[conf] && c[conf][sid] && c[conf][sid][opt] != null)
 				return c[conf][sid][opt];
 
 			/* return base value */
@@ -275,6 +305,8 @@ return L.Class.extend({
 		    c = this.state.changes,
 		    d = this.state.deletes;
 
+		sid = this.resolveSID(conf, sid);
+
 		if (sid == null || opt == null || opt.charAt(0) == '.')
 			return;
 
@@ -356,6 +388,9 @@ return L.Class.extend({
 		var sa = this.sections(conf),
 		    s1 = null, s2 = null;
 
+		sid1 = this.resolveSID(conf, sid1);
+		sid2 = this.resolveSID(conf, sid2);
+
 		for (var i = 0; i < sa.length; i++) {
 			if (sa[i]['.name'] != sid1)
 				continue;
-- 
2.25.1