diff --git a/interface/web/mail/lib/lang/en_mail_transport.lng b/interface/web/mail/lib/lang/en_mail_transport.lng index 530d1e20bf9dcd1de3eeb7da0c8e65266268aa93..e86a465ea69ee4a7a294d9a3a5915eb3e6bef93c 100644 --- a/interface/web/mail/lib/lang/en_mail_transport.lng +++ b/interface/web/mail/lib/lang/en_mail_transport.lng @@ -1,7 +1,9 @@ <?php $wb["server_id_txt"] = 'Server'; $wb["domain_txt"] = 'Domain'; -$wb["transport_txt"] = 'Transport'; +$wb["destination_txt"] = 'Destination'; +$wb["type_txt"] = 'Type'; +$wb["mx_txt"] = 'No MX lookup'; $wb["sort_order_txt"] = 'Sort by'; $wb["active_txt"] = 'Active'; $wb["btn_save_txt"] = 'Save'; diff --git a/interface/web/mail/mail_transport_edit.php b/interface/web/mail/mail_transport_edit.php index 161321d31edc9c0cf383c5bbd8dfe6d0e79ae236..6ee7f056c16467607817de72b8590de2f23b866d 100644 --- a/interface/web/mail/mail_transport_edit.php +++ b/interface/web/mail/mail_transport_edit.php @@ -1,6 +1,6 @@ <?php /* -Copyright (c) 2005, Till Brehm, projektfarm Gmbh +Copyright (c) 2007, Till Brehm, projektfarm Gmbh All rights reserved. Redistribution and use in source and binary forms, with or without modification, @@ -42,7 +42,7 @@ require_once('../../lib/config.inc.php'); require_once('../../lib/app.inc.php'); // Checking module permissions -if(!stristr($_SESSION["s"]["user"]["modules"],$_SESSION["s"]["module"]["name"])) { +if(!stristr($_SESSION["s"]["user"]["modules"],'mail')) { header("Location: ../index.php"); exit; } @@ -64,7 +64,7 @@ class page_action extends tform_actions { $client_group_id = $_SESSION["s"]["user"]["default_group"]; $client = $app->db->queryOneRecord("SELECT limit_mailrouting FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = $client_group_id"); - // Check if the user may add another mailbox. + // Check if the user may add another transport. if($client["limit_mailrouting"] >= 0) { $tmp = $app->db->queryOneRecord("SELECT count(transport_id) as number FROM mail_transport WHERE sys_groupid = $client_group_id"); if($tmp["number"] >= $client["limit_mailrouting"]) { @@ -76,6 +76,39 @@ class page_action extends tform_actions { parent::onShowNew(); } + function onShowEnd() { + global $app, $conf; + + $tmp_parts = explode(":",$this->dataRecord["transport"]); + if(empty($this->id) && empty($tmp_parts[0])) { + $rec["type"] = 'smtp'; + } else { + $rec["type"] = $tmp_parts[0]; + } + if(@substr($tmp_parts[1],0,1) == '[') { + $rec["mx"] = 'checked="CHECKED"'; + $rec["destination"] = @substr($tmp_parts[1],1,-1); + } else { + $rec["mx"] = ''; + $rec["destination"] = @$tmp_parts[1]; + } + + $types = array('smtp' => 'smtp','uucp' => 'uucp','slow' => 'slow', 'error' => 'error', '' => 'null'); + $type_select = ''; + if(is_array($types)) { + foreach( $types as $key => $val) { + $selected = ($key == $rec["type"])?'SELECTED':''; + $type_select .= "<option value='$key' $selected>$val</option>\r\n"; + } + } + $rec["type"] = $type_select; + $app->tpl->setVar($rec); + unset($type); + unset($types); + + parent::onShowEnd(); + } + function onSubmit() { global $app, $conf; @@ -85,7 +118,7 @@ class page_action extends tform_actions { $client_group_id = $_SESSION["s"]["user"]["default_group"]; $client = $app->db->queryOneRecord("SELECT limit_mailrouting FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = $client_group_id"); - // Check if the user may add another mailbox. + // Check if the user may add another transport. if($this->id == 0 && $client["limit_mailrouting"] >= 0) { $tmp = $app->db->queryOneRecord("SELECT count(transport_id) as number FROM mail_transport WHERE sys_groupid = $client_group_id"); if($tmp["number"] >= $client["limit_mailrouting"]) { @@ -95,6 +128,17 @@ class page_action extends tform_actions { } } // end if user is not admin + //* Compose transport field + if($this->dataRecord["mx"] == 'y') { + $transport = '['.$this->dataRecord["destination"].']'; + } else { + $transport = $this->dataRecord["destination"]; + } + $this->dataRecord["transport"] = $this->dataRecord["type"].':'.$transport; + unset($this->dataRecord["type"]); + unset($this->dataRecord["mx"]); + unset($this->dataRecord["destination"]); + parent::onSubmit(); } diff --git a/interface/web/mail/templates/mail_transport_edit.htm b/interface/web/mail/templates/mail_transport_edit.htm index 693ba7ca597a282ea44134270329e8f36939c1e4..57417882e83c24e76fb492f747931ff099d318c5 100644 --- a/interface/web/mail/templates/mail_transport_edit.htm +++ b/interface/web/mail/templates/mail_transport_edit.htm @@ -12,8 +12,19 @@ <td class="frmText11"><input name="domain" type="text" class="text" value="{tmpl_var name='domain'}" size="30" maxlength="255"></td> </tr> <tr> - <td class="frmText11">{tmpl_var name='transport_txt'}:</td> - <td class="frmText11"><input name="transport" type="text" class="text" value="{tmpl_var name='transport'}" size="30" maxlength="255"></td> + <td class="frmText11">{tmpl_var name='type_txt'}:</td> + <td class="frmText11"> + <select name="type" class="text"> + {tmpl_var name='type'} + </select></td> + </tr> + <tr> + <td class="frmText11">{tmpl_var name='mx_txt'}:</td> + <td class="frmText11"><input name="mx" value="y" {tmpl_var name='mx'} type="checkbox"></td> + </tr> + <tr> + <td class="frmText11">{tmpl_var name='destination_txt'}:</td> + <td class="frmText11"><input name="destination" type="text" class="text" value="{tmpl_var name='destination'}" size="30" maxlength="255"></td> </tr> <tr> <td class="frmText11">{tmpl_var name='sort_order_txt'}:</td>