we want this in order toto support keep&forward for a real mailbox, creating an alias for a mailbox that already exists.
Patch against r562.
Hi, I think you've re-indented (beautification?) some of the code; although I'm happy in principal to merge it, I'd need to test it..... can someone else let meknow if the code works?
I reindented just the SQL queries I modified, to improve the readability.
Additionaly, I should mention that I didn't test it with PostgreSQL.
Hi, in the process of merging...
See changeset 572. There were two typos - one missing } and an unnecessary , in some SQL.
Unfortunately your patch caused some problems:
- it undermines the $CONF[*alias_control*] settings more or less - mailbox aliases with non-default targets are always shown in the "Aliases" section - see comment from 2009-05-04 on https://sourceforge.net/tracker/?func=detail&aid=1902476&group_id=191583&atid=937964
- it introduced some "funny" bugs - a nice example is http://sourceforge.net/tracker/?func=detail&aid=2786284&group_id=191583&atid=937964
Therefore we decided to revert most parts of this patch. (See https://sourceforge.net/mailarchive/forum.php?thread_name=E1M2Cvx-0006qQ-V2%40bj8yhf1.ch3.sourceforge.com&forum_name=postfixadmin-svn for details and what we kept, SVN r652)
For your needs, please set
$CONF['alias_control'] = 'YES';
$CONF['alias_control_admin'] = 'YES';
$CONF['special_alias_control'] = 'YES';
and you'll have an "Alias" link in the mailbox list to edit the alias.
The only missing part is that you can't see where the mailbox aliases point to. This is already tracked in
I hope you are not too disappointed that large parts of this patch were reverted.
If you have good arguments why your patch should be included, feel free to speak up. If your arguments are good enough, we'll consider to include this patch again.
Log in to post a comment.
Sign up for the SourceForge newsletter:
You seem to have CSS turned off.
Please don't fill out this field.