diff --git a/model/MailboxHandler.php b/model/MailboxHandler.php index 38d1c594..b78cb339 100644 --- a/model/MailboxHandler.php +++ b/model/MailboxHandler.php @@ -239,17 +239,17 @@ protected function read_from_db_postprocess($db_result) protected function preSave(): bool { - if (isset($this->values['quota']) && $this->values['quota'] != -1 && is_numeric($this->values['quota'])) { - $multiplier = Config::read_string('quota_multiplier'); - if ($multiplier == 0 || !is_numeric($multiplier)) { // or empty string, or null, or false... - $multiplier = 1; + if (isset($this->values['quota'])) { + if ($this->values['quota'] != -1 && is_numeric($this->values['quota'])) { + $multiplier = Config::read_string('quota_multiplier'); + if ($multiplier == 0 || !is_numeric($multiplier)) { // or empty string, or null, or false... + $multiplier = 1; + } + $this->values['quota'] = $this->values['quota'] * $multiplier; # convert quota from MB to bytes + // Avoid trying to store '' in an integer field + if ($this->values['quota'] === '') { + $this->values['quota'] = 0; } - $this->values['quota'] = $this->values['quota'] * $multiplier; # convert quota from MB to bytes - } - - // Avoid trying to store '' in an integer field - if ($this->values['quota'] === '') { - $this->values['quota'] = 0; } $ah = new AliasHandler($this->new, $this->admin_username);