diff options
-rw-r--r-- | i18n/english/luasrc/i18n/cbi.en.lua | 10 | ||||
-rw-r--r-- | i18n/english/luasrc/i18n/cbi.en.xml | 10 | ||||
-rw-r--r-- | i18n/german/luasrc/i18n/cbi.de.lua | 10 | ||||
-rw-r--r-- | i18n/german/luasrc/i18n/cbi.de.xml | 10 | ||||
-rw-r--r-- | libs/cbi/luasrc/cbi.lua | 71 |
5 files changed, 52 insertions, 59 deletions
diff --git a/i18n/english/luasrc/i18n/cbi.en.lua b/i18n/english/luasrc/i18n/cbi.en.lua index 6b1168e2e..14d630901 100644 --- 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 35599d87e..89484e4ae 100644 --- 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 075ed834a..968d31cff 100644 --- 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 e1ecd2df6..dd37f2f49 100644 --- 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 9c5501d69..6740141ac 100644 --- 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) |