From 2571a73a6948b4f3d442962e1ef9349f53480ff9 Mon Sep 17 00:00:00 2001
From: opensides <opensides@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Sun, 25 Sep 2005 19:26:11 +0000
Subject: [PATCH] still fixing some really bad coding ;-(

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@1411 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 .../personal/nagios/class_nagiosAccount.inc   | 75 ++++++++++++-------
 1 file changed, 46 insertions(+), 29 deletions(-)

diff --git a/plugins/personal/nagios/class_nagiosAccount.inc b/plugins/personal/nagios/class_nagiosAccount.inc
index a6ebedf56..6006e6e7b 100644
--- a/plugins/personal/nagios/class_nagiosAccount.inc
+++ b/plugins/personal/nagios/class_nagiosAccount.inc
@@ -109,6 +109,11 @@ class nagiosAccount extends plugin
     /* Call parents save to prepare $this->attrs */
 	  plugin::save();
 
+    /* Adapt attributes if needed */
+    $method= new $this->method($this->config);
+    $id= $method->uattrib;
+    $method->fixAttributesOnStore($this);
+                
 
 	  /* Write back to ldap */
 	  $ldap->cd($this->dn);
@@ -126,37 +131,49 @@ class nagiosAccount extends plugin
 	
   }
 
-function save_object()
-{
+  function check()
+  {
+  }
+  
+  function save_object()
+  {
 
-	if (!$this->is_account && ($_POST['nagiosAuth'] == "B" || $_POST['nagiosContact'] == "B")) {
-		$this->is_account= TRUE;
-	}
-	else {
-		$this->is_account= FALSE;
-	}
-	plugin::save_object();
-}
+    if (isset($_POST['mailTab'])){
+      /* Save ldap attributes */
+      plugin::save_object();
 
-function remove_from_parent()
-{
-	/* Cancel if there's nothing to do here */
-	if (!$this->initially_was_account){
-	return;
-	}
-	
-	plugin::remove_from_parent();
-	$ldap= $this->config->get_ldap_link();
-	
-	$ldap->cd($this->dn);
-	@DEBUG (DEBUG_LDAP, __LINE__, __FUNCTION__, __FILE__,
-	$this->attributes, "Save");
-	$ldap->modify($this->attrs);
-	show_ldap_error($ldap->get_error());
-	
-	/* Optionally execute a command after we're done */
-	$this->handle_post_events('remove');
-}
+  }
+
+  function remove_from_parent()
+  {
+	  /* Cancel if there's nothing to do here */
+	  if (!$this->initially_was_account){
+	    return;
+	  }
+
+    /* include global link_info */
+    $ldap= $this->config->get_ldap_link();
+
+    /* Remove and write to LDAP */
+    plugin::remove_from_parent();
+
+    /* Adapt attributes if needed */
+    $method= new $this->method($this->config);
+    $method->fixAttributesOnRemove($this);
+    
+
+    @DEBUG (DEBUG_LDAP, __LINE__, __FUNCTION__, __FILE__,
+      $this->attributes, "Save");
+    $ldap->cd($this->dn);
+    $ldap->modify($this->attrs);
+    show_ldap_error($ldap->get_error());
+                        
+    /* remove the entry from LDAP */
+    //unset ($this->attrs['uid']);
+                
+    /* Optionally execute a command after we're done */
+    $this->handle_post_events('remove');
+  }
 
 }
 
-- 
2.30.2