From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Wed, 29 Oct 2008 09:15:33 +0000 (+0000)
Subject: Updated Post handling for ACLs.
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=717279bff579021ce037544e92d276731bf1a6b3;p=gosa.git

Updated Post handling for ACLs.
-Deselected ACLs were not always deselected, if no acl was left checked.


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

diff --git a/gosa-core/include/class_acl.inc b/gosa-core/include/class_acl.inc
index 0ddb62128..193b9b519 100644
--- a/gosa-core/include/class_acl.inc
+++ b/gosa-core/include/class_acl.inc
@@ -306,54 +306,49 @@ class acl extends plugin
         continue;
       }
 
-      /* ACL saving... */
-      if (preg_match('/^acl_.*_[^xy]$/', $name)){
-        $aclDialog= TRUE;
-        list($dummy, $object, $attribute, $value)= split('_', $name);
-
-        /* Skip for detection entry */
-        if ($object == 'dummy') {
-          continue;
-        }
-
-        /* Ordinary ACLs */
-        if (!isset($new_acl[$object])){
-          $new_acl[$object]= array();
-        }
-        if (isset($new_acl[$object][$attribute])){
-          $new_acl[$object][$attribute].= $value;
-        } else {
-          $new_acl[$object][$attribute]= $value;
-        }
-      }
-
       if(isset($_POST['selected_role'])){
         $this->aclContents = "";
         $this->aclContents = base64_decode($_POST['selected_role']);
       }
     }
 
-    if($this->acl_is_writeable("")){
-      
-      /* Only be interested in new acl's, if we're in the right _POST place */
-      if ($aclDialog && $this->aclObject != "" && is_array($this->ocMapping[$this->aclObject])){
-
-        foreach ($this->ocMapping[$this->aclObject] as $oc){
 
-          if(isset($this->aclContents[$oc]) && is_array($this->aclContents)){
-            unset($this->aclContents[$oc]);
-            unset($this->aclContents[$this->aclObject.'/'.$oc]);
-          }else{
-#          trigger_error("Huhm?");
-          }
-          if (isset($new_acl[$oc]) && is_array($new_acl)){
-            $this->aclContents[$oc]= $new_acl[$oc];
+    /* Check Posts 
+     */
+    if(isset($_POST['acl_dummy_0_0_0'])){
+      $aclDialog= TRUE;
+      $tmp= session::get('plist');
+      $plist= $tmp->info;
+      $list = $this->ocMapping[$this->aclObject];
+      foreach($list as $plugin){
+        foreach(array("r","w","c","d","s","m") as $acl){
+          $overall = "acl_{$this->aclObject}/{$plugin}_0_{$acl}";
+          if(isset($_POST[$overall])){
+            if(!isset($new_acl["{$this->aclObject}/{$plugin}"][0])) {
+              $new_acl["{$this->aclObject}/{$plugin}"][0] = "";
+            }
+            $new_acl["{$this->aclObject}/{$plugin}"][0] .= $acl;
           }
-          if (isset($new_acl[$this->aclObject.'/'.$oc]) && is_array($new_acl)){
-            $this->aclContents[$this->aclObject.'/'.$oc]= $new_acl[$this->aclObject.'/'.$oc];
+          if(!isset($plist[$plugin]) || !is_array($plist[$plugin])) continue;
+          foreach($plist[$plugin]['plProvidedAcls'] as $attribute => $description){
+            $aclstr = "acl_{$this->aclObject}/{$plugin}_{$attribute}_{$acl}";
+            if(isset($_POST[$aclstr])){
+              if(!isset($new_acl["{$this->aclObject}/{$plugin}"][$attribute])) {
+                $new_acl["{$this->aclObject}/{$plugin}"][$attribute] = "";
+              }
+              $new_acl["{$this->aclObject}/{$plugin}"][$attribute] .= $acl;
+            }
           }
         }
       }
+    }
+
+    if($this->acl_is_writeable("")){
+      
+      /* Only be interested in new acl's, if we're in the right _POST place */
+      if ($aclDialog && $this->aclObject != "" && is_array($this->ocMapping[$this->aclObject])){
+        $this->aclContents = $new_acl;
+      }
 
       /* Save new acl in case of base edit mode */
       if ($this->aclType == 'base' && !$firstedit){
@@ -658,6 +653,8 @@ class acl extends plugin
 
   function buildAclSelector($list)
   {
+    /* Do not change the dummy, it is used to detect whether the dialog was opnened or not
+     */
     $display= "<input type='hidden' name='acl_dummy_0_0_0' value='1'>";
     $cols= 3;
     $tmp= session::get('plist');
@@ -702,11 +699,10 @@ class acl extends plugin
     foreach ($list as $key => $name){
 
       /* Create sub acl if it does not exist */
-      if (!isset($this->aclContents[$key])){
-        $this->aclContents[$key]= array();
-        $this->aclContents[$key][0]= '';
+      $currentAcl = array("0" => "");
+      if (isset($this->aclContents[$key])){
+        $currentAcl= $this->aclContents[$key];
       }
-      $currentAcl= $this->aclContents[$key];
 
       /* Get the overall plugin acls 
        */