From: cajus <cajus@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Fri, 28 Mar 2008 14:27:17 +0000 (+0000)
Subject: Dialog unifying
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=c91e30faebe6f0dd5c4513cbbf5737cf556383a3;p=gosa.git

Dialog unifying


git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@10062 594d385d-05f5-0310-b6e9-bd551577e9d8
---

diff --git a/gosa-plugins/mail/personal/mail/class_mailAccount.inc b/gosa-plugins/mail/personal/mail/class_mailAccount.inc
index 6a5c47c2f..b4a2eae80 100644
--- a/gosa-plugins/mail/personal/mail/class_mailAccount.inc
+++ b/gosa-plugins/mail/personal/mail/class_mailAccount.inc
@@ -327,21 +327,26 @@ class mailAccount extends plugin
     /* Finished adding of locals? */
     if (isset($_POST['add_locals_finish'])){
 
-      /* Check if we are able to write gosaMailForwardingAddress */
-      if($this->acl_is_writeable("gosaMailForwardingAddress")){
+      if (isset($_POST['local_list'])){
 
-        /* Walk through list of forwarders, ignore own addresses */
-        foreach ($_POST['local_list'] as $val){
-          if (!in_array ($val, $this->gosaMailAlternateAddress) &&
-              $val != $this->mail){
+        /* Check if we are able to write gosaMailForwardingAddress */
+        if($this->acl_is_writeable("gosaMailForwardingAddress")){
 
-            $this->addForwarder($val);
-            $this->is_modified= TRUE;
+          /* Walk through list of forwarders, ignore own addresses */
+          foreach ($_POST['local_list'] as $val){
+            if (!in_array ($val, $this->gosaMailAlternateAddress) &&
+                $val != $this->mail){
+
+              $this->addForwarder($val);
+              $this->is_modified= TRUE;
+            }
           }
         }
+        $this->forward_dialog= FALSE;
+        $this->dialog= FALSE;
+      } else {
+        msg_dialog::display(_("Error"), _("Please select an entry!"), ERROR_DIALOG);
       }
-      $this->forward_dialog= FALSE;
-      $this->dialog= FALSE;
     }
 
     /* Add forward email addresses */
diff --git a/gosa-plugins/mail/personal/mail/mail_locals.tpl b/gosa-plugins/mail/personal/mail/mail_locals.tpl
index a06a98e08..686c7395f 100644
--- a/gosa-plugins/mail/personal/mail/mail_locals.tpl
+++ b/gosa-plugins/mail/personal/mail/mail_locals.tpl
@@ -1,6 +1,6 @@
 <table summary="" style="width:100%; vertical-align:top; text-align:left;" cellpadding=4>
   <tr>
-    <td style="vertical-align:top;width:600px">
+    <td style="vertical-align:top;">
       <div class="contentboxh" style="height:20px;">
         <p class="contentboxh" style="font-size:12px">
           <b>
@@ -12,13 +12,13 @@
       </div>
       <div class="contentboxb">
         <p class="contentboxb" style="border-top:1px solid #B0B0B0;background-color:#F8F8F8">
-          <select id="local_list" style="width:600px; margin-top:4px; height:450px;" name="local_list[]" size="15" multiple>
+          <select id="local_list" style="width:100%; margin-top:4px; height:450px;" name="local_list[]" size="15" multiple>
         	  {html_options options=$mailusers}
           </select>
         </p>
       </div>
     </td>
-    <td style="vertical-align:top;">
+    <td style="vertical-align:top;width:350px">
       <div class="contentboxh" style="height:20px;">
         <p class="contentboxh" style="font-size:12px">
           <img src="{$launchimage}" align="right" alt="[F]">