fix FS#3659

Patrick Anders

2014-11-24 eebf5bbfb99ad3ee969b8a25b0d0e91f491911fd

 interface/web/mail/mail_transport_edit.php

@@ -70,6 +70,7 @@
   function onShowEnd() {
      global $app, $conf;

      $types = array('smtp' => 'smtp', 'uucp' => 'uucp', 'slow' => 'slow', 'error' => 'error', 'custom' => 'custom', '' => 'null');
      $tmp_parts = explode(":", $this->dataRecord["transport"]);
      if(!empty($this->id) && !stristr($this->dataRecord["transport"], ':')) {
         $rec["type"] = 'custom';
@@ -77,12 +78,14 @@
         if(empty($this->id) && empty($tmp_parts[0])) {
            $rec["type"] = 'smtp';
         } else {
            $rec["type"] = $tmp_parts[0];
            $rec["type"] = $types[$tmp_parts[0]] ? $tmp_parts[0] : 'custom';
         }
      }
      if(!empty($tmp_parts[2])) {
      if($rec["type"] == 'custom') {
         $dest = $this->dataRecord["transport"];
      } elseif(!empty($tmp_parts[2])) {
         $dest = @$tmp_parts[1].':'.@$tmp_parts[2];
      } elseif(!empty($tmp_parts[1])) {
      } elseif(!empty($tmp_parts[1]) || $this->dataRecord["transport"] == ":") {
         $dest = $tmp_parts[1];
      } else {
         $dest = $this->dataRecord["transport"];
@@ -95,7 +98,6 @@
         $rec["destination"] = @$dest;
      }

      $types = array('smtp' => 'smtp', 'uucp' => 'uucp', 'slow' => 'slow', 'error' => 'error', 'custom' => 'custom', '' => 'null');
      $type_select = '';
      if(is_array($types)) {
         foreach( $types as $key => $val) {
@@ -133,7 +135,7 @@
      // Check the client limits, if user is not the admin
      if($_SESSION["s"]["user"]["typ"] != 'admin') { // if user is not admin
         // Get the limits of the client
         $client_group_id = $_SESSION["s"]["user"]["default_group"];
         $client_group_id = $app->functions->intval($_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 transport.