luci-app-upnp: Critical hotfix revert 7a7f9ec
authorPaul Donald <newtwen+github@gmail.com>
Mon, 2 Dec 2024 19:56:05 +0000 (20:56 +0100)
committerPaul Donald <newtwen+github@gmail.com>
Mon, 2 Dec 2024 19:56:05 +0000 (20:56 +0100)
Revert "luci-app-upnp: remove dangling lines and split translations"
This reverts commit 7a7f9ec, as it introduced a regression.

Close #7444

Signed-off-by: Self-Hosting-Group <selfhostinggroup-git+openwrt@shost.ing>
applications/luci-app-upnp/htdocs/luci-static/resources/view/status/include/80_upnp.js
applications/luci-app-upnp/htdocs/luci-static/resources/view/upnp/upnp.js

index dcb6963734a25fd40eab948774b964c5c6488779..3173fb35bcc5dd296eb4d73b0d9a84a69b82c335 100644 (file)
@@ -37,6 +37,7 @@ return baseclass.extend({
        },
 
        render: function(data) {
+
                var table = E('table', { 'class': 'table', 'id': 'upnp_status_table' }, [
                        E('tr', { 'class': 'tr table-titles' }, [
                                E('th', { 'class': 'th' }, _('Client Name')),
@@ -50,6 +51,7 @@ return baseclass.extend({
                ]);
 
                var rules = Array.isArray(data[0].rules) ? data[0].rules : [];
+
                var rows = rules.map(function(rule) {
                        return [
                                rule.host_hint || _('Unknown'),
@@ -64,7 +66,9 @@ return baseclass.extend({
                                }, [ _('Delete') ])
                        ];
                });
+
                cbi_update_table(table, rows, E('em', _('There are no active port maps.')));
+
                return table;
        }
 });
index a51cd72a010097b2c47aea7f4e6c773d79d41b56..f216dd0e1cde862b83affb8a8dfc7d5ff9c2911b 100644 (file)
@@ -45,7 +45,9 @@ return view.extend({
        },
 
        poll_status: function(nodes, data) {
+
                var rules = Array.isArray(data[0].rules) ? data[0].rules : [];
+
                var rows = rules.map(function(rule) {
                        return [
                                rule.host_hint || _('Unknown'),
@@ -60,10 +62,14 @@ return view.extend({
                                }, [ _('Delete') ])
                        ];
                });
+
                cbi_update_table(nodes.querySelector('#upnp_status_table'), rows, E('em', _('There are no active port maps.')));
+
+               return;
        },
 
        render: function(data) {
+
                var m, s, o;
 
                var protocols = '%s & %s/%s'.format(
@@ -92,6 +98,7 @@ return view.extend({
                        ]);
 
                        var rules = Array.isArray(data[0].rules) ? data[0].rules : [];
+
                        var rows = rules.map(function(rule) {
                                return [
                                        rule.host_hint || _('Unknown'),
@@ -106,7 +113,9 @@ return view.extend({
                                        }, [ _('Delete') ])
                                ];
                        });
+
                        cbi_update_table(table, rows, E('em', _('There are no active port maps.')));
+
                        return E('div', { 'class': 'cbi-section cbi-tblsection' }, [
                                        E('h3', _('Active Service Port Maps')), table ]);
                }, o, this);
@@ -120,11 +129,9 @@ return view.extend({
                        _('Start autonomous port mapping service'));
                o.rmempty = false;
 
-               s.taboption('setup', form.Flag, 'enable_upnp', _('Enable UPnP IGD protocol'))
-               s.default = '1';
+               s.taboption('setup', form.Flag, 'enable_upnp', _('Enable UPnP IGD protocol')).default = '1';
 
-               s.taboption('setup', form.Flag, 'enable_natpmp', _('Enable PCP/NAT-PMP protocols'))
-               s.default = '1';
+               s.taboption('setup', form.Flag, 'enable_natpmp', _('Enable PCP/NAT-PMP protocols')).default = '1';
 
                o = s.taboption('setup', form.Flag, 'igdv1', _('UPnP IGDv1 compatibility mode'),
                        _('Advertise as IGDv1 (IPv4 only) device instead of IGDv2'));