luci-base: network.js: gracefully handle deleteConfiguration() rejects

Signed-off-by: Jo-Philipp Wich <[email protected]>
(cherry picked from commit d5f30d335ff0651a3291e350602cba7409cf1e4a)

diff --git a/modules/luci-base/htdocs/luci-static/resources/network.js b/modules/luci-base/htdocs/luci-static/resources/network.js
index b4694e0f1faa7c8fa2d29851baec8a14522e08a8..8c34b4cfedd6ab9b99410a8d2a3c5a8134a2dd6a 100644 (file)
--- a/modules/luci-base/htdocs/luci-static/resources/network.js
+++ b/modules/luci-base/htdocs/luci-static/resources/network.js
@@ -1021,6 +1021,8 @@ Network = L.Class.extend(/** @lends LuCI.Network.prototype */ {
                                                return L.firewall.deleteNetwork(name).then(function() { return true });
 
                                        return true;
+                               }).catch(function() {
+                                       return false;
                                });
                        }
 
@@ -2657,7 +2659,8 @@ Protocol = L.Class.extend(/** @lends LuCI.Network.Protocol.prototype */ {
         * @returns {*|Promise<*>}
         * This function may return a promise which is awaited before the rest of
         * the configuration is removed. Any non-promise return value and any
-        * resolved promise value is ignored.
+        * resolved promise value is ignored. If the returned promise is rejected,
+        * the interface removal will be aborted.
         */
        deleteConfiguration: function() {}
 });