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:34:26 +0000 (17:34 +0200)
commitfcc9cd6a9ab0bd7eaff6b3f3b4dd96fa31f79ebb
treea6c4e3955c20372c6bae07a278294281574d09de
parent251c5100a976e9c4be76cfe32178a615e0a24202
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>
modules/luci-base/luasrc/view/cbi/tblsection.htm