Optimized UVL validation handling

diff --git a/i18n/english/luasrc/i18n/cbi.en.lua b/i18n/english/luasrc/i18n/cbi.en.lua
index 6b1168e2e24b9a544f9531e26b1a9ea2901ccdaf..14d630901ecba93f013fd0d4d8687777df4dd8a2 100644 (file)
--- a/i18n/english/luasrc/i18n/cbi.en.lua
+++ b/i18n/english/luasrc/i18n/cbi.en.lua
@@ -1,10 +1,10 @@
 cbi_add = [[Add entry]]
 cbi_del = [[Remove entry]]
-cbi_invalid = [[<strong>Error:</strong> Invalid input value]]
-cbi_invalid_section = [[<strong>Error:</strong> Validation failed]]
-cbi_missing = [[<strong>Error:</strong> This field is mandatory]]
-cbi_deperror = [[<strong>Error:</strong> At least one mandatory field has no or an invalid value]]
-cbi_reqerror = [[<strong>Error:</strong> A requirement for this option was not met]]
+cbi_invalid = [[Invalid input value]]
+cbi_invalid_section = [[<strong>Validation failed:</strong> Please check any input fields for mistakes.]]
+cbi_missing = [[This field is mandatory]]
+cbi_deperror = [[<strong>Validation failed:</strong> At least one mandatory field has no or an invalid value]]
+cbi_reqerror = [[A requirement for this option was not met]]
 cbi_addopt = [[-- Additional Field --]]
 cbi_optional = [[ (optional)]]
 cbi_sectempty = [[This section contains no values yet]]

diff --git a/i18n/english/luasrc/i18n/cbi.en.xml b/i18n/english/luasrc/i18n/cbi.en.xml
index 35599d87e854e96eece107e202907c05a2184a8e..89484e4aec643252327b4cc023cedcdcb77a2592 100644 (file)
--- a/i18n/english/luasrc/i18n/cbi.en.xml
+++ b/i18n/english/luasrc/i18n/cbi.en.xml
@@ -4,11 +4,11 @@
 
 <i18n:msg xml:id="cbi_add">Add entry</i18n:msg>
 <i18n:msg xml:id="cbi_del">Remove entry</i18n:msg>
-<i18n:msg xml:id="cbi_invalid"><strong>Error:</strong> Invalid input value</i18n:msg>
-<i18n:msg xml:id="cbi_invalid_section"><strong>Error:</strong> Validation failed</i18n:msg>
-<i18n:msg xml:id="cbi_missing"><strong>Error:</strong> This field is mandatory</i18n:msg>
-<i18n:msg xml:id="cbi_deperror"><strong>Error:</strong> At least one mandatory field has no or an invalid value</i18n:msg>
-<i18n:msg xml:id="cbi_reqerror"><strong>Error:</strong> A requirement for this option was not met</i18n:msg>
+<i18n:msg xml:id="cbi_invalid">Invalid input value</i18n:msg>
+<i18n:msg xml:id="cbi_invalid_section"><strong>Validation failed:</strong> Please check any input fields for mistakes.</i18n:msg>
+<i18n:msg xml:id="cbi_missing">This field is mandatory</i18n:msg>
+<i18n:msg xml:id="cbi_deperror"><strong>Validation failed:</strong> At least one mandatory field has no or an invalid value</i18n:msg>
+<i18n:msg xml:id="cbi_reqerror">A requirement for this option was not met</i18n:msg>
 <i18n:msg xml:id="cbi_addopt">-- Additional Field --</i18n:msg>
 <i18n:msg xml:id="cbi_optional"> (optional)</i18n:msg>
 <i18n:msg xml:id="cbi_sectempty">This section contains no values yet</i18n:msg>

diff --git a/i18n/german/luasrc/i18n/cbi.de.lua b/i18n/german/luasrc/i18n/cbi.de.lua
index 075ed834abbfc63221d85bc6df25e13281e25f93..968d31cff9910a55029a77518ed60136d9804ffe 100644 (file)
--- a/i18n/german/luasrc/i18n/cbi.de.lua
+++ b/i18n/german/luasrc/i18n/cbi.de.lua
@@ -1,10 +1,10 @@
 cbi_add = [[Eintrag hinzufügen]]
 cbi_del = [[Eintrag entfernen]]
