From e6ff0f39bbd1b8c74f07fdee78bb9c154d981f50 Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Mon, 22 Jul 2019 08:58:59 +0200 Subject: [PATCH] luci-base: widgets.js: CBIZoneSelect: improve initial rendering Ref: https://github.com/openwrt/luci/issues/2889 Signed-off-by: Jo-Philipp Wich --- .../luci-static/resources/tools/widgets.js | 31 +++++++++++-------- 1 file changed, 18 insertions(+), 13 deletions(-) diff --git a/modules/luci-base/htdocs/luci-static/resources/tools/widgets.js b/modules/luci-base/htdocs/luci-static/resources/tools/widgets.js index e79cb2aef..3de1f8258 100644 --- a/modules/luci-base/htdocs/luci-static/resources/tools/widgets.js +++ b/modules/luci-base/htdocs/luci-static/resources/tools/widgets.js @@ -30,8 +30,22 @@ var CBIZoneSelect = form.ListValue.extend({ renderWidget: function(section_id, option_index, cfgvalue) { var values = L.toArray((cfgvalue != null) ? cfgvalue : this.default), + isOutputOnly = false, choices = {}; + if (this.option == 'dest') { + for (var i = 0; i < this.section.children.length; i++) { + var opt = this.section.children[i]; + if (opt.option == 'src') { + var val = opt.cfgvalue(section_id) || opt.default; + isOutputOnly = (val == null || val == ''); + break; + } + } + + this.title = isOutputOnly ? _('Output zone') : _('Destination zone'); + } + if (this.allowlocal) { choices[''] = E('span', { 'class': 'zonebadge', @@ -55,7 +69,7 @@ var CBIZoneSelect = form.ListValue.extend({ 'style': 'background-color:' + firewall.getColorForName(null) }, [ E('strong', _('Any zone')), - (this.allowany && this.allowlocal && this.option != 'src') ? ' (%s)'.format(_('forward')) : '' + (this.allowany && this.allowlocal && !isOutputOnly) ? ' (%s)'.format(_('forward')) : '' ]); } @@ -172,18 +186,9 @@ var CBIZoneSelect = form.ListValue.extend({ }, this)); } - else if (this.option == 'dest') { - for (var i = 0; i < this.section.children.length; i++) { - var opt = this.section.children[i]; - if (opt.option == 'src') { - if (!opt.cfgvalue(section_id) && !opt.default) { - var emptyval = elem.querySelector('[data-value=""]'); - - if (emptyval != null) - emptyval.parentNode.removeChild(emptyval); - } - } - } + else if (isOutputOnly) { + var emptyval = elem.querySelector('[data-value=""]'); + emptyval.parentNode.removeChild(emptyval); } return elem; -- 2.25.1