luci-base: network.js: gracefully handle deleteConfiguration() rejects
authorJo-Philipp Wich <jo@mein.io>
Tue, 3 Mar 2020 20:30:32 +0000 (21:30 +0100)
committerJo-Philipp Wich <jo@mein.io>
Tue, 3 Mar 2020 20:30:32 +0000 (21:30 +0100)
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
modules/luci-base/htdocs/luci-static/resources/network.js

index 3c5530bdedb28e2093363af07eb1f05306743ff7..796b613beeb2cb20ffa237bee0faa907925ff2b2 100644 (file)
@@ -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() {}
 });