From 1f93194a9ae8da51d8285ee8f0adb516edbe6113 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20K=C3=BCthe?= Date: Sun, 30 Jul 2023 15:27:43 +0200 Subject: [PATCH] luci-app-lldpd: Update filter matrix MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The old filter matrix was based on the man pages, but the selected value in the range 1-20 was also assigned to the configuration and thus lldpd was configured incorrectly. lldpd expects a value in the range 0-19. The webapp matrix has now been changed so that the values 0-19 also appear there. This leads to a unification of configuration and webapp. Signed-off-by: Marek Küthe --- .../luci-app-lldpd/htdocs/luci-static/resources/lldpd.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/applications/luci-app-lldpd/htdocs/luci-static/resources/lldpd.js b/applications/luci-app-lldpd/htdocs/luci-static/resources/lldpd.js index d901457fa4c5..bc39ed50182a 100644 --- a/applications/luci-app-lldpd/htdocs/luci-static/resources/lldpd.js +++ b/applications/luci-app-lldpd/htdocs/luci-static/resources/lldpd.js @@ -124,20 +124,20 @@ var cbiFilterSelect = form.Value.extend({ tbody.push(E('tr', { 'class': ((selected == i) ? 'lldpd-filter-selected' : ''), 'click': L.bind(this.handleRowClick, this, section_id), - 'data-filter': i + 1, + 'data-filter': i, }, [ E('td', {}, [ E('input', { 'class': 'cbi-input-radio', 'data-update': 'click change', 'type': 'radio', - 'id': this.cbid(section_id) + '-' + (i + 1), + 'id': this.cbid(section_id) + '-' + i, 'name': this.cbid(section_id), 'checked': (selected == i) ? '' : null, - 'value': i + 1 + 'value': i }) ]), - E('td', {}, i + 1), + E('td', {}, i), E('td', {}, renderFilterVal(i, 0)), E('td', {}, renderFilterVal(i, 1)), E('td', {}, renderFilterVal(i, 2)),