luci-app-firewall: allow "open ports" when no wan zone exists 2100/head
authorJo-Philipp Wich <jo@mein.io>
Mon, 13 Aug 2018 07:20:43 +0000 (09:20 +0200)
committerJo-Philipp Wich <jo@mein.io>
Thu, 16 Aug 2018 08:52:26 +0000 (10:52 +0200)
Arguably this makes little if no wan zone exists but prefer consistency
over heuristics and always render the "open port" shortcut.

Fixes #2056
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
(cherry picked from commit bf123fb7c4918120f5f9f638866bfae1e5b23a54)

applications/luci-app-firewall/luasrc/view/firewall/cbi_addrule.htm

index 273675cd30226fcfea872097153bd99a9f043ca7..82168712ae54a0bf3816f175c2db3be37bffe865 100644 (file)
@@ -1,11 +1,9 @@
 <%
        local fw = require "luci.model.firewall".init()
-       local wz = fw:get_zone("wan")
-       local lz = fw:get_zone("lan")
        local zones = fw:get_zones()
 %>
 
-<% if wz then %>
+<% if #zones > 0 then %>
        <h4><%:Open ports on router%></h4>
        <div class="table">
                <div class="tr cbi-section-table-titles">
@@ -71,7 +69,7 @@
        <input type="submit" class="cbi-button cbi-button-add" name="cbi.cts.<%=self.config%>.<%=self.sectiontype%>.<%=section%>" value="<%:Add%>" />
 <% end %>
 
-<% if wz then %>
+<% if #zones > 0 then %>
        <script type="text/javascript">//<![CDATA[
                cbi_validate_field('_newopen.extport', true, 'list(neg(portrange))');
                cbi_bind(document.getElementById('_newopen.extport'), 'blur',