diff options
| author | Karl Palsson <karlp@remake.is> | 2015-09-02 11:24:29 +0000 |
|---|---|---|
| committer | Karl Palsson <karlp@remake.is> | 2015-09-02 15:27:55 +0000 |
| commit | 879d9ff2b8f80b7c8e4adadf0702154d9ae932f4 (patch) | |
| tree | 7c133cb360fd5cd60b3d94529eaa81513eef9a70 /modules | |
| parent | f0083abed10e8a397ebe9e62801e3b5d536ba7b6 (diff) | |
uci:set_list: Delete option if the list is empty
Allows lists fetched with get_list to be modified and simply passed back
to set_list. Explicitly calling set_list() with an empty list is clearly
requesting that there be zero list items, ie, deletion of the option
altogether.
Signed-off-by: Karl Palsson <karlp@remake.is>
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/luci-base/luasrc/model/uci.lua | 3 | ||||
| -rw-r--r-- | modules/luci-base/luasrc/model/uci.luadoc | 5 |
2 files changed, 6 insertions, 2 deletions
diff --git a/modules/luci-base/luasrc/model/uci.lua b/modules/luci-base/luasrc/model/uci.lua index 165913774..e9bac6ec0 100644 --- a/modules/luci-base/luasrc/model/uci.lua +++ b/modules/luci-base/luasrc/model/uci.lua @@ -139,6 +139,9 @@ end function Cursor.set_list(self, config, section, option, value) if config and section and option then + if not value or #value == 0 then + return self:delete(config, section, option) + end return self:set( config, section, option, ( type(value) == "table" and value or { value } ) diff --git a/modules/luci-base/luasrc/model/uci.luadoc b/modules/luci-base/luasrc/model/uci.luadoc index 80464f7ce..f3aa609d5 100644 --- a/modules/luci-base/luasrc/model/uci.luadoc +++ b/modules/luci-base/luasrc/model/uci.luadoc @@ -104,14 +104,15 @@ Get the given option from the first section with the given type. ]] ---[[ -Set given values as list. +Set given values as list. Setting a list option to an empty list +has the same effect as deleting the option. @class function @name Cursor.set_list @param config UCI config @param section UCI section name @param option UCI option -@param value UCI value +@param value value or table. Raw values will become a single item table. @return Boolean whether operation succeeded ]] |