From c807814b50bd2fe9c925bdd9601d3080943515b0 Mon Sep 17 00:00:00 2001
From: cajus <cajus@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Tue, 16 May 2006 07:28:34 +0000
Subject: [PATCH] Fixed a set of errors

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@3357 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 include/class_MultiSelectWindow.inc                      | 6 +++++-
 include/class_mail-methods-cyrus.inc                     | 2 +-
 plugins/admin/departments/class_departmentManagement.inc | 5 +++--
 plugins/personal/mail/class_mailAccount.inc              | 4 +++-
 4 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/include/class_MultiSelectWindow.inc b/include/class_MultiSelectWindow.inc
index 8e9f0feda..cee65eec4 100644
--- a/include/class_MultiSelectWindow.inc
+++ b/include/class_MultiSelectWindow.inc
@@ -370,7 +370,11 @@ class MultiSelectWindow{
 			   this is posted by the parent class MultiSelectWindow */
 			if(isset($_GET['act'])&& ($_GET['act'] == "dep_open")){
 				$s_entry = base64_decode($_GET['dep_id']);
-				$this->selectedBase = $this->config->departments[trim($s_entry)];
+        if (!isset($this->config->departments[$s_entry])){
+          print_red(_("Error: The requested subtree has an inconsistent DN encoding, check your LDAP!"));
+        } else {
+  				$this->selectedBase = $this->config->departments[$s_entry];
+        }
 			}
 
 			/* back to the roots ^^ */
diff --git a/include/class_mail-methods-cyrus.inc b/include/class_mail-methods-cyrus.inc
index 76e3908ea..a8e563b60 100644
--- a/include/class_mail-methods-cyrus.inc
+++ b/include/class_mail-methods-cyrus.inc
@@ -36,7 +36,7 @@ class mailMethodCyrus extends mailMethod
     /* Connect to IMAP server. I don't want to see these warning here... */
     $this->gosaMailServer= $gosaMailServer;
     if (!isset($this->config[$gosaMailServer])){
-      print_red(_("Warning: Account has an invalid mailserver string! Please check the mail server settings in the mail tab."));
+      print_red(_("Warning: Account has an invalid mailserver string!"));
     } else {
       $cfg= $this->config[$gosaMailServer];
     }
diff --git a/plugins/admin/departments/class_departmentManagement.inc b/plugins/admin/departments/class_departmentManagement.inc
index d77066f69..cdfea0c47 100644
--- a/plugins/admin/departments/class_departmentManagement.inc
+++ b/plugins/admin/departments/class_departmentManagement.inc
@@ -55,10 +55,11 @@ class departmentManagement extends plugin
 
   function execute()
   {
+    global $config;
+  
     /* Call parent execute */
     plugin::execute();
 
-
     /***************
       Var init 
      ***************/
@@ -68,7 +69,7 @@ class departmentManagement extends plugin
     /* Reload departments */
     $this->config->get_departments();
     $this->config->make_idepartments();
-    $_SESSION['config']= $this->config;
+    $config= $this->config;
 
     $smarty						= get_smarty();
     $display 					= "";
diff --git a/plugins/personal/mail/class_mailAccount.inc b/plugins/personal/mail/class_mailAccount.inc
index 08abd5fb1..29ab414ef 100644
--- a/plugins/personal/mail/class_mailAccount.inc
+++ b/plugins/personal/mail/class_mailAccount.inc
@@ -126,7 +126,9 @@ class mailAccount extends plugin
         }else{
           /* Could not connect to ldap.
            */
-          $this->gosaMailQuota = $this->attrs['gosaMailQuota'][0];
+          if (isset($this->attrs['gosaMailQuota'][0])){
+            $this->gosaMailQuota = $this->attrs['gosaMailQuota'][0];
+          }
         }
       }
     }
-- 
2.30.2