From e4a1a763d9ad60d77c71d2d86bee66a102715153 Mon Sep 17 00:00:00 2001
From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Wed, 4 Oct 2006 10:52:11 +0000
Subject: [PATCH] Fixed object removement

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@4852 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 plugins/admin/fai/class_faiManagement.inc | 50 ++++++++++++-----------
 1 file changed, 26 insertions(+), 24 deletions(-)

diff --git a/plugins/admin/fai/class_faiManagement.inc b/plugins/admin/fai/class_faiManagement.inc
index 6480f8d5e..63ffb647f 100644
--- a/plugins/admin/fai/class_faiManagement.inc
+++ b/plugins/admin/fai/class_faiManagement.inc
@@ -144,14 +144,14 @@ class faiManagement extends plugin
 
 		if ($s_action=="delete"){
 
+      /* Get 'dn' from posted termlinst */
+      $this->dn= $this->objects[$s_entry]['dn'];
+
 			/* Load permissions for selected 'dn' and check if
 			   we're allowed to remove this 'dn' */
       $acl  = $this->ui->get_permissions($this->dn,"fai/".$type_acl_mapping[$this->objects[$s_entry]['type']]);
       if(preg_match("/d/",$acl)){
 
-        /* Get 'dn' from posted termlinst */
-        $this->dn= $this->objects[$s_entry]['dn'];
-
 				/* Check locking, save current plugin in 'back_plugin', so the dialog knows where to return. */
 				if (($user= get_lock($this->dn)) != ""){
 					return(gen_locked_message ($user, $this->dn));
@@ -190,20 +190,22 @@ class faiManagement extends plugin
 
 			/* Some nice guy may send this as POST, so we've to check
 			   for the permissions again. */
-      $acl  = $this->ui->get_permissions($this->dn,"fai/".$type_acl_mapping[$this->objects[$s_entry]['type']]);
+
+      /* Find out more about the object type */
+      $ldap	  = $this->config->get_ldap_link();
+      $ldap->cat($this->dn, array('objectClass'));
+      $attrs	= $ldap->fetch();
+      $type	  = $this->get_type($attrs);			
+
+      $acl  = $this->ui->get_permissions($this->dn,"fai/".$type[1]);
       if(preg_match("/d/",$acl)){
 
-				/* Find out more about the object type */
-				$ldap	  = $this->config->get_ldap_link();
-				$ldap->cat($this->dn, array('objectClass'));
-				$attrs	= $ldap->fetch();
-				$type	  = $this->get_type($attrs);			
-				$this->dialog = new $type[0]($this->config,	$this->config->data['TABS'][$type[2]], $this->dn,"fai");
+        $this->dialog = new $type[0]($this->config,	$this->config->data['TABS'][$type[2]], $this->dn,"fai");
         $this->dialog->set_acl_base($this->dn);
-				$this->dialog->by_object[$type[1]]->remove_from_parent ();
-				unset ($this->dialog);
-				gosa_log ("FAI class '".$this->dn."' has been tagged as removed");
-				$this->dialog= NULL;
+        $this->dialog->by_object[$type[1]]->remove_from_parent ();
+        unset ($this->dialog);
+        gosa_log ("FAI class '".$this->dn."' has been tagged as removed");
+        $this->dialog= NULL;
         $to_del = clean_up_releases($this->dn);
         save_release_changes_now();
 
@@ -212,18 +214,18 @@ class faiManagement extends plugin
           gosa_log(sprintf(_("Release cleanup : Removing object (tagged as remvoed) that is no longer in use '%s'."),$dn));
         }
 
-			} else {
+      } else {
 
-				/* Normally this shouldn't be reached, send some extra
-				   logs to notify the administrator */
-				print_red (_("You are not allowed to delete this component!"));
-				gosa_log ("Warning: '".$this->ui->uid."' tried to trick system ".
-						"deletion.");
-			}
+        /* Normally this shouldn't be reached, send some extra
+           logs to notify the administrator */
+        print_red (_("You are not allowed to delete this component!"));
+        gosa_log ("Warning: '".$this->ui->uid."' tried to trick system ".
+            "deletion.");
+      }
 
-			/* Remove lock file after successfull deletion */
-			del_lock ($this->dn);
-		}
+      /* Remove lock file after successfull deletion */
+      del_lock ($this->dn);
+    }
 
 
     /****************
-- 
2.30.2