From 7acef043f2231a6f06967a5eea48a55c95fea6cf Mon Sep 17 00:00:00 2001 From: tbrehm Date: Sun, 2 Aug 2009 06:07:15 +0000 Subject: [PATCH] Check only for active records when performing a duplicate check for mailboxes / aliases / forwarders --- interface/web/mail/mail_alias_edit.php | 4 ++-- interface/web/mail/mail_forward_edit.php | 4 ++-- interface/web/mail/mail_user_edit.php | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/interface/web/mail/mail_alias_edit.php b/interface/web/mail/mail_alias_edit.php index b6ce17f116..ead9bc277c 100644 --- a/interface/web/mail/mail_alias_edit.php +++ b/interface/web/mail/mail_alias_edit.php @@ -120,8 +120,8 @@ class page_action extends tform_actions { unset($this->dataRecord["email_local_part"]); unset($this->dataRecord["email_domain"]); - //* Check if there is no mailbox with this address - $tmp = $app->db->queryOneRecord("SELECT count(mailuser_id) as number FROM mail_user WHERE email = '".$app->db->quote($this->dataRecord["source"])."'"); + //* Check if there is no active mailbox with this address + $tmp = $app->db->queryOneRecord("SELECT count(mailuser_id) as number FROM mail_user WHERE postfix = 'y' AND email = '".$app->db->quote($this->dataRecord["source"])."'"); if($tmp['number'] > 0) $app->tform->errorMessage .= $app->tform->lng("duplicate_mailbox_txt")."
"; unset($tmp); diff --git a/interface/web/mail/mail_forward_edit.php b/interface/web/mail/mail_forward_edit.php index f89cd3606f..33f86f5d6e 100644 --- a/interface/web/mail/mail_forward_edit.php +++ b/interface/web/mail/mail_forward_edit.php @@ -117,8 +117,8 @@ class page_action extends tform_actions { unset($this->dataRecord["email_local_part"]); unset($this->dataRecord["email_domain"]); - //* Check if there is no mailbox with this address - $tmp = $app->db->queryOneRecord("SELECT count(mailuser_id) as number FROM mail_user WHERE email = '".$app->db->quote($this->dataRecord["source"])."'"); + //* Check if there is no active mailbox with this address + $tmp = $app->db->queryOneRecord("SELECT count(mailuser_id) as number FROM mail_user WHERE postfix = 'y' AND email = '".$app->db->quote($this->dataRecord["source"])."'"); if($tmp['number'] > 0) $app->tform->errorMessage .= $app->tform->lng("duplicate_mailbox_txt")."
"; unset($tmp); diff --git a/interface/web/mail/mail_user_edit.php b/interface/web/mail/mail_user_edit.php index 9289eeba25..05ddeebb59 100644 --- a/interface/web/mail/mail_user_edit.php +++ b/interface/web/mail/mail_user_edit.php @@ -182,7 +182,7 @@ class page_action extends tform_actions { $this->dataRecord["gid"] = $mail_config["mailuser_gid"]; //* Check if there is no alias or forward with this address - $tmp = $app->db->queryOneRecord("SELECT count(forwarding_id) as number FROM mail_forwarding WHERE source = '".$app->db->quote($this->dataRecord["email"])."'"); + $tmp = $app->db->queryOneRecord("SELECT count(forwarding_id) as number FROM mail_forwarding WHERE active = 'y' AND source = '".$app->db->quote($this->dataRecord["email"])."'"); if($tmp['number'] > 0) $app->tform->errorMessage .= $app->tform->lng("duplicate_alias_or_forward_txt")."
"; unset($tmp); -- GitLab