X-Git-Url: https://git.archive.openwrt.org/?p=project%2Fluci.git;a=blobdiff_plain;f=libs%2Fuvl%2Fluasrc%2Fuvl%2Ferrors.lua;h=9cd8d19cfae641966a0ad586d664963c5da2d4e4;hp=c547030a82a554d8af0eac60308056d37094372e;hb=de80e2a708653e825434753a1b48559648942a3b;hpb=4cfd1d25d16aaac1f2e0556a7cc589f0e5200669

diff --git a/libs/uvl/luasrc/uvl/errors.lua b/libs/uvl/luasrc/uvl/errors.lua
index c547030a8..9cd8d19cf 100644
--- a/libs/uvl/luasrc/uvl/errors.lua
+++ b/libs/uvl/luasrc/uvl/errors.lua
@@ -14,11 +14,19 @@ $Id$
 
 ]]--
 
-module( "luci.uvl.errors", package.seeall )
+local uci = require "luci.model.uci"
+local uvl = require "luci.uvl"
+local util = require "luci.util"
+local string = require "string"
 
-require("luci.util")
+local ipairs, error, type = ipairs, error, type 
+local tonumber, unpack = tonumber, unpack
 
 
+local luci = luci
+
+module "luci.uvl.errors"
+
 ERRCODES = {
 	{ 'UCILOAD', 		'Unable to load config "%p": %1' },
 
@@ -53,11 +61,12 @@ ERRCODES = {
 
 	{ 'OPT_UNKNOWN',	'Option "%i" (%I) not found in scheme' },
 	{ 'OPT_REQUIRED',	'Required option "%i" has no value' },
-	{ 'OPT_BADVALUE', 	'Value "%v" of option "%i" is not defined in enum %1' },
-	{ 'OPT_INVVALUE',	'Value "%v" of given option "%i" does not validate as datatype "%1"' },
+	{ 'OPT_BADVALUE', 	'Value "%1" of option "%i" is not defined in enum %2' },
+	{ 'OPT_INVVALUE',	'Value "%1" of option "%i" does not validate as datatype "%2"' },
 	{ 'OPT_NOTLIST',	'Option "%i" is defined as list but stored as plain value' },
 	{ 'OPT_DATATYPE',	'Option "%i" has unknown datatype "%1"' },
 	{ 'OPT_NOTFOUND',	'Option "%p.%s.%o" not found in config' },
+	{ 'OPT_RANGE',		'Option "%p.%s.%o" is not within the specified range' },
 
 	{ 'DEP_NOTEQUAL',	'Dependency (%1) failed:\nOption "%i" is not eqal "%2"' },
 	{ 'DEP_NOVALUE',	'Dependency (%1) failed:\nOption "%i" has no value' },
@@ -68,11 +77,11 @@ ERRCODES = {
 
 -- build error constants and instance constructors
 for i, v in ipairs(ERRCODES) do
-	luci.uvl.errors[v[1]] = function(...)
+	_M[v[1]] = function(...)
 		return error(i, ...)
 	end
 
-	luci.uvl.errors['ERR_'..v[1]] = i
+	_M['ERR_'..v[1]] = i
 end
 
 
@@ -85,14 +94,14 @@ function i18n(key, def)
 end
 
 
-error = luci.util.class()
+error = util.class()
 
 function error.__init__(self, code, pso, args)
 
 	self.code = code
 	self.args = ( type(args) == "table" and args or { args } )
 
-	if luci.util.instanceof( pso, luci.uvl.uvlitem ) then
+	if util.instanceof( pso, uvl.uvlitem ) then
 		self.stype = pso.sref[2]
 		self.package, self.section, self.option, self.value = unpack(pso.cref)
 		self.object = pso
@@ -101,7 +110,7 @@ function error.__init__(self, code, pso, args)
 		pso = ( type(pso) == "table" and pso or { pso } )
 
 		if pso[2] then
-			local uci = luci.model.uci.cursor()
+			local uci = uci.cursor()
 			self.stype = uci:get(pso[1], pso[2]) or pso[2]
 		end
 
@@ -113,7 +122,7 @@ function error.child(self, err)
 	if not self.childs then
 		self.childs = { err }
 	else
-		table.insert( self.childs, err )
+		self.childs[#self.childs+1] = err
 	end
 	return self
 end
@@ -128,14 +137,14 @@ function error.string(self,pad)
 		:gsub("\n", "\n"..pad)
 		:gsub("%%i", self:cid())
 		:gsub("%%I", self:sid())
-		:gsub("%%p", self.package or '<nil>')
-		:gsub("%%s", self.section or '<nil>')
-		:gsub("%%S", self.stype   or '<nil>')
-		:gsub("%%o", self.option  or '<nil>')
-		:gsub("%%v", self.value   or '<nil>')
-		:gsub("%%t", self.object and self.object:type()  or '<nil>' )
-		:gsub("%%T", self.object and self.object:title() or '<nil>' )
-		:gsub("%%([1-9])", function(n) return self.args[tonumber(n)] or '<nil>' end)
+		:gsub("%%p", self.package or '(nil)')
+		:gsub("%%s", self.section or '(nil)')
+		:gsub("%%S", self.stype   or '(nil)')
+		:gsub("%%o", self.option  or '(nil)')
+		:gsub("%%v", self.value   or '(nil)')
+		:gsub("%%t", self.object and self.object:type()  or '(nil)' )
+		:gsub("%%T", self.object and self.object:title() or '(nil)' )
+		:gsub("%%([1-9])", function(n) return self.args[tonumber(n)] or '(nil)' end)
 		:gsub("%%c",
 			function()
 				local s = ""