[Tickets #3710] NEW: Get domain is invalid in some cases deleting virtual address

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=3710
-----------------------------------------------------------------------
 Ticket             | 3710
 Created By         | guilleva at gmail.com
 Summary            | Get domain is invalid in some cases deleting virtual address
 Queue              | Vilma
 State              | Unconfirmed
 Priority           | 1. Low
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


guilleva at gmail.com (2006-03-29 08:20) wrote:

The next code of virtuals/delete.php isn't fine when I delete some virtual
address which destination is a remote address.
line 21 to 28:

$vars = Variables::getDefaultVariables();
$domain = Vilma::getDomain();
$virtual_id = $vars->get('virtual_id');
$formname = $vars->get('formname');

$virtual = $vilma->getVirtual($virtual_id);
$domain = Vilma::stripDomain($virtual['virtual_destination']);
$domain = $vilma->getDomainByName($domain);


The line 27 must be $domain =
Vilma::stripDomain($virtual['virtual_email']);
because virtual_destination can be someuser at gmail.com

I don't know if the line 22 ("$domain = Vilma::getDomain();") have any sense
if after that, in
the line 27, $domain is overwrited.