Difuse/luci
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

luci-base: form.js: prevent section creation on modal cbi save failure

Browse source 
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
This commit is contained in:
Jo-Philipp Wich 2019-06-06 21:02:04 +02:00
parent 4a0f8b5541
commit bf3bf6a8a9
1 changed files with 11 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
modules/luci-base/htdocs/luci-static/resources/form.js
View file

@ -120,7 +120,10 @@ var CBIMap = CBINode.extend({
.then(uci.save.bind(uci)) .then(uci.save.bind(uci))
.then(this.load.bind(this)) .then(this.load.bind(this))
.then(this.renderContents.bind(this)) .then(this.renderContents.bind(this))
.catch(function() { alert('Cannot save due to invalid values') }); .catch(function(e) {
alert('Cannot save due to invalid values')
return Promise.reject();
});
}, },
reset: function() { reset: function() {
@ -1049,13 +1052,14 @@ var CBITableSection = CBITypedSection.extend({
}, },
handleModalCancel: function(modalMap, ev) { handleModalCancel: function(modalMap, ev) {
L.ui.hideModal(); return Promise.resolve(L.ui.hideModal());
}, },
handleModalSave: function(modalMap, ev) { handleModalSave: function(modalMap, ev) {
modalMap.save() return modalMap.save()
.then(L.bind(this.map.reset, this.map)) .then(L.bind(this.map.reset, this.map))
.then(L.ui.hideModal); .then(L.ui.hideModal)
.catch(function() {});
}, },
renderMoreOptionsModal: function(section_id, ev) { renderMoreOptionsModal: function(section_id, ev) {
@ -1139,8 +1143,8 @@ var CBIGridSection = CBITableSection.extend({
}, },
handleModalSave: function(/* ... */) { handleModalSave: function(/* ... */) {
this.super('handleModalSave', arguments); return this.super('handleModalSave', arguments)
this.addedSection = null; .then(L.bind(function() { this.addedSection = null }, this));
}, },
handleModalCancel: function(/* ... */) { handleModalCancel: function(/* ... */) {
@ -1151,7 +1155,7 @@ var CBIGridSection = CBITableSection.extend({
this.addedSection = null; this.addedSection = null;
} }
this.super('handleModalCancel', arguments); return this.super('handleModalCancel', arguments);
}, },
renderUCISection: function(section_id) { renderUCISection: function(section_id) {