From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Thu, 29 Jul 2010 12:14:24 +0000 (+0000)
Subject: Updated post handling
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=8b065f710f206551102b42ca21e649c4762d9067;p=gosa.git

Updated post handling

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

diff --git a/gosa-plugins/goto/admin/ogroups/goto/class_termgroup.inc b/gosa-plugins/goto/admin/ogroups/goto/class_termgroup.inc
index f1d484396..1a4ff8c7c 100644
--- a/gosa-plugins/goto/admin/ogroups/goto/class_termgroup.inc
+++ b/gosa-plugins/goto/admin/ogroups/goto/class_termgroup.inc
@@ -2,443 +2,443 @@
 
 class termgroup extends plugin
 {
-  /* attribute list for save action */
-  var $members= array();
-
-  /* Stores a list of the member dn's */
-  var $member_dn = array();
-
-  var $gotoMode           = "locked";
-  var $gotoSyslogServer   = "";
-  var $gotoNtpServer      = array();
-  var $gotoTerminalPath   = array();
-  var $gotoSwapServer     = array();
-  var $modes              = array();
-  var $inheritTimeServer  = true;
-  var $is_account		      = true; 
-  var $orig_dn            = "";
-  var $didAction          = FALSE;
-  var $mapActions   = array("halt"            => "halt",
-                            "reboot"          => "reboot",
-                            "update"          => "update",
-                            "reinstall"       => "reinstall",
-                            "rescan"          => "rescan",
-                            "wake"            => "wakeup",
-			   # These are currently not supported by the tftp daemon
-                           # "memcheck"        => "memcheck",
-                           # "sysinfo"         => "sysinfo"
-			   );
-
-  var $attributes     = array("gotoMode","gotoSyslogServer", "gotoNtpServer", "gotoTerminalPath", "gotoSwapServer");
-  var $objectclasses  = array("gotoWorkstationTemplate");
-  var $CopyPasteVars  = array("gotoNtpServers","modes","inheritTimeServer","members");
-  var $view_logged    = FALSE;
-  var $nfsservers     = array();
-  var $swapservers    = array();
-  var $member_of_ogroup= false;
-  var $members_inherit_from_group = false;
-
-  function termgroup (&$config, $dn= NULL, $parent= NULL)
-  {
-    /***************
-      Some  initialisations
-     ***************/
-
-    plugin::plugin($config, $dn, $parent);
-    $ldap= $config->get_ldap_link();
-
-    $this->is_account = true;
-    $this->modes["active"]= _("Activated");
-    $this->modes["locked"]= _("Locked");
-//    $this->modes["memcheck"]= _("Memory test");
-//    $this->modes["sysinfo"]= _("System analysis");
-
-    $this->orig_dn =	$this->dn;
-
-    /*************** 
-      Get mac addresses from member objects  
-     ***************/
-
-    /* We're only interested in the terminal members here, evaluate
-       these... */
-    if(isset($this->attrs['member'])){
-      for ($i= 0; $i<$this->attrs['member']['count']; $i++){
-        $member= $this->attrs['member'][$i];
-        array_push($this->member_dn, $member);
-        $ldap->cat($member, array('objectClass', 'macAddress', 'cn'));
-        if ($ldap->success()){
-          $attrs = $ldap->fetch();
-          if (in_array("gotoTerminal", $attrs['objectClass']) ||
-              in_array("gotoWorkstation", $attrs['objectClass'])){
-            if (isset($attrs['macAddress'])){
-              $this->members[$attrs['cn'][0]]= $attrs['macAddress'][0];
-            } else {
-              $this->members[$attrs['cn'][0]]= "";
+    /* attribute list for save action */
+    var $members= array();
+
+    /* Stores a list of the member dn's */
+    var $member_dn = array();
+
+    var $gotoMode           = "locked";
+    var $gotoSyslogServer   = "";
+    var $gotoNtpServer      = array();
+    var $gotoTerminalPath   = array();
+    var $gotoSwapServer     = array();
+    var $modes              = array();
+    var $inheritTimeServer  = true;
+    var $is_account		      = true; 
+    var $orig_dn            = "";
+    var $didAction          = FALSE;
+    var $mapActions   = array("halt"            => "halt",
+            "reboot"          => "reboot",
+            "update"          => "update",
+            "reinstall"       => "reinstall",
+            "rescan"          => "rescan",
+            "wake"            => "wakeup",
+            );
+
+    var $attributes     = array("gotoMode","gotoSyslogServer", "gotoNtpServer", "gotoTerminalPath", "gotoSwapServer");
+    var $objectclasses  = array("gotoWorkstationTemplate");
+    var $CopyPasteVars  = array("gotoNtpServers","modes","inheritTimeServer","members");
+    var $view_logged    = FALSE;
+    var $nfsservers     = array();
+    var $swapservers    = array();
+    var $member_of_ogroup= false;
+    var $members_inherit_from_group = false;
+
+    function termgroup (&$config, $dn= NULL, $parent= NULL)
+    {
+        /***************
+          Some  initialisations
+         ***************/
+
+        plugin::plugin($config, $dn, $parent);
+        $ldap= $config->get_ldap_link();
+
+        $this->is_account = true;
+        $this->modes["active"]= _("Activated");
+        $this->modes["locked"]= _("Locked");
+        //    $this->modes["memcheck"]= _("Memory test");
+        //    $this->modes["sysinfo"]= _("System analysis");
+
+        $this->orig_dn =	$this->dn;
+
+        /*************** 
+          Get mac addresses from member objects  
+         ***************/
+
+        /* We're only interested in the terminal members here, evaluate
+           these... */
+        if(isset($this->attrs['member'])){
+            for ($i= 0; $i<$this->attrs['member']['count']; $i++){
+                $member= $this->attrs['member'][$i];
+                array_push($this->member_dn, $member);
+                $ldap->cat($member, array('objectClass', 'macAddress', 'cn'));
+                if ($ldap->success()){
+                    $attrs = $ldap->fetch();
+                    if (in_array("gotoTerminal", $attrs['objectClass']) ||
+                            in_array("gotoWorkstation", $attrs['objectClass'])){
+                        if (isset($attrs['macAddress'])){
+                            $this->members[$attrs['cn'][0]]= $attrs['macAddress'][0];
+                        } else {
+                            $this->members[$attrs['cn'][0]]= "";
+                        }
+                    }
+                }
             }
-          }
         }
-      }
-    }
 
-    /*************** 
-      Perpare NTP settings 
-     ***************/
-
-    /* Create used ntp server array */
-    $this->gotoNtpServer= array();
-    if(isset($this->attrs['gotoNtpServer'])){
-      $this->inheritTimeServer = false;
-      unset($this->attrs['gotoNtpServer']['count']);
-      foreach($this->attrs['gotoNtpServer'] as $server){
-        $this->gotoNtpServer[$server] = $server;
-      }
-    }
+        /*************** 
+          Perpare NTP settings 
+         ***************/
+
+        /* Create used ntp server array */
+        $this->gotoNtpServer= array();
+        if(isset($this->attrs['gotoNtpServer'])){
+            $this->inheritTimeServer = false;
+            unset($this->attrs['gotoNtpServer']['count']);
+            foreach($this->attrs['gotoNtpServer'] as $server){
+                $this->gotoNtpServer[$server] = $server;
+            }
+        }
 
-    /* Get Share servers */
-    $tmp2 = array();
-    $tmp2['!']= _("Local swap");
-    foreach($this->config->data['SERVERS']['NBD'] as $server){
-      if($server != "default"){
-        $tmp2[$server]= $server;
-      }else{
-        if($this->member_of_ogroup){
-          $tmp2[$server]="["._("inherited")."]";
+        /* Get Share servers */
+        $tmp2 = array();
+        $tmp2['!']= _("Local swap");
+        foreach($this->config->data['SERVERS']['NBD'] as $server){
+            if($server != "default"){
+                $tmp2[$server]= $server;
+            }else{
+                if($this->member_of_ogroup){
+                    $tmp2[$server]="["._("inherited")."]";
+                }
+            }
         }
-      }
-    }
-    $this->swapservers= $tmp2;
-
-    $tmp2 = array();
-    foreach($this->config->data['SERVERS']['NFS'] as $server){
-      if($server != "default"){
-        $tmp2[$server]= $server;
-      }else{
-        if($this->member_of_ogroup){
-          $tmp2[$server]="["._("inherited")."]";
+        $this->swapservers= $tmp2;
+
+        $tmp2 = array();
+        foreach($this->config->data['SERVERS']['NFS'] as $server){
+            if($server != "default"){
+                $tmp2[$server]= $server;
+            }else{
+                if($this->member_of_ogroup){
+                    $tmp2[$server]="["._("inherited")."]";
+                }
+            }
         }
-      }
-    }
-    $this->nfsservers= $tmp2;
+        $this->nfsservers= $tmp2;
 
-    /* Set inherit checkbox state */
-    if(in_array("default",$this->gotoNtpServer)){
-      $this->inheritTimeServer = true;
-      $this->gotoNtpServer=array();
-    }
+        /* Set inherit checkbox state */
+        if(in_array("default",$this->gotoNtpServer)){
+            $this->inheritTimeServer = true;
+            $this->gotoNtpServer=array();
+        }
 
-    /* Create available ntp options */
-    $this->gotoNtpServers = $this->config->data['SERVERS']['NTP'];
-    foreach($this->gotoNtpServers as $key => $server){
-      if($server == "default"){
-        unset($this->gotoNtpServers[$key]);
-      }
+        /* Create available ntp options */
+        $this->gotoNtpServers = $this->config->data['SERVERS']['NTP'];
+        foreach($this->gotoNtpServers as $key => $server){
+            if($server == "default"){
+                unset($this->gotoNtpServers[$key]);
+            }
+        }
     }
-  }
 
-  function check()
-  {
-     /* Call common method to give check the hook */
-    $message= plugin::check();
+    function check()
+    {
+        /* Call common method to give check the hook */
+        $message= plugin::check();
 
-    if (!$this->acl_is_createable() && $this->dn == "new"){
-      $message[]= msgPool::permCreate();
-    }
+        if (!$this->acl_is_createable() && $this->dn == "new"){
+            $message[]= msgPool::permCreate();
+        }
 
-    /* Check for valid ntpServer selection */
-    if((!$this->inheritTimeServer) && (!count($this->gotoNtpServer))){
-      $message[]= msgPool::required(_("NTP server"));
+        /* Check for valid ntpServer selection */
+        if((!$this->inheritTimeServer) && (!count($this->gotoNtpServer))){
+            $message[]= msgPool::required(_("NTP server"));
+        }
+        return($message);
     }
-    return($message);
-  }
-
-  function remove_from_parent()
-  {
-    /* Workstation startup is using gotoWorkstationTemplate too,
-        if we remove this oc all other not manged attributes will cause errors */
-    if(isset($this->attrs['gotoKernelParameters'])){
-      $this->objectclasses = array();
+
+    function remove_from_parent()
+    {
+        /* Workstation startup is using gotoWorkstationTemplate too,
+           if we remove this oc all other not manged attributes will cause errors */
+        if(isset($this->attrs['gotoKernelParameters'])){
+            $this->objectclasses = array();
+        }
+
+        /* Remove acc */
+        plugin::remove_from_parent();
+        $ldap = $this->config->get_ldap_link();
+        $ldap->cd($this->orig_dn);
+        $ldap->modify($this->attrs);
+        $this->handle_post_events("remove");
+        new log("remove","ogroups/".get_class($this),$this->dn,array_keys($this->attrs),$ldap->get_error());
     }
 
-    /* Remove acc */
-    plugin::remove_from_parent();
-    $ldap = $this->config->get_ldap_link();
-    $ldap->cd($this->orig_dn);
-    $ldap->modify($this->attrs);
-    $this->handle_post_events("remove");
-    new log("remove","ogroups/".get_class($this),$this->dn,array_keys($this->attrs),$ldap->get_error());
-  }
 
 
+    function execute()
+    {
+        /* Call parent execute */
+        plugin::execute();
 
-  function execute()
-  {
-    /* Call parent execute */
-    plugin::execute();
+        if(!$this->view_logged){
+            $this->view_logged = TRUE;
+            new log("view","ogroups/".get_class($this),$this->dn);
+        }
 
-    if(!$this->view_logged){
-      $this->view_logged = TRUE;
-      new log("view","ogroups/".get_class($this),$this->dn);
-    }
 
+        /*************** 
+          Handle requested action
+         ***************/
+
+        /* Watch for events */
+        if ((isset($_POST['action'])) && ($this->acl_is_writeable("FAIstate")) && isset($this->mapActions[$_POST['saction']]) ){
+
+            /* Check if we have an DaemonEvent for this action */
+            $action = $this->mapActions[get_post('saction')];
+            if(class_available("DaemonEvent")){
+                $events = DaemonEvent::get_event_types(SYSTEM_EVENT | HIDDEN_EVENT);
+                $macaddresses= array();
+                foreach ($this->members as $cn => $macAddress){
+                    $macaddresses[]= $macAddress;
+                }
+
+                if(isset($events['TRIGGERED']["DaemonEvent_".$action])){
+                    $evt = $events['TRIGGERED']["DaemonEvent_".$action];
+                    $tmp = new $evt['CLASS_NAME']($this->config);
+                    $tmp->add_targets($macaddresses);
+                    $tmp->set_type(TRIGGERED_EVENT);
+                    $o_queue = new gosaSupportDaemon();
+                    if(!$o_queue->append($tmp)){
+                        msg_dialog::display(_("Service infrastructure"),msgPool::siError($o_queue->get_error()),ERROR_DIALOG);
+                    }
+                }
+            } else {
+                msg_dialog::display(_("Event error"),
+                        sprintf(_("Event '%s' is not available!"),$action),ERROR_DIALOG);
+            }
+        }
 
-    /*************** 
-      Handle requested action
-     ***************/
 
-    /* Watch for events */
-    if ((isset($_POST['action'])) && ($this->acl_is_writeable("FAIstate")) && isset($this->mapActions[$_POST['saction']]) ){
+        /*************** 
+          Add remove NTP server
+         ***************/
 
-      /* Check if we have an DaemonEvent for this action */
-      $action = $this->mapActions[$_POST['saction']];
-      if(class_available("DaemonEvent")){
-        $events = DaemonEvent::get_event_types(SYSTEM_EVENT | HIDDEN_EVENT);
-        $macaddresses= array();
-        foreach ($this->members as $cn => $macAddress){
-          $macaddresses[]= $macAddress;
+        /* Add new ntp Server to our list */
+        if((isset($_POST['addNtpServer'])) && (isset($_POST['gotoNtpServers']))){
+            $this->gotoNtpServer[get_post('gotoNtpServers')] = get_post('gotoNtpServers');
         }
 
-        if(isset($events['TRIGGERED']["DaemonEvent_".$action])){
-          $evt = $events['TRIGGERED']["DaemonEvent_".$action];
-          $tmp = new $evt['CLASS_NAME']($this->config);
-          $tmp->add_targets($macaddresses);
-          $tmp->set_type(TRIGGERED_EVENT);
-          $o_queue = new gosaSupportDaemon();
-          if(!$o_queue->append($tmp)){
-            msg_dialog::display(_("Service infrastructure"),msgPool::siError($o_queue->get_error()),ERROR_DIALOG);
-          }
+        /* Delete selected NtpServer for list of used servers  */
+        if((isset($_POST['delNtpServer'])) && (isset($_POST['gotoNtpServerSelected']))){
+            foreach(get_post('gotoNtpServerSelected') as $name){
+                unset($this->gotoNtpServer[$name]);
+            }
         }
-      } else {
-        msg_dialog::display(_("Event error"),
-            sprintf(_("Event '%s' is not available!"),$action),ERROR_DIALOG);
-      }
-    }
 
 
-    /*************** 
-      Add remove NTP server
-     ***************/
+        /*************** 
+          Prepare smarty 
+         ***************/
 
-    /* Add new ntp Server to our list */
-    if((isset($_POST['addNtpServer'])) && (isset($_POST['gotoNtpServers']))){
-      $this->gotoNtpServer[$_POST['gotoNtpServers']] = $_POST['gotoNtpServers'];
-    }
+        /* Set government mode */
+        $smarty= get_smarty();
 
-    /* Delete selected NtpServer for list of used servers  */
-    if((isset($_POST['delNtpServer'])) && (isset($_POST['gotoNtpServerSelected']))){
-      foreach($_POST['gotoNtpServerSelected'] as $name){
-        unset($this->gotoNtpServer[$name]);
-      }
-    }
+        if (isset($this->parent->by_name['termstartup'])){
+            $smarty->assign("is_termgroup", "1");
+        } else {
+            $smarty->assign("is_termgroup", "0");
+        }
 
+        $tmp = $this->plInfo();
+        foreach($tmp['plProvidedAcls'] as $name => $translated) {
+            $smarty->assign($name."ACL",$this->getacl($name));
+        }
 
-    /*************** 
-      Prepare smarty 
-     ***************/
+        foreach($this->attributes as $attr){
+            $smarty->assign($attr,set_post($this->$attr));
+        }
 
-    /* Set government mode */
-    $smarty= get_smarty();
+        /* Variables */
+        foreach(array("gotoMode","gotoNtpServer") as $val){
+            $smarty->assign($val."_select", set_post($this->$val));
+        }
 
-    if (isset($this->parent->by_name['termstartup'])){
-    	$smarty->assign("is_termgroup", "1");
-    } else {
-    	$smarty->assign("is_termgroup", "0");
-    }
+        $smarty->assign("actions", 
+                set_post(array(
+                        "halt" => _("Switch off"), 
+                        "reboot" => _("Reboot"),
+                        "instant_update" => _("Instant update"),
+                        "update" => _("Scheduled update"),
+                        "wake" => _("Wake"),
+                        "reinstall" => _("Reinstall"),
+                        "rescan" => _("Rescan hardware"),
+                        "memcheck" => _("Memory test"),
+                        "sysinfo"  => _("System analysis"))));
+
+        $smarty->assign("inheritTimeServer",$this->inheritTimeServer);
+        $smarty->assign("modes", set_post($this->modes));
+
+        $tmp = array();
+        foreach($this->gotoNtpServers as $server){
+            if(!in_array($server,$this->gotoNtpServer)){
+                $tmp[$server] = $server;
+            }
+        }
 
-    $tmp = $this->plInfo();
-    foreach($tmp['plProvidedAcls'] as $name => $translated) {
-      $smarty->assign($name."ACL",$this->getacl($name));
-    }
+        $smarty->assign("gotoNtpServers", set_post($tmp)); 
 
-    foreach($this->attributes as $attr){
-      $smarty->assign($attr,      $this->$attr);
-    }
+        $smarty->assign("nfsservers",    set_post($this->nfsservers));
+        $smarty->assign("swapservers",   set_post($this->swapservers));
 
-    /* Variables */
-    foreach(array("gotoMode","gotoNtpServer") as $val){
-      $smarty->assign($val."_select", $this->$val);
-    }
+        $smarty->assign("syslogservers", set_post($this->config->data['SERVERS']['SYSLOG'])); 
+        $smarty->assign("gotoSyslogServer_select", set_post($this->gotoSyslogServer)); 
+
+        /* Variables */
+        foreach(array("gotoTerminalPath", "gotoSwapServer") as $val){
+            $smarty->assign($val."_select", set_post($this->$val));
+        }
 
-    $smarty->assign("actions", array("halt" => _("Switch off"), "reboot" => _("Reboot"),
-          "instant_update" => _("Instant update"),
-          "update" => _("Scheduled update"),
-          "wake" => _("Wake"),
-          "reinstall" => _("Reinstall"),
-          "rescan" => _("Rescan hardware"),
-          "memcheck" => _("Memory test"),
-          "sysinfo"  => _("System analysis")));
-
-    $smarty->assign("inheritTimeServer",$this->inheritTimeServer);
-    $smarty->assign("modes", $this->modes);
-
-    $tmp = array();
-    foreach($this->gotoNtpServers as $server){
-      if(!in_array($server,$this->gotoNtpServer)){
-        $tmp[$server] = $server;
-      }
+
+        /* Show main page */
+        $smarty->assign("members_inherit_from_group", $this->members_inherit_from_group);
+        return ($smarty->fetch (get_template_path('termgroup.tpl', TRUE, dirname(__FILE__))));
     }
 
-    $smarty->assign("gotoNtpServers",$tmp); 
 
-    $smarty->assign("nfsservers",     $this->nfsservers);
-    $smarty->assign("swapservers",    $this->swapservers);
+    function save_object()
+    {
+        plugin::save_object();  
+        /* Set inherit mode */
+        if(isset($_POST['workgeneric_posted'])){
 
-    $smarty->assign("syslogservers", $this->config->data['SERVERS']['SYSLOG']); 
-    $smarty->assign("gotoSyslogServer_select", $this->gotoSyslogServer); 
+            /* Handle the inherit to members button */
+            $this->members_inherit_from_group = isset($_POST['members_inherit_from_group']);
 
-    /* Variables */
-    foreach(array("gotoTerminalPath", "gotoSwapServer") as $val){
-	    $smarty->assign($val."_select", $this->$val);
+            if(isset($_POST["inheritTimeServer"])){
+                $this->inheritTimeServer = true;
+            }else{
+                $this->inheritTimeServer = false;
+            }
+        }
     }
 
+    /* Save to LDAP */
+    function save()
+    {
+        if (isset($this->parent->by_name['termstartup'])){
+            $this->objectclasses= array("gotoTerminalTemplate");
+        } else {
+            $this->objectclasses= array("gotoWorkstationTemplate");
+        }
 
-    /* Show main page */
-    $smarty->assign("members_inherit_from_group", $this->members_inherit_from_group);
-    return ($smarty->fetch (get_template_path('termgroup.tpl', TRUE, dirname(__FILE__))));
-  }
+        plugin::save();
 
+        /* Tell members to inherit all attributes if the flag is set  */
+        $this->inherit_attributes_to_members();
 
-  function save_object()
-  {
-    plugin::save_object();  
-    /* Set inherit mode */
-    if(isset($_POST['workgeneric_posted'])){
+        /***************
+          Prepare special vars 
+         ***************/
 
-      /* Handle the inherit to members button */
-      $this->members_inherit_from_group = isset($_POST['members_inherit_from_group']);
+        /* Unset some special vars ... */
+        foreach (array("gotoSyslogServer") as $val){
+            if ($this->attrs[$val] == "default"){
+                $this->attrs[$val]= array();
+            }
+        }
 
-      if(isset($_POST["inheritTimeServer"])){
-        $this->inheritTimeServer = true;
-      }else{
-        $this->inheritTimeServer = false;
-      }
-    }
-  }
-
-  /* Save to LDAP */
-  function save()
-  {
-    if (isset($this->parent->by_name['termstartup'])){
-    	$this->objectclasses= array("gotoTerminalTemplate");
-    } else {
-    	$this->objectclasses= array("gotoWorkstationTemplate");
-    }
+        /* Update ntp server settings */
+        if($this->inheritTimeServer){
+            $this->attrs['gotoNtpServer'] = "default";
+        }else{
+            /* Set ntpServers */
+            $this->attrs['gotoNtpServer'] = array();
+            foreach($this->gotoNtpServer as $server){
+                $this->attrs['gotoNtpServer'][] = $server;
+            }
+        }
 
-    plugin::save();
 
-    /* Tell members to inherit all attributes if the flag is set  */
-    $this->inherit_attributes_to_members();
+        /***************
+          Write to ldap 
+         ***************/
 
-    /***************
-      Prepare special vars 
-     ***************/
+        /* Write back to ldap */
+        $ldap= $this->config->get_ldap_link();
+        $ldap->cd($this->dn);
+        $this->cleanup();
+        $ldap->modify ($this->attrs);
 
-    /* Unset some special vars ... */
-    foreach (array("gotoSyslogServer") as $val){
-      if ($this->attrs[$val] == "default"){
-        $this->attrs[$val]= array();
-      }
-    }
+        if($this->initially_was_account){
+            new log("modify","ogroups/".get_class($this),$this->dn,array_keys($this->attrs),$ldap->get_error());
+        }else{
+            new log("create","ogroups/".get_class($this),$this->dn,array_keys($this->attrs),$ldap->get_error());
+        }
 
-    /* Update ntp server settings */
-    if($this->inheritTimeServer){
-      $this->attrs['gotoNtpServer'] = "default";
-    }else{
-      /* Set ntpServers */
-      $this->attrs['gotoNtpServer'] = array();
-      foreach($this->gotoNtpServer as $server){
-        $this->attrs['gotoNtpServer'][] = $server;
-      }
+        if(!$this->didAction){
+            $this->handle_post_events("modify");
+        }
+        if (!$ldap->success()){
+            msg_dialog::display(_("LDAP error"), msgPool::ldaperror($ldap->get_error(), $this->dn, LDAP_MOD, get_class()));
+        }
     }
 
 
-    /***************
-      Write to ldap 
-     ***************/
-
-    /* Write back to ldap */
-    $ldap= $this->config->get_ldap_link();
-    $ldap->cd($this->dn);
-    $this->cleanup();
-    $ldap->modify ($this->attrs);
-
-    if($this->initially_was_account){
-      new log("modify","ogroups/".get_class($this),$this->dn,array_keys($this->attrs),$ldap->get_error());
-    }else{
-      new log("create","ogroups/".get_class($this),$this->dn,array_keys($this->attrs),$ldap->get_error());
+    static function plInfo()
+    {
+        return (array(
+                    "plShortName"   => _("System"),
+                    "plDescription" => _("System group"),
+                    "plSelfModify"  => FALSE,
+                    "plDepends"     => array(),
+                    "plPriority"    => 5,
+                    "plSection"     => array("administration"),
+                    "plCategory"    => array("ogroups"),
+                    "plProvidedAcls"=> array(
+                        "gotoMode"          => _("Mode"),     
+                        "gotoSyslogServer"  => _("Syslog server"), 
+                        "FAIstate"          => _("Action flag"), 
+                        "gotoNtpServer"     => _("Ntp server"),
+                        "gotoTerminalPath"    => _("Root server"),
+                        "gotoSwapServer"      => _("Swap server"))
+                    ));
     }
 
-    if(!$this->didAction){
-      $this->handle_post_events("modify");
-    }
-    if (!$ldap->success()){
-      msg_dialog::display(_("LDAP error"), msgPool::ldaperror($ldap->get_error(), $this->dn, LDAP_MOD, get_class()));
-    }
-  }
-
-  
-  static function plInfo()
-  {
-    return (array(
-          "plShortName"   => _("System"),
-          "plDescription" => _("System group"),
-          "plSelfModify"  => FALSE,
-          "plDepends"     => array(),
-          "plPriority"    => 5,
-          "plSection"     => array("administration"),
-          "plCategory"    => array("ogroups"),
-          "plProvidedAcls"=> array(
-            "gotoMode"          => _("Mode"),     
-            "gotoSyslogServer"  => _("Syslog server"), 
-            "FAIstate"          => _("Action flag"), 
-            "gotoNtpServer"     => _("Ntp server"),
-            "gotoTerminalPath"    => _("Root server"),
-            "gotoSwapServer"      => _("Swap server"))
-          ));
-  }
-
-  function inherit_attributes_to_members()
-  {
-    if ($this->members_inherit_from_group) {
-      foreach ($this->member_dn as $dn) {
-        $member_obj = new worktabs($this->config, $this->config->data['TABS']['WORKTABS'], $dn, "workstation");
-        $member_obj->by_object['workgeneric']->set_everything_to_inherited();
-        $member_obj->save();
-      }
+    function inherit_attributes_to_members()
+    {
+        if ($this->members_inherit_from_group) {
+            foreach ($this->member_dn as $dn) {
+                $member_obj = new worktabs($this->config, $this->config->data['TABS']['WORKTABS'], $dn, "workstation");
+                $member_obj->by_object['workgeneric']->set_everything_to_inherited();
+                $member_obj->save();
+            }
+        }
     }
-  }
 
-  function PrepareForCopyPaste($source)
-  {
-    /* Create used ntp server array */
-    $this->gotoNtpServer= array();
+    function PrepareForCopyPaste($source)
+    {
+        /* Create used ntp server array */
+        $this->gotoNtpServer= array();
 
-    $source_o = new termgroup ($this->config, $source['dn']);
-    foreach(array("gotoTerminalPath","gotoSwapServer","gotoSyslogServer","gotoMode") as $attr){
-      $this->$attr = $source_o->$attr;
-    }
+        $source_o = new termgroup ($this->config, $source['dn']);
+        foreach(array("gotoTerminalPath","gotoSwapServer","gotoSyslogServer","gotoMode") as $attr){
+            $this->$attr = $source_o->$attr;
+        }
 
-    if(isset($source['gotoNtpServer'])){
-      $this->inheritTimeServer = false;
-      unset($source['gotoNtpServer']['count']);
-      foreach($source['gotoNtpServer'] as $server){
-        $this->gotoNtpServer[$server] = $server;
-      }
-    }
+        if(isset($source['gotoNtpServer'])){
+            $this->inheritTimeServer = false;
+            unset($source['gotoNtpServer']['count']);
+            foreach($source['gotoNtpServer'] as $server){
+                $this->gotoNtpServer[$server] = $server;
+            }
+        }
 
-    /* Set inherit checkbox state */
-    if(in_array("default",$this->gotoNtpServer)){
-      $this->inheritTimeServer = true;
-      $this->gotoNtpServer=array();
-    }
+        /* Set inherit checkbox state */
+        if(in_array("default",$this->gotoNtpServer)){
+            $this->inheritTimeServer = true;
+            $this->gotoNtpServer=array();
+        }
 
-    /* Create available ntp options */
-    $this->gotoNtpServers = $this->config->data['SERVERS']['NTP'];
-    foreach($this->gotoNtpServers as $key => $server){
-      if($server == "default"){
-        unset($this->gotoNtpServers[$key]);
-      }
+        /* Create available ntp options */
+        $this->gotoNtpServers = $this->config->data['SERVERS']['NTP'];
+        foreach($this->gotoNtpServers as $key => $server){
+            if($server == "default"){
+                unset($this->gotoNtpServers[$key]);
+            }
+        }
     }
-  }
 }
 
 // vim:tabstop=2:expandtab:shiftwidth=2:filetype=php:syntax:ruler: