Revert "luci-base: support hiding the "Back to Overview" button in cbi maps"
authorJo-Philipp Wich <jo@mein.io>
Thu, 3 May 2018 06:24:15 +0000 (08:24 +0200)
committerJo-Philipp Wich <jo@mein.io>
Thu, 3 May 2018 06:24:36 +0000 (08:24 +0200)
This reverts commit 52cf265c9d12537d5f37043350328d30ca11bab4.

I accidentally committed unrelated changes.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
applications/luci-app-travelmate/luasrc/model/cbi/travelmate/overview_tab.lua
modules/luci-base/luasrc/view/cbi/footer.htm

index e2f4e3f..7b6b965 100644 (file)
@@ -19,12 +19,8 @@ m = Map("travelmate", translate("Travelmate"),
        .. "<a href=\"%s\" target=\"_blank\">"
        .. "see online documentation</a>", "https://github.com/openwrt/packages/blob/master/net/travelmate/files/README.md"))
 
--- We might modify these configs too:
-m:chain("network")
-m:chain("firewall")
-
 function m.on_after_commit(self)
-       uci:apply(true)
+       luci.sys.call("env -i /etc/init.d/travelmate restart >/dev/null 2>&1")
        luci.http.redirect(luci.dispatcher.build_url("admin", "services", "travelmate"))
 end
 
@@ -46,24 +42,25 @@ if uplink == "" then
        btn.disabled = false
 
        function btn.write(self, section)
-               -- Do uci changes. Note that we must not commit here as the rollback handler
-               -- needs to see "unsaved changes" in order to roll them back.
                local iface = o:formvalue(section)
                if iface then
                        uci:set("travelmate", section, "trm_iface", iface)
+                       uci:save("travelmate")
+                       uci:commit("travelmate")
                        local net = nw:add_network(iface, { proto = "dhcp" })
                        if net then
+                               nw:save("network")
+                               nw:commit("network")
                                local zone = fw:get_zone_by_network("wan")
                                if zone then
                                        zone:add_network(iface)
+                                       fw:save("firewall")
+                                       fw:commit("firewall")
                                end
                        end
+                       luci.sys.call("env -i /bin/ubus call network reload >/dev/null 2>&1")
                end
-
-               -- Since we're prematurely terminating the cbi logic flow here and since we trigger
-               -- our modifications in a button write callback and not via cbi.apply, we do need to
-               -- tell cbi to trigger apply/rollback.
-               m.flow.autoapply = true
+               luci.http.redirect(luci.dispatcher.build_url("admin", "services", "travelmate"))
        end
        return m
 end
index e6acfb0..4d332fd 100644 (file)
@@ -1,6 +1,6 @@
        <%- if pageaction then -%>
        <div class="cbi-page-actions">
-               <% if redirect and not flow.hidebackbtn then %>
+               <% if redirect then %>
                <div style="float:left">
                        <input class="cbi-button cbi-button-link" type="button" value="<%:Back to Overview%>" onclick="location.href='<%=pcdata(redirect)%>'" />
                </div>