From 0c8e4714b85485253eac88c27596700e56d66d9e Mon Sep 17 00:00:00 2001
From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Wed, 15 Sep 2010 09:00:23 +0000
Subject: [PATCH] Simplified attribute mapping

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@19676 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 .../personal/groupware/class_groupware.inc    | 52 +++++++++++--------
 1 file changed, 29 insertions(+), 23 deletions(-)

diff --git a/gosa-plugins/groupware/personal/groupware/class_groupware.inc b/gosa-plugins/groupware/personal/groupware/class_groupware.inc
index 7a4fa3965..2739d02e6 100644
--- a/gosa-plugins/groupware/personal/groupware/class_groupware.inc
+++ b/gosa-plugins/groupware/personal/groupware/class_groupware.inc
@@ -815,29 +815,35 @@ class groupware extends plugin
      */
     function mapComprehensiveUserData($callBackMap)
     {
-        //dropdown
-        $this->mailLocations = $callBackMap["mailLocations"];
-        //data from the groupware
-        $this->mailAddress = $callBackMap["primaryMail"];
-        $this->mailLocation = $callBackMap["mailLocation"];
-        $this->quotaUsage = $callBackMap["quotaUsage"];
-        $this->quotaSize = $callBackMap["quotaSize"];
-        $this->alternateAddresses = $callBackMap["alternateAddresses"];
-        $this->forwardingAddresses = $callBackMap["forwardingAddresses"];
-        $this->vacationEnabled = $callBackMap["vacationEnabled"];
-        $this->vacationStart = $callBackMap["vacationStart"];
-        $this->vacationStop = $callBackMap["vacationStop"];
-        $this->vacationMessage = $callBackMap["vacationMessage"];
-        $this->mailBoxWarnLimitEnabled = $callBackMap["mailBoxWarnLimitEnabled"];
-        $this->mailBoxWarnLimitValue = $callBackMap["mailBoxWarnLimitValue"];
-        $this->mailBoxSendSizelimitEnabled = $callBackMap["mailBoxSendSizelimitEnabled"];
-        $this->mailBoxSendSizelimitValue = $callBackMap["mailBoxSendSizelimitValue"];
-        $this->mailBoxHardSizelimitEnabled = $callBackMap["mailBoxHardSizelimitEnabled"];
-        $this->mailBoxHardSizelimitValue = $callBackMap["mailBoxHardSizelimitValue"];
-        $this->mailBoxAutomaticRemovalEnabled = $callBackMap["mailBoxAutomaticRemovalEnabled"];
-        $this->mailBoxAutomaticRemovalValue = $callBackMap["mailBoxAutomaticRemovalValue"];
-        $this->localDeliveryOnly = $callBackMap["localDeliveryOnly"];
-        $this->dropOwnMails = $callBackMap["dropOwnMails"];
+        $map = array(
+                "mailLocations" => "mailLocations",
+                "mailAddress" => "primaryMail",
+                "mailLocation" => "mailLocation",
+                "quotaUsage" => "quotaUsage",
+                "quotaSize" => "quotaSize",
+                "alternateAddresses" => "alternateAddresses",
+                "forwardingAddresses" => "forwardingAddresses",
+                "vacationEnabled" => "vacationEnabled",
+                "vacationStart" => "vacationStart",
+                "vacationStop" => "vacationStop",
+                "vacationMessage" => "vacationMessage",
+                "mailBoxWarnLimitEnabled" => "mailBoxWarnLimitEnabled",
+                "mailBoxWarnLimitValue" => "mailBoxWarnLimitValue",
+                "mailBoxSendSizelimitEnabled" => "mailBoxSendSizelimitEnabled",
+                "mailBoxSendSizelimitValue" => "mailBoxSendSizelimitValue",
+                "mailBoxHardSizelimitEnabled" => "mailBoxHardSizelimitEnabled",
+                "mailBoxHardSizelimitValue" => "mailBoxHardSizelimitValue",
+                "mailBoxAutomaticRemovalEnabled" => "mailBoxAutomaticRemovalEnabled",
+                "mailBoxAutomaticRemovalValue" => "mailBoxAutomaticRemovalValue",
+                "localDeliveryOnly" => "localDeliveryOnly",
+                "dropOwnMails" => "dropOwnMails");
+
+        // Map values from source array to class members
+        foreach($map as $target => $source){
+            if(isset($callBackMap[$source])){
+                $this->$target = $callBackMap[$source];
+            }
+        }
     }
 }
 
-- 
2.30.2