From 3f451371e33e251fea46e47a35a2cfa830be264e Mon Sep 17 00:00:00 2001 From: Christian Boltz Date: Sat, 4 Apr 2015 14:37:46 +0000 Subject: [PATCH] PFAHandler: - set(): if errormsg is set for a field, assume it's invalid (even if the validator functions did not (or forgot to) return False) In theory this should never happen, but it's a nice safety net against programming errors in validator functions that don't have an explicit return False; git-svn-id: https://svn.code.sf.net/p/postfixadmin/code/trunk@1768 a1433add-5e2c-0410-b055-b7f2511e0802 --- model/PFAHandler.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/model/PFAHandler.php b/model/PFAHandler.php index 61b0914a..c1e82e88 100644 --- a/model/PFAHandler.php +++ b/model/PFAHandler.php @@ -422,6 +422,8 @@ abstract class PFAHandler { if (!$this->{$func}($key, $values[$key])) $valid = false; } + if (isset($this->errormsg[$key]) && $this->errormsg[$key] != '') $valid = false; + if ($valid) { $this->values[$key] = $values[$key]; }