[ postfixadmin-Feature Requests-2054502 ] Allow adding mailboxes and aliases to non-active domains

[SourceForge.net <no...@so...>]

Feature Requests item #2054502, was opened at 2008-08-16 11:27
Message generated for change (Comment added) made by christian_boltz
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=937967&aid=2054502&group_id=191583

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: shario (shario)
Assigned to: Nobody/Anonymous (nobody)
Summary: Allow adding mailboxes and aliases to non-active domains

Initial Comment:
When configuring a new domain it is usually better to create the domain, the mailboxes in advance and after everything is ready, activate the domain. However, Postfixadmin does not allow adding mailboxes and aliases to non-active domains.

This should be allowed, easy change.


----------------------------------------------------------------------

>Comment By: Christian Boltz (christian_boltz)
Date: 2008-08-16 14:09

Message:
Logged In: YES 
user_id=593261
Originator: NO

Good point, creating mailboxes and aliases on disabled domains should be
allowed. (To be exact: Postfixadmin "just" didn't display the "create ..."
links ;-)  

Thanks for the report and the patch. Commited to SVN r447.

----------------------------------------------------------------------

Comment By: shario (shario)
Date: 2008-08-16 11:37

Message:
Logged In: YES 
user_id=1660028
Originator: YES

Here's a diff.

--- list-virtual.php.old        2008-08-16 12:33:48.000000000 +0300
+++ list-virtual.php    2008-08-16 12:33:02.000000000 +0300
@@ -146,7 +146,6 @@
    }

    $active = $limit['active'];
-   if($active == 't' || $active == 1) {
       $backup_mx = $limit['backupmx'];
       if($backup_mx == 'f' || $backup_mx == 0) {
          if($limit['aliases'] == 0) {
@@ -162,7 +161,6 @@
             $tCanAddMailbox = true;
          }
       }
-   }
 }

 // this is why we need a proper template layer.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=937967&aid=2054502&group_id=191583