diff options
author | Steven Barth <steven@midlink.org> | 2008-08-19 15:33:04 +0000 |
---|---|---|
committer | Steven Barth <steven@midlink.org> | 2008-08-19 15:33:04 +0000 |
commit | dc638500b580ed785810d472edbbb9b7b9f7dffd (patch) | |
tree | 27fe77b6ed34d732457549506f7252cb3401133d /libs/cbi | |
parent | 7e61238f9662b492285b07b2c47a5c64966784a5 (diff) |
libs/cbi: Moved typecasting to the appropriate place
Diffstat (limited to 'libs/cbi')
-rw-r--r-- | libs/cbi/luasrc/cbi.lua | 27 |
1 files changed, 15 insertions, 12 deletions
diff --git a/libs/cbi/luasrc/cbi.lua b/libs/cbi/luasrc/cbi.lua index 7a05d8922e..48a91d3b5a 100644 --- a/libs/cbi/luasrc/cbi.lua +++ b/libs/cbi/luasrc/cbi.lua @@ -367,7 +367,6 @@ function AbstractSection.__init__(self, map, sectiontype, ...) self.config = map.config self.optionals = {} self.defaults = {} - self.cast = "string" self.optional = true self.addremove = false @@ -447,16 +446,7 @@ end -- Returns the section's UCI table function AbstractSection.cfgvalue(self, section) - local value = self.map:get(section) - if not self.cast or self.cast == type(value) then - return value - elseif self.cast == "string" then - if type(value) == "table" then - return value[1] - end - elseif self.cast == "table" then - return {value} - end + return self.map:get(section) end -- Removes the section @@ -651,6 +641,9 @@ function TypedSection.parse(self) crval = REMOVE_PREFIX .. self.config name = luci.http.formvaluetable(crval) for k,v in pairs(name) do + luci.util.perror(k) + luci.util.perror(self:cfgvalue(k)) + luci.util.perror(self:checkscope(k)) if self:cfgvalue(k) and self:checkscope(k) then self:remove(k) end @@ -719,6 +712,7 @@ function AbstractValue.__init__(self, map, option, ...) self.tag_missing = {} self.tag_error = {} self.deps = {} + self.cast = "string" self.track_missing = false self.rmempty = false @@ -816,7 +810,16 @@ end -- Return the UCI value of this object function AbstractValue.cfgvalue(self, section) - return self.map:get(section, self.option) + local value = self.map:get(section, self.option) + if not self.cast or self.cast == type(value) then + return value + elseif self.cast == "string" then + if type(value) == "table" then + return value[1] + end + elseif self.cast == "table" then + return {value} + end end -- Validate the form value |