Code

Fixed FAI.schmea dependencies 2/2
authorhickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Thu, 7 Dec 2006 11:18:58 +0000 (11:18 +0000)
committerhickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Thu, 7 Dec 2006 11:18:58 +0000 (11:18 +0000)
git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@5338 594d385d-05f5-0310-b6e9-bd551577e9d8

plugins/admin/systems/class_systemManagement.inc
plugins/admin/systems/class_workstationStartup.inc

index ec97ddc4ca61fc4cb3cd16abfb051ecc252c80e5..96dc554d0b5fd544bd471213b684d31d1df7ae88 100644 (file)
@@ -696,14 +696,23 @@ class systems extends plugin
       $userregex = "(gotoLastUser=".$this->DivListSystem->UserRegex.")";
     }
 
+    /* Attributes to fetch */
+    $sys_attrs        = array("cn", "description", "macAddress", "objectClass", "sambaDomainName");
+    $sys_categories   = array("terminal", "workstation", "server", "phone" ,"printer");
+
+    /* Add FAIstate to attributes if FAI is activated */
+    $tmp = search_config($this->config->data,"faiManagement","CLASS");
+    if(!empty($tmp)){
+      $sys_attrs[] = "FAIstate";
+    }    
+
     /* Walk through all possible search combinations, and search for some objects if the checkbox is enabled  */
     foreach($objs as $checkBox => $oc){
       if($this->DivListSystem->$checkBox){
         if($this->DivListSystem->SubSearch){
           if($oc['CLASS'] != ""){
             $filter = "(|(&".$userregex."(objectClass=".$oc['CLASS'].")(cn=".$this->DivListSystem->Regex.")))";
-            $new_res = get_list($filter, array("terminal", "workstation", "server", "phone" ,"printer"), $base,
-                  array("cn", "description", "macAddress", "objectClass", "sambaDomainName", "FAIstate"), GL_NONE | GL_SUBSEARCH | GL_SIZELIMIT);
+            $new_res = get_list($filter, $sys_categories , $base,$sys_attrs, GL_NONE | GL_SUBSEARCH | GL_SIZELIMIT);
       
             /* Remove all objects that are not in the expected sub department */
             foreach($new_res as $key => $obj){
@@ -716,8 +725,7 @@ class systems extends plugin
           /* User filter? */
           if($oc['CLASS'] != ""){
             $filter = "(|(&".$userregex."(objectClass=".$oc['CLASS'].")(cn=".$this->DivListSystem->Regex.")))";
-            $res = array_merge($res,get_list($filter,array("terminal", "workstation", "server", "phone" ,"printer"),$oc['TREE'].$base,
-                  array("cn", "description", "macAddress", "objectClass", "sambaDomainName", "FAIstate"), GL_NONE | GL_SIZELIMIT));
+            $res = array_merge($res,get_list($filter,$sys_categories,$oc['TREE'].$base, $sys_attrs, GL_NONE | GL_SIZELIMIT));
           }
         }
       } 
@@ -725,8 +733,7 @@ class systems extends plugin
 
     /* Search for incoming objects */ 
     $filter = "(|(&".$userregex."(objectClass=goHard)(cn=".$this->DivListSystem->Regex.")))";
-    $res = array_merge($res,get_list($filter,array("terminal", "workstation", "server", "phone" ,"printer"),"ou=incoming,".$base,
-          array("cn", "description", "macAddress", "objectClass", "sambaDomainName"), GL_NONE | GL_SIZELIMIT));
+    $res = array_merge($res,get_list($filter,$sys_categories,"ou=incoming,".$base,$sys_attrs, GL_NONE | GL_SIZELIMIT));
 
     /* Get all gotoTerminal's */
     foreach ($res as $value){
index 8c757bcb52e20511ae756bd0ffdbe91ee7305106..56192c9bfe0fa5e7c2d793b50ad6fa3ba5c85e35 100644 (file)
@@ -54,8 +54,19 @@ class workstartup extends plugin
 
   var $CopyPasteVars = array("gotoModules","gotoShares");
 
+  var $fai_activated = FALSE;
+
   function workstartup ($config, $dn= NULL, $parent= NULL)
   {
+    /* Check if FAI is active */
+    $tmp = search_config($config->data,"faiManagement","CLASS");
+    if(!empty($tmp)){
+      $this->fai_activated = TRUE;
+    }else{
+      $this->attributes = array("gotoLdapServer", "gotoBootKernel", "gotoKernelParameters", "gotoShare");
+      $this->objectclasses  = array("GOhard");
+    }
+
     plugin::plugin ($config, $dn, $parent);
 
     /* Creating a list of valid Mirrors 
@@ -67,137 +78,164 @@ class workstartup extends plugin
     foreach($this->config->data['SERVERS']['LDAP'] as $server) {
       $this->goLdapServerList[$server]= $server; // $this->config->data['SERVERS']['LDAP'];
     }
-   
-    $_SESSION['getAvailableClassesForThisRelease_CACHE'] = array();
-    $_SESSION['getAvailableClassesForThisRelease_CACHED_CLASSES'] = array();
+  
 
-    /* Search all FAI objects */
-    $ldap->search("(|(objectClass=FAIpackageList)(objectClass=FAItemplate)(objectClass=FAIvariable)(objectClass=FAIscript)(objectClass=FAIhook)(objectClass=FAIprofile)(objectClass=FAIpartitionTable))",array("cn","objectClass","FAIdebianSection"));
-    /* Sort all entries, and attach elementtype.
-     * To be able to show the types in the listbox.
+    /* FAI Initialization
+       Skip this if FAI is not activated 
      */
-    while($attr = $ldap->fetch()){
-      $cn = $attr['cn'][0];
-    
-      $_SESSION['getAvailableClassesForThisRelease_CACHED_CLASSES'][] = $attr;
-      if(in_array('FAIpackageList',$attr['objectClass'])){
-        $tmp2[$cn]['FAIpackageList']['obj']   = 'FAIpackageList'; 
-        $tmp2[$cn]['FAIpackageList']['kzl']   = 'Pl';
-        $tmp2[$cn]['FAIpackageList']['sec']   = $attr['FAIdebianSection'];
-        $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
-      }
-      if(in_array('FAItemplate',$attr['objectClass'])){
-        $tmp2[$cn]['FAItemplate']['obj']      = 'FAItemplate'; 
-        $tmp2[$cn]['FAItemplate']['kzl']      = 'T'; 
-        $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
-      }
-      if(in_array('FAIvariable',$attr['objectClass'])){
-        $tmp2[$cn]['FAIvariable']['obj']      = 'FAIvariable'; 
-        $tmp2[$cn]['FAIvariable']['kzl']      = 'V'; 
-        $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
-      }
-      if(in_array('FAIscript',$attr['objectClass'])){
-        $tmp2[$cn]['FAIscript']['obj']        = 'FAIscript'; 
-        $tmp2[$cn]['FAIscript']['kzl']        = 'S'; 
-        $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
-      }
-      if(in_array('FAIhook',$attr['objectClass'])){
-        $tmp2[$cn]['FAIhook']['obj']          = 'FAIhook'; 
-        $tmp2[$cn]['FAIhook']['kzl']          = 'H'; 
-        $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
-      }
-      if(in_array('FAIpartitionTable',$attr['objectClass'])){
-        $tmp2[$cn]['FAIpartitionTable']['obj']= 'FAIpartitionTable'; 
-        $tmp2[$cn]['FAIpartitionTable']['kzl']= 'Pt'; 
-        $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+      if($this->fai_activated) {
+
+      $_SESSION['getAvailableClassesForThisRelease_CACHE'] = array();
+      $_SESSION['getAvailableClassesForThisRelease_CACHED_CLASSES'] = array();
+
+      /* Search all FAI objects */
+      $ldap->search("(|(objectClass=FAIpackageList)(objectClass=FAItemplate)(objectClass=FAIvariable)(objectClass=FAIscript)(objectClass=FAIhook)(objectClass=FAIprofile)(objectClass=FAIpartitionTable))",array("cn","objectClass","FAIdebianSection"));
+      /* Sort all entries, and attach elementtype.
+       * To be able to show the types in the listbox.
+       */
+      while($attr = $ldap->fetch()){
+        $cn = $attr['cn'][0];
+      
+        $_SESSION['getAvailableClassesForThisRelease_CACHED_CLASSES'][] = $attr;
+   
+        if(in_array('FAIpackageList',$attr['objectClass'])){
+          $tmp2[$cn]['FAIpackageList']['obj']   = 'FAIpackageList'; 
+          $tmp2[$cn]['FAIpackageList']['kzl']   = 'Pl';
+          $tmp2[$cn]['FAIpackageList']['sec']   = $attr['FAIdebianSection'];
+          $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+        }
+        if(in_array('FAItemplate',$attr['objectClass'])){
+          $tmp2[$cn]['FAItemplate']['obj']      = 'FAItemplate'; 
+          $tmp2[$cn]['FAItemplate']['kzl']      = 'T'; 
+          $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+        }
+        if(in_array('FAIvariable',$attr['objectClass'])){
+          $tmp2[$cn]['FAIvariable']['obj']      = 'FAIvariable'; 
+          $tmp2[$cn]['FAIvariable']['kzl']      = 'V'; 
+          $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+        }
+        if(in_array('FAIscript',$attr['objectClass'])){
+          $tmp2[$cn]['FAIscript']['obj']        = 'FAIscript'; 
+          $tmp2[$cn]['FAIscript']['kzl']        = 'S'; 
+          $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+        }
+        if(in_array('FAIhook',$attr['objectClass'])){
+          $tmp2[$cn]['FAIhook']['obj']          = 'FAIhook'; 
+          $tmp2[$cn]['FAIhook']['kzl']          = 'H'; 
+          $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+        }
+        if(in_array('FAIpartitionTable',$attr['objectClass'])){
+          $tmp2[$cn]['FAIpartitionTable']['obj']= 'FAIpartitionTable'; 
+          $tmp2[$cn]['FAIpartitionTable']['kzl']= 'Pt'; 
+          $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+        }
+        if(in_array('FAIprofile',$attr['objectClass'])){
+          $tmp2[$cn]['FAIprofile']['obj']= 'FAIprofile'; 
+          $tmp2[$cn]['FAIprofile']['kzl']= 'P'; 
+          $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+        }
       }
-      if(in_array('FAIprofile',$attr['objectClass'])){
-        $tmp2[$cn]['FAIprofile']['obj']= 'FAIprofile'; 
-        $tmp2[$cn]['FAIprofile']['kzl']= 'P'; 
-        $this->FAIclasses[$attr['cn'][0]]=$attr['cn'][0];
+      if(is_array($this->FAIclasses)){
+        natcasesort($this->FAIclasses);
       }
-    }
-    if(is_array($this->FAIclasses)){
-      natcasesort($this->FAIclasses);
-    }
 
-    if(isset($tmp2)){
-      $this->FAIclassInfo = $tmp2;
-    } else {
-      $this->FAIclassInfo = array();
-    }
+      if(isset($tmp2)){
+        $this->FAIclassInfo = $tmp2;
+      } else {
+        $this->FAIclassInfo = array();
+      }
 
-    /* Build up an array like this one :
-        [$url]['SERVER'] = 'srv1-002';
-        [$url]['RELEASE']['siga/rc9.0.2']
-                                           ['SECTIONS'][0] "main";
-                                           ['SECTIONS'][1] "non-free";
-        [$url]['RELEASE']['siga/rc9.0.2']
-                                           ['PACKAGES'][0] "pkg1";
-                                           ['PACKAGES'][1] "postfix";
-     */
+      /* Build up an array like this one :
+          [$url]['SERVER'] = 'srv1-002';
+          [$url]['RELEASE']['siga/rc9.0.2']
+                                             ['SECTIONS'][0] "main";
+                                             ['SECTIONS'][1] "non-free";
+          [$url]['RELEASE']['siga/rc9.0.2']
+                                             ['PACKAGES'][0] "pkg1";
+                                             ['PACKAGES'][1] "postfix";
+       */
 
-    $ldap->search("(&(FAIrepository=*)(objectClass=FAIrepositoryServer))",array("FAIrepository"));
-    $test = array();
-    while($attr = $ldap->fetch()){
-      if(isset($attr['FAIrepository'])){
+      $ldap->search("(&(FAIrepository=*)(objectClass=FAIrepositoryServer))",array("FAIrepository"));
+      $test = array();
+      while($attr = $ldap->fetch()){
+        if(isset($attr['FAIrepository'])){
 
-        unset($attr['FAIrepository']['count']);
+          unset($attr['FAIrepository']['count']);
 
-        foreach($attr['FAIrepository'] as $rep){
-          $tmp = split("\|",$rep);
+          foreach($attr['FAIrepository'] as $rep){
+            $tmp = split("\|",$rep);
 
-          if(count($tmp)==4){
-            $sections = split(",",$tmp[3]);
-            $release  = $tmp[2];
-            $server   = $tmp[1];
-            $url      = $tmp[0];
-           
-            $test[$url]['RELEASE'][$release]['SECTIONS'] = $sections;
-    
-            /* Result will be cached
-             */
-            $test[$url]['RELEASE'][$release]['PACKAGES'] = $this->getAvailableClassesForThisRelease($release);
-            $test[$url]['SERVER'] = $server;
-
-            /* auto gets all releases/classes 
-             */
-            $test['auto']['RELEASE'][$release]['SECTION'] = $sections;
-            $test['auto']['RELEASE'][$release]['PACKAGES'] = $this->getAvailableClassesForThisRelease($release);
+            if(count($tmp)==4){
+              $sections = split(",",$tmp[3]);
+              $release  = $tmp[2];
+              $server   = $tmp[1];
+              $url      = $tmp[0];
+             
+              $test[$url]['RELEASE'][$release]['SECTIONS'] = $sections;
+      
+              /* Result will be cached
+               */
+              $test[$url]['RELEASE'][$release]['PACKAGES'] = $this->getAvailableClassesForThisRelease($release);
+              $test[$url]['SERVER'] = $server;
+
+              /* auto gets all releases/classes 
+               */
+              $test['auto']['RELEASE'][$release]['SECTION'] = $sections;
+              $test['auto']['RELEASE'][$release]['PACKAGES'] = $this->getAvailableClassesForThisRelease($release);
+            }
           }
         }
       }
-    }
-    
-    /* Add possible elements from hook */
-    $lines= $this->GetHookElements();
-    foreach ($lines as $hline){
-      $entries= split(";", $hline);
-      if (isset($entries[1]) && !isset($test[$entries[1]])){
-        $test[$entries[1]]['RELEASE']= array();
-
-        /* Split releases */
-        if (isset($entries[2])){
-          $releases= split(",", $entries[2]);
-
-          foreach ($releases as $release){
-            $rname= preg_replace('/:.*$/', '', $release);
-            $sections= split(':', preg_replace('/^[^:]+:([^|]+)|.*$/', '\1', $release));
-            $classes= split('\|', preg_replace('/^[^|]+\|(.*)$/', '\1', $release));
-            $test[$entries[1]]['RELEASE'][$rname]= array();
-            $test[$entries[1]]['RELEASE'][$rname]['SECTION']= $sections;
-            foreach ($classes as $class){
-              if ($class != ""){
-                $test[$entries[1]]['RELEASE'][$rname]['PACKAGES'][$class]= $class;
+      
+      /* Add possible elements from hook */
+      $lines= $this->GetHookElements();
+      foreach ($lines as $hline){
+        $entries= split(";", $hline);
+        if (isset($entries[1]) && !isset($test[$entries[1]])){
+          $test[$entries[1]]['RELEASE']= array();
+
+          /* Split releases */
+          if (isset($entries[2])){
+            $releases= split(",", $entries[2]);
+
+            foreach ($releases as $release){
+              $rname= preg_replace('/:.*$/', '', $release);
+              $sections= split(':', preg_replace('/^[^:]+:([^|]+)|.*$/', '\1', $release));
+              $classes= split('\|', preg_replace('/^[^|]+\|(.*)$/', '\1', $release));
+              $test[$entries[1]]['RELEASE'][$rname]= array();
+              $test[$entries[1]]['RELEASE'][$rname]['SECTION']= $sections;
+              foreach ($classes as $class){
+                if ($class != ""){
+                  $test[$entries[1]]['RELEASE'][$rname]['PACKAGES'][$class]= $class;
+                }
               }
             }
           }
         }
       }
-    }
-    
-    $this->FAIServRepConfig= $test;
+      
+      $this->FAIServRepConfig= $test;
+
+      if((isset($this->FAIclass))&&(!is_array($this->FAIclass))){
+        $tmp = array();
+        $tmp = split(" ",$this->FAIclass);
+        $tmp2 =array();  
+
+        foreach($tmp as $class){
+          if( ":" == $class[0] ) {
+            $this->FAIrelease = substr( $class, 1 );
+          }
+          else
+            $tmp2[$class] = $class;
+        }
+        $this->FAIclass = $tmp2;
+      }
+
+      if(!is_array($this->FAIclass)){
+        $this->FAIclass =array();
+      }
+    }// END of FAI initialization stuff 
+
 
     /* Get arrays */
     foreach (array("gotoModules", "gotoAutoFs", "gotoFilesystem") as $val){
@@ -245,24 +283,6 @@ class workstartup extends plugin
     $this->gotoAvailableShares= $config->getShareList(false);
     $tmp2 = array();
   
-    if((isset($this->FAIclass))&&(!is_array($this->FAIclass))){
-      $tmp = array();
-      $tmp = split(" ",$this->FAIclass);
-      $tmp2 =array();  
-    
-      foreach($tmp as $class){
-        if( ":" == $class[0] ) {
-          $this->FAIrelease = substr( $class, 1 );
-        }
-        else
-          $tmp2[$class] = $class;
-      }
-      $this->FAIclass = $tmp2;
-    }
-
-    if(!is_array($this->FAIclass)){
-      $this->FAIclass =array();
-    }
 
     $this->orig_dn= $this->dn;
 
@@ -274,7 +294,12 @@ class workstartup extends plugin
     $ldap->cd($this->config->current['BASE']);
     $ldap->search("(&(objectClass=gotoWorkstationTemplate)(member=".$this->dn."))");
     if ($ldap->count() == 1){
-      $map= array("gotoLdapServer","FAIclass","FAIdebianMirror");
+
+      if($this->fai_activated){
+        $map= array("gotoLdapServer","FAIclass","FAIdebianMirror");
+      }else{
+        $map= array("gotoLdapServer");
+      }
       $attrs= $ldap->fetch();
 
       foreach ($map as $name){
@@ -489,107 +514,109 @@ class workstartup extends plugin
     }
 
     /* FAI class management */
-    if(((isset($_POST['AddClass']))&&(isset($_POST['FAIclassesSel']))) && ($this->acl_is_writeable("FAIclass"))){
-      $found = 0 ; 
+    if($this->fai_activated){
+      if(((isset($_POST['AddClass']))&&(isset($_POST['FAIclassesSel']))) && ($this->acl_is_writeable("FAIclass"))){
+        $found = 0 ; 
 
-      /* If this new class/profile will attach a second partition table
-       * to our list of classes, abort and show a message.
-       */
-      foreach($this->FAIclass as $name){
-        if(isset($this->FAIclassInfo[$name])){
-          foreach($this->FAIclassInfo[$name] as $atr){
-            if(isset($atr['obj'])){
-              if($atr['obj'] == "FAIpartitionTable"){
-                $found ++ ; 
+        /* If this new class/profile will attach a second partition table
+         * to our list of classes, abort and show a message.
+         */
+        foreach($this->FAIclass as $name){
+          if(isset($this->FAIclassInfo[$name])){
+            foreach($this->FAIclassInfo[$name] as $atr){
+              if(isset($atr['obj'])){
+                if($atr['obj'] == "FAIpartitionTable"){
+                  $found ++ ; 
+                }
               }
             }
           }
         }
-      }
 
-      if((isset($this->FAIclassInfo[$_POST['FAIclassesSel']]['FAIpartitionTable']))&&($found>0)){
-        print_red(_("There is already a profile in your selection that contain partition table configurations."));
-      }else{
-        $this->FAIclass[$_POST['FAIclassesSel']]=$_POST['FAIclassesSel'];
+        if((isset($this->FAIclassInfo[$_POST['FAIclassesSel']]['FAIpartitionTable']))&&($found>0)){
+          print_red(_("There is already a profile in your selection that contain partition table configurations."));
+        }else{
+          $this->FAIclass[$_POST['FAIclassesSel']]=$_POST['FAIclassesSel'];
+        }
       }
-    }
 
-    $sort = false;
+      $sort = false;
 
-    /* Move one used class class one position up or down */
-    if($this->acl_is_writeable("FAIclass")){
-      foreach($_POST as $name => $val){
+      /* Move one used class class one position up or down */
+      if($this->acl_is_writeable("FAIclass")){
+        foreach($_POST as $name => $val){
 
-        $sort_type = false;
-        if((preg_match("/sort_up/",$name))&&(!$sort)){
-          $sort_type = "sort_up_";
-        }
-        if((preg_match("/sort_down/",$name))&&(!$sort)){
-          $sort_type = "sort_down_";
-        }
+          $sort_type = false;
+          if((preg_match("/sort_up/",$name))&&(!$sort)){
+            $sort_type = "sort_up_";
+          }
+          if((preg_match("/sort_down/",$name))&&(!$sort)){
+            $sort_type = "sort_down_";
+          }
 
-        if(($sort_type)&&(!$sort)){
-          $value = base64_decode(preg_replace("/_.*$/i","",preg_replace("/".$sort_type."/i","",$name)));
-          $sort = true;
+          if(($sort_type)&&(!$sort)){
+            $value = base64_decode(preg_replace("/_.*$/i","",preg_replace("/".$sort_type."/i","",$name)));
+            $sort = true;
 
-          $last = -1;
-          $change_down  = -1;
+            $last = -1;
+            $change_down  = -1;
 
-          /* Create array with numeric index */ 
-          $tmp = array();
-          foreach($this->FAIclass as $class){
-            $tmp [] = $class;
-          }
+            /* Create array with numeric index */ 
+            $tmp = array();
+            foreach($this->FAIclass as $class){
+              $tmp [] = $class;
+            }
 
-          /* Walk trough array */
-          foreach($tmp as $key => $faiName){
-            if($faiName == $value){
-              if($sort_type == "sort_up_"){
-                if($last != -1){
-                  $change_down= $last;
-                }
-              }else{
-                if(isset($tmp[$key+1])){
-                  $change_down = $key;
+            /* Walk trough array */
+            foreach($tmp as $key => $faiName){
+              if($faiName == $value){
+                if($sort_type == "sort_up_"){
+                  if($last != -1){
+                    $change_down= $last;
+                  }
+                }else{
+                  if(isset($tmp[$key+1])){
+                    $change_down = $key;
+                  }
                 }
               }
+              $last = $key;
             }
-            $last = $key;
-          }
 
-          $tmp2 = array();
-          $skip = false;    
+            $tmp2 = array();
+            $skip = false;    
 
-          foreach($tmp as $ky => $vl){
+            foreach($tmp as $ky => $vl){
 
-            if($ky == $change_down){
-              $skip = $vl;
-            }else{
-              $tmp2[$vl] = $vl;
-            }
-            if(($skip != false)&&($ky != $change_down)){
-              $tmp2[$skip]  = $skip;
-              $skip =false;
-            }
-          }   
-          $this->FAIclass = $tmp2; 
-        }
+              if($ky == $change_down){
+                $skip = $vl;
+              }else{
+                $tmp2[$vl] = $vl;
+              }
+              if(($skip != false)&&($ky != $change_down)){
+                $tmp2[$skip]  = $skip;
+                $skip =false;
+              }
+            }   
+            $this->FAIclass = $tmp2; 
+          }
 
-        if(preg_match("/fai_remove/i",$name)){
-          $value = base64_decode(preg_replace("/_.*$/i","",preg_replace("/fai_remove_/i","",$name)));
-          unset($this->FAIclass[$value]);
+          if(preg_match("/fai_remove/i",$name)){
+            $value = base64_decode(preg_replace("/_.*$/i","",preg_replace("/fai_remove_/i","",$name)));
+            unset($this->FAIclass[$value]);
+          }
         }
       }
-    }
 
-    /* Delete selected class from our list */
-    if($this->acl_is_writeable("FAIclass")){
-      if((isset($_POST['DelClass']))&&(isset($_POST['FAIclassSel']))){
-        if(isset($this->FAIclass[$_POST['FAIclassSel']])){
-          unset($this->FAIclass[$_POST['FAIclassSel']]);
+      /* Delete selected class from our list */
+      if($this->acl_is_writeable("FAIclass")){
+        if((isset($_POST['DelClass']))&&(isset($_POST['FAIclassSel']))){
+          if(isset($this->FAIclass[$_POST['FAIclassSel']])){
+            unset($this->FAIclass[$_POST['FAIclassSel']]);
+          }
         }
       }
-    }
+    }// END fai handling
 
     /* Show main page */
     $smarty= get_smarty();
@@ -687,62 +714,67 @@ class workstartup extends plugin
       $smarty->assign($val, $this->$val);
     }
 
-    $smarty->assign("FAIdebianMirrors",$this->getFAIdebianMirrors());
-    $smarty->assign("FAIdebianMirror",$this->FAIdebianMirror);
-    $smarty->assign("FAIreleases",$this->getFAIreleases());
-    $smarty->assign("FAIrelease",$this->FAIrelease);
-    $smarty->assign("FAIclasses",$this->selectFriendlyClasses());
-    $smarty->assign("FAIclassesKeys",array_flip($this->selectFriendlyClasses()));
-    $smarty->assign("FAIclassKeys",$this->FAIclass);
-  
-    $inheritedRelease = array();
-    if(!empty($this->InheritedFAIrelease)){
-      $inheritedRelease[$this->InheritedFAIrelease]= $this->InheritedFAIrelease;
-    } 
-    $smarty->assign("InheritedFAIrelease",$inheritedRelease);
-    $div = new divSelectBox("WSFAIscriptClasses");
-    $div -> SetHeight("110");
-    $str_up     = " &nbsp;<input type='image' src='images/sort_up.png'    name='sort_up_%s'    value='%s'>";
-    $str_down   = " &nbsp;<input type='image' src='images/sort_down.png'  name='sort_down_%s'  value='%s'>";
-    $str_remove = " &nbsp;<input type='image' src='images/edittrash.png'  name='fai_remove_%s' value='%s'>";
-    $str_empty  = " &nbsp;<img src='images/empty.png' alt=\"\" width='7'>"; 
-
-    $i = 1;
+    $smarty->assign("fai_activated",$this->fai_activated);
+
+    /* Create FAI output */
+    if($this->fai_activated){
+      $smarty->assign("FAIdebianMirrors",$this->getFAIdebianMirrors());
+      $smarty->assign("FAIdebianMirror",$this->FAIdebianMirror);
+      $smarty->assign("FAIreleases",$this->getFAIreleases());
+      $smarty->assign("FAIrelease",$this->FAIrelease);
+      $smarty->assign("FAIclasses",$this->selectFriendlyClasses());
+      $smarty->assign("FAIclassesKeys",array_flip($this->selectFriendlyClasses()));
+      $smarty->assign("FAIclassKeys",$this->FAIclass);
+
+      $inheritedRelease = array();
+      if(!empty($this->InheritedFAIrelease)){
+        $inheritedRelease[$this->InheritedFAIrelease]= $this->InheritedFAIrelease;
+      } 
+      $smarty->assign("InheritedFAIrelease",$inheritedRelease);
+
+      $div = new divSelectBox("WSFAIscriptClasses");
+      $div -> SetHeight("110");
+      $str_up     = " &nbsp;<input type='image' src='images/sort_up.png'    name='sort_up_%s'    value='%s'>";
+      $str_down   = " &nbsp;<input type='image' src='images/sort_down.png'  name='sort_down_%s'  value='%s'>";
+      $str_remove = " &nbsp;<input type='image' src='images/edittrash.png'  name='fai_remove_%s' value='%s'>";
+      $str_empty  = " &nbsp;<img src='images/empty.png' alt=\"\" width='7'>"; 
+
+      $i = 1;
 
-    if($this->FAIdebianMirror == "inherited"){
-      $tmp = $this->InheritedFAIclass;
-    }else{
-      $tmp = $this->FAIclass;
-    }
+      if($this->FAIdebianMirror == "inherited"){
+        $tmp = $this->InheritedFAIclass;
+      }else{
+        $tmp = $this->FAIclass;
+      }
 
-    foreach($tmp as $class){
+      foreach($tmp as $class){
 
-      $marker = "";
-      if(in_array_ics($class,$this->unresolved_classes)){
-        $marker = "&nbsp;<font color='red'>("._("Not available in current setup").")</font>";
-      }
+        $marker = "";
+        if(in_array_ics($class,$this->unresolved_classes)){
+          $marker = "&nbsp;<font color='red'>("._("Not available in current setup").")</font>";
+        }
 
-      if($this->FAIdebianMirror == "inherited"){
-        $str = "";
-      }else{
-        if($i==1){
-          $str = $str_empty.$str_down.$str_remove;
-        }elseif($i == count($this->FAIclass)){
-          $str = $str_up.$str_empty.$str_remove;
+        if($this->FAIdebianMirror == "inherited"){
+          $str = "";
         }else{
-          $str = $str_up.$str_down.$str_remove;
+          if($i==1){
+            $str = $str_empty.$str_down.$str_remove;
+          }elseif($i == count($this->FAIclass)){
+            $str = $str_up.$str_empty.$str_remove;
+          }else{
+            $str = $str_up.$str_down.$str_remove;
+          }
         }
-      }
-      $i ++ ; 
+        $i ++ ; 
 
-      $div->AddEntry(array(
-            array("string"=>$class.$marker),
-            array("string"=>preg_replace("/\%s/",base64_encode($class),$str),"attach"=>"style='width:50px;border-right:none;'")
-            ));
-    }  
+        $div->AddEntry(array(
+              array("string"=>$class.$marker),
+              array("string"=>preg_replace("/\%s/",base64_encode($class),$str),"attach"=>"style='width:50px;border-right:none;'")
+              ));
+      }  
 
-    $smarty->assign("FAIScriptlist",$div->DrawList()); 
+      $smarty->assign("FAIScriptlist",$div->DrawList()); 
+    }// END FAI output generation 
 
     /* Radio button group */
     if (preg_match("/G/", $this->bootmode)) {
@@ -912,16 +944,21 @@ class workstartup extends plugin
      * if we are currently editing from ogroup menu we use (array("gotWorkstationTemplate","GOhard", "FAIobject"))
      */
     if(isset($this->parent->by_object['ogroup'])){
-      $this->objectclasses = array("gotoWorkstationTemplate", "FAIobject");
+      $this->objectclasses = array("gotoWorkstationTemplate");
     }elseif(isset($this->parent->by_object['workgeneric'])){
-      $this->objectclasses = array("GOhard", "FAIobject");
+      $this->objectclasses = array("GOhard");
     }elseif(isset($this->parent->by_object['servgeneric'])){
-      $this->objectclasses = array("GOhard", "FAIobject","gotoWorkstationTemplate");
+      $this->objectclasses = array("GOhard","gotoWorkstationTemplate");
     }else{
       print "Object Type Configuration : unknown";
       exit();
     }
 
+    /* Append FAI class */
+    if($this->fai_activated){
+      $this->objectclasses[]  = "FAIobject";
+    }
+
     /* Find proper terminal path for tftp configuration
        FIXME: This is suboptimal when the default has changed to
        another location! */
@@ -974,24 +1011,24 @@ class workstartup extends plugin
     
     $str = "";
 
-    if($this->FAIdebianMirror == "inherited"){
-  
-      $this->attrs['FAIclass'] = $this->attrs['FAIrelease'] =  $this->attrs['FAIdebianMirror'] = array(); 
-
-    }else{
-
-      foreach($this->FAIclass as $class){
-        $str .= $class." ";
-      }
-      $str .= ":" . $this->FAIrelease;
-      $this->attrs['FAIclass']= "";
-      $this->attrs['FAIclass']= trim($str);
+    /* Skip FAI attribute handling if not necessary */
+    if($this->fai_activated){
+      if($this->FAIdebianMirror == "inherited"){
+        $this->attrs['FAIclass'] = $this->attrs['FAIrelease'] =  $this->attrs['FAIdebianMirror'] = array(); 
+      }else{
+        foreach($this->FAIclass as $class){
+          $str .= $class." ";
+        }
+        $str .= ":" . $this->FAIrelease;
+        $this->attrs['FAIclass']= "";
+        $this->attrs['FAIclass']= trim($str);
 
-      if(empty($this->attrs['FAIclass'])){
-        $this->attrs['FAIclass'] = array();
+        if(empty($this->attrs['FAIclass'])){
+          $this->attrs['FAIclass'] = array();
+        }
       }
     }
-
+  
     /* Add missing arrays */
     foreach (array("gotoFilesystem", "gotoAutoFs", "gotoModules") as $val){
       if (isset ($this->$val) && count ($this->$val) != 0){
@@ -1018,15 +1055,21 @@ class workstartup extends plugin
     /* Get FAIstate from object, the generic tab could have changed it during execute */
     $ldap= $this->config->get_ldap_link();
     $ldap->cd($this->dn);
-    $ldap->cat($this->dn,array("FAIstate"));
-    $checkFAIstate = $ldap->fetch();
 
-    /* Remove FAI objects if no FAI class is selected */ 
-    if((count($this->FAIclass)==0) && (!isset($checkFAIstate['FAIstate']))){
-      $this->attrs['FAIclass']        = array();
-      $this->attrs['FAIdebianMirror'] = array();
+
+    /* Skip FAI attribute handling if not necessary */
+    if($this->fai_activated){
+      $ldap->cat($this->dn,array("FAIstate"));
+      $checkFAIstate = $ldap->fetch();
+
+      /* Remove FAI objects if no FAI class is selected */ 
+      if((count($this->FAIclass)==0) && (!isset($checkFAIstate['FAIstate']))){
+        $this->attrs['FAIclass']        = array();
+        $this->attrs['FAIdebianMirror'] = array();
+      }
     }
 
+
     /* prepare share settings */
     $tmp = array();
     foreach($this->gotoShares as $name => $settings){