From d25f95c93dab254a7bbbb2a5e766f852f26b667b Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Fri, 26 Jul 2019 12:36:04 +0200 Subject: [PATCH] luci-app-firewall: explicitely set 'DNAT' target on new forwards Fixes: #2920 Signed-off-by: Jo-Philipp Wich --- .../luci-static/resources/view/firewall/forwards.js | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/applications/luci-app-firewall/htdocs/luci-static/resources/view/firewall/forwards.js b/applications/luci-app-firewall/htdocs/luci-static/resources/view/firewall/forwards.js index ffbfe3ccd0..63af69f8a9 100644 --- a/applications/luci-app-firewall/htdocs/luci-static/resources/view/firewall/forwards.js +++ b/applications/luci-app-firewall/htdocs/luci-static/resources/view/firewall/forwards.js @@ -107,6 +107,16 @@ return L.view.extend({ return uci.get('firewall', section_id, 'name') || _('Unnamed forward'); }; + s.handleAdd = function(ev) { + var config_name = this.uciconfig || this.map.config, + section_id = uci.add(config_name, this.sectiontype); + + uci.set(config_name, section_id, 'target', 'DNAT'); + + this.addedSection = section_id; + this.renderMoreOptionsModal(section_id); + }; + o = s.taboption('general', form.Value, 'name', _('Name')); o.placeholder = _('Unnamed forward'); o.modalonly = true; -- 2.30.2