luci-app-firewall: explicitely set 'DNAT' target on new forwards

Fixes: #2920
Signed-off-by: Jo-Philipp Wich <jo@mein.io>

1 files changed, 10 insertions, 0 deletions

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;