From: Jo-Philipp Wich <jow@openwrt.org>
Date: Thu, 9 Aug 2012 10:16:03 +0000 (+0000)
Subject: applications/luci-firewall: javascript & validation fixes in quickaction forms
X-Git-Tag: 0.11.0~403
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=6c66d913367af8e468244762b83e274f5ea93e92;p=project%2Fluci.git

applications/luci-firewall: javascript & validation fixes in quickaction forms
---

diff --git a/applications/luci-firewall/luasrc/view/firewall/cbi_addrule.htm b/applications/luci-firewall/luasrc/view/firewall/cbi_addrule.htm
index 473f9f4519..463b2e05f4 100644
--- a/applications/luci-firewall/luasrc/view/firewall/cbi_addrule.htm
+++ b/applications/luci-firewall/luasrc/view/firewall/cbi_addrule.htm
@@ -73,7 +73,7 @@
 		</table>
 
 		<script type="text/javascript">//<![CDATA[
-			cbi_validate_field('_newopen.extport', true, 'list(portrange)');
+			cbi_validate_field('_newopen.extport', true, 'list(neg(portrange))');
 			cbi_bind(document.getElementById('_newopen.extport'), 'blur',
 				function() {
 					var n = document.getElementById('_newopen.name');
diff --git a/applications/luci-firewall/luasrc/view/firewall/cbi_addsnat.htm b/applications/luci-firewall/luasrc/view/firewall/cbi_addsnat.htm
index 28bb22d40a..4e1681c11d 100644
--- a/applications/luci-firewall/luasrc/view/firewall/cbi_addsnat.htm
+++ b/applications/luci-firewall/luasrc/view/firewall/cbi_addsnat.htm
@@ -52,10 +52,11 @@
 
 		<script type="text/javascript">//<![CDATA[
 			cbi_validate_field('_newsnat.dport', true, 'portrange');
+			cbi_validate_field('_newsnat.dip', true, 'ip4addr');
 			cbi_combobox_init('_newsnat.dip', {
 				<% local c, k, v = 0; for k, v in ipairs(nw:get_interfaces()) do -%>
 					<%- local a; for k, a in ipairs(v:ipaddrs()) do c = c + 1 -%>
-						<% if c > 1 then %>,<% end %>'<%=a:host():string()%>': '<%=a:host():string()%> (<%=v:shortname()%>)',
+						<% if c > 1 then %>,<% end %>'<%=a:host():string()%>': '<%=a:host():string()%> (<%=v:shortname()%>)'
 					<%- end %>
 				<%- end %> }, '<%: -- Please choose -- %>', '<%: -- custom -- %>');
 		//]]></script>