diff options
| author | Jo-Philipp Wich <jo@mein.io> | 2019-02-12 08:32:02 +0100 |
|---|---|---|
| committer | Jo-Philipp Wich <jo@mein.io> | 2019-02-12 08:32:02 +0100 |
| commit | eda8f02dac3caa4d0f52cd1e860d7a392c295df3 (patch) | |
| tree | d4d740fedc7f485b7ab3702f62c6cb47cbfb0444 /applications/luci-app-banip/luasrc/view | |
| parent | 4141243762aafb7960d67f871c97907307005f87 (diff) | |
treewide: avoid double-escaping CBI section labels
Since the section labels are already HTML-escaped implicitely by the
striptags() function, we must not escape them again in attr() or
ifattr().
Fixes: #2524
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Diffstat (limited to 'applications/luci-app-banip/luasrc/view')
| -rw-r--r-- | applications/luci-app-banip/luasrc/view/banip/sourcelist.htm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/applications/luci-app-banip/luasrc/view/banip/sourcelist.htm b/applications/luci-app-banip/luasrc/view/banip/sourcelist.htm index 743886f884..12240e5ae4 100644 --- a/applications/luci-app-banip/luasrc/view/banip/sourcelist.htm +++ b/applications/luci-app-banip/luasrc/view/banip/sourcelist.htm @@ -27,7 +27,7 @@ local anonclass = (not self.anonymous or self.sectiontitle) and "named" or "ano for i, k in ipairs(self:cfgsections()) do section = k local sectionname = striptags((type(self.sectiontitle) == "function") and self:sectiontitle(section) or k) - local sectiontitle = ifattr(sectionname and (not self.anonymous or self.sectiontitle), "data-title", sectionname) + local sectiontitle = ifattr(sectionname and (not self.anonymous or self.sectiontitle), "data-title", sectionname, true) isempty = false scope = { valueheader = "cbi/cell_valueheader", valuefooter = "cbi/cell_valuefooter" } -%> |