-cbi_invalid = [[Fehler: Ungültige Eingabe]]
-cbi_invalid_section = [[<strong>Fehler:</strong> Validierung fehlgeschlagen]]
-cbi_missing = [[Fehler: Dieses Feld muss ausgefüllt werden]]
-cbi_deperror = [[<strong>Fehler:</strong> Mindestens ein benötigtes Feld ist leer oder hat einen ungültigen Wert]]
-cbi_reqerror = [[<strong>Fehler:</strong> Nicht erfüllte Abhängigkeit für dieses Feld]]
+cbi_invalid = [[Ungültige Eingabe]]
+cbi_invalid_section = [[<strong>Validierung fehlgeschlagen:</strong> Bitte die Eingabefelder auf Fehler überprüfen.]]
+cbi_missing = [[Dieses Feld muss ausgefüllt werden]]
+cbi_deperror = [[<strong>Validierung fehlgeschlagen:</strong> Mindestens ein benötigtes Feld ist leer oder hat einen ungültigen Wert]]
+cbi_reqerror = [[Nicht erfüllte Abhängigkeit für dieses Feld]]
 cbi_addopt = [[-- Zusätzliches Feld --]]
 cbi_sectempty = [[Diese Sektion enthält noch keine Einträge]]
 cbi_manual = [[-- benutzerdefiniert --]]

diff --git a/i18n/german/luasrc/i18n/cbi.de.xml b/i18n/german/luasrc/i18n/cbi.de.xml
index e1ecd2df6d4c4ccb1659a63074f01d09a7693c7f..dd37f2f49a886a017e6c41027a25698a81ae00d0 100644 (file)
--- a/i18n/german/luasrc/i18n/cbi.de.xml
+++ b/i18n/german/luasrc/i18n/cbi.de.xml
@@ -4,11 +4,11 @@
 
 <i18n:msg xml:id="cbi_add">Eintrag hinzufügen</i18n:msg>
 <i18n:msg xml:id="cbi_del">Eintrag entfernen</i18n:msg>
-<i18n:msg xml:id="cbi_invalid">Fehler: Ungültige Eingabe</i18n:msg>
-<i18n:msg xml:id="cbi_invalid_section"><strong>Fehler:</strong> Validierung fehlgeschlagen</i18n:msg>
-<i18n:msg xml:id="cbi_missing">Fehler: Dieses Feld muss ausgefüllt werden</i18n:msg>
-<i18n:msg xml:id="cbi_deperror"><strong>Fehler:</strong> Mindestens ein benötigtes Feld ist leer oder hat einen ungültigen Wert</i18n:msg>
-<i18n:msg xml:id="cbi_reqerror"><strong>Fehler:</strong> Nicht erfüllte Abhängigkeit für dieses Feld</i18n:msg>
+<i18n:msg xml:id="cbi_invalid">Ungültige Eingabe</i18n:msg>
+<i18n:msg xml:id="cbi_invalid_section"><strong>Validierung fehlgeschlagen:</strong> Bitte die Eingabefelder auf Fehler überprüfen.</i18n:msg>
+<i18n:msg xml:id="cbi_missing">Dieses Feld muss ausgefüllt werden</i18n:msg>
+<i18n:msg xml:id="cbi_deperror"><strong>Validierung fehlgeschlagen:</strong> Mindestens ein benötigtes Feld ist leer oder hat einen ungültigen Wert</i18n:msg>
+<i18n:msg xml:id="cbi_reqerror">Nicht erfüllte Abhängigkeit für dieses Feld</i18n:msg>
 <i18n:msg xml:id="cbi_addopt">-- Zusätzliches Feld --</i18n:msg>
 <i18n:msg xml:id="cbi_sectempty">Diese Sektion enthält noch keine Einträge</i18n:msg>
 <i18n:msg xml:id="cbi_manual">-- benutzerdefiniert --</i18n:msg>

diff --git a/libs/cbi/luasrc/cbi.lua b/libs/cbi/luasrc/cbi.lua
index 9c5501d69b5810bbd93a9c0e2bac10dcfbb0acd5..6740141acc35c69f9078bdde1e53173a8714336f 100644 (file)
--- a/libs/cbi/luasrc/cbi.lua
+++ b/libs/cbi/luasrc/cbi.lua
@@ -78,8 +78,37 @@ function load(cbimap, ...)
        return maps
 end
 
+local function _uvl_validate_section(node, name)
+       local co = node.map:get()
+       luci.uvl.STRICT_UNKNOWN_OPTIONS = false
+       local stat, err = node.map.validator:validate_section(node.config, name, co)
+       if err then
+               node.map.save = false
+               if err.code == luci.uvl.errors.ERR_DEPENDENCY then
+                       node.tag_deperror[name] = true
+               else
+                       node.tag_invalid[name] = true
+               end
+               for i, v in ipairs(err.childs) do
+                       if v.option and node.fields[v.option] then
+                               if v.code == luci.uvl.errors.ERR_OPTION then
+                                       local subcode = v.childs and v.childs[1] and v.childs[1].code
+                                       if subcode == luci.uvl.errors.ERR_DEPENDENCY then
+                                               node.fields[v.option].tag_reqerror[name] = true
+                                       elseif subcode == luci.uvl.errors.ERR_OPT_REQUIRED then
+                                               node.fields[v.option].tag_missing[name] = true
+                                               node.tag_deperror[name] = true
+                                       else
+                                               node.fields[v.option].tag_invalid[name] = true
+                                       end
+                               end
+                       end
+               end
+       end
+
+end
 
-function _uvl_strip_remote_dependencies(deps)
+local function _uvl_strip_remote_dependencies(deps)
        local clean = {}
 
        for k, v in pairs(deps) do
@@ -626,25 +655,7 @@ function NamedSection.parse(self)
                        Node.parse(self, s)
                        
                        if not self.override_scheme and self.map.scheme then
-                               local co = self.map:get()
-                               local stat, err = self.map.validator:validate_section(self.config, s, co)
-                               if err then
-                                       --self.map.save = false
-                                       if err.code == luci.uvl.errors.ERR_DEPENDENCY then
-                                               self.tag_deperror[s] = true
-                                       else
-                                               self.tag_invalid[s] = true
-                                       end
-                                       for i, v in ipairs(err.childs) do
-                                               if v.option and self.fields[v.option] then
-                                                       if v.code == luci.uvl.errors.ERR_DEPENDENCY then
-                                                               self.fields[v.option].tag_reqerror[s] = true
-                                                       elseif v.code == luci.uvl.errors.ERR_OPTION then
-                                                               self.fields[v.option].tag_invalid[s] = true
-                                                       end
-                                               end
-                                       end
-                               end
+                               _uvl_validate_section(self, s)
                        end
                end
                AbstractSection.parse_optionals(self, s)
@@ -716,25 +727,7 @@ function TypedSection.parse(self)
                        Node.parse(self, k)
                        
                        if not self.override_scheme and self.map.scheme then
-                               local co = self.map:get()
-                               local stat, err = self.map.validator:validate_section(self.config, k, co)
-                               if err then
-                                       --self.map.save = false
-                                       if err.code == luci.uvl.errors.ERR_DEPENDENCY then
-                                               self.tag_deperror[k] = true
-                                       else
-                                               self.tag_invalid[k] = true
-                                       end
-                                       for i, v in ipairs(err.childs) do
-                                               if v.option and self.fields[v.option] then
-                                                       if v.code == luci.uvl.errors.ERR_DEPENDENCY then
-                                                               self.fields[v.option].tag_reqerror[k] = true
-                                                       elseif v.code == luci.uvl.errors.ERR_OPTION then
-                                                               self.fields[v.option].tag_invalid[k] = true
-                                                       end
-                                               end
-                                       end
-                               end
+                               _uvl_validate_section(self, k)
                        end
                end
                AbstractSection.parse_optionals(self, k)