luci-base: fix field section add button/input field names
authorJo-Philipp Wich <jo@mein.io>
Mon, 16 Jul 2018 15:28:50 +0000 (17:28 +0200)
committerJo-Philipp Wich <jo@mein.io>
Mon, 16 Jul 2018 15:35:07 +0000 (17:35 +0200)
commit4da137e8928f936bcab30104d74e4d77d6e64629
tree5f24c8ab104b6c809d690e63dc9b39a7acb931d6
parent911219898f5e65e8bb867a8195812550cff583d5
luci-base: fix field section add button/input field names

Some CBI map models, mainly the Network -> VLAN page, expect a valid
previous section ID in their Section:create() callback.

Previous refactoring of the tblsection markup broke this behaviour as
the "section" loop variable was accidentally localized, causing it to
be undefined outside of the loop body which caused the section add
button and name input fields to get rendered with a wrong "name"
attribute.

Fix this by moving the "section" variable declaration out of the loop
and by readding references to it in the non-anonymous section add case.

Fixes FS#1657
Fixes 002c4d1d5 ("luci-base: add "Name" label to autogenerated title column")
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
(cherry picked from commit fcc9cd6a9ab0bd7eaff6b3f3b4dd96fa31f79ebb)
modules/luci-base/luasrc/view/cbi/tblsection.htm