luci-app-banip: fix "Apply & Restart" button call chain

Signed-off-by: Dirk Brenken <[email protected]>
freifunk-berlin · Apr 27, 2024 · d4b8290 · d4b8290
1 parent c75232d
commit d4b8290
Show file tree

Hide file tree

Showing 2 changed files with 236 additions and 233 deletions.
diff --git a/applications/luci-app-banip/htdocs/luci-static/resources/view/banip/overview.js b/applications/luci-app-banip/htdocs/luci-static/resources/view/banip/overview.js
@@ -14,11 +14,14 @@
 function handleAction(ev) {
 	if (ev === 'restart') {
 		let map = document.querySelector('.cbi-map');
-		dom.callClassMethod(map, 'save').then(function() {
-			classes.ui.changes.apply(false);
+		return dom.callClassMethod(map, 'save')
+		.then(L.bind(ui.changes.apply, ui.changes))
+		.then(function() {
+			return fs.exec_direct('/etc/init.d/banip', [ev]);
 		});
+	} else {
+		return fs.exec_direct('/etc/init.d/banip', [ev]);
 	}
-	return fs.exec_direct('/etc/init.d/banip', [ev])
 }
 
 return view.extend({