Code

Fixed o
[gosa.git] / plugins / personal / environment / class_environment.inc
index 365ccc98033127278636eb3df7d850ea7fd8e0e0..c42b2ad25eafb075c7c754b6b20ab8f66cd4d0ef 100644 (file)
@@ -15,6 +15,9 @@ class environment extends plugin
   
   var $in_dialog            = false;
   var $uid                  = "";
+
+  var $is_group             = false;
+  
   /* Attribute definition
    */
 
@@ -25,7 +28,7 @@ class environment extends plugin
   var $gotoProfileFlags   = "";     // Flags enabled  ? only used to set ACL and save 
   var $gotoProfileFlag_C  = "";     // Flag is set to C if we have the profile caching fucntion enabled 
   
-  var $gotoXResolution    = "";     // The selected resolution eg: 1024x768
+  var $gotoXResolution    = "1024x768";     // The selected resolution eg: 1024x768
   var $gotoXResolutions   = array();// Contains all available resolutions for this account
   var $gotoProfileFlag_L  = "";     // Flag is set to L to enable runtime resolution change 
   var $gotoProfileQuota   = "";     // User Quota Settings
@@ -35,11 +38,11 @@ class environment extends plugin
   var $gotoLogonScript    = "";     // The selected Logon Script
 
   /* Printer */
-  var $gotoPrinters       = array();// All available Printer
-  var $gotoPrinter        = "";     //  The selected Printer
+  var $gotoPrinter        = array();// All available Printer, with their configurations
+  var $gotoPrinterSel     = "";     //  The selected Printer
 
   /* Share */
-  var $gotoShares         = array();// Currently Share Option
+  var $gotoShares         = array();// Current Share Options
   var $gotoShare          = "";     // currently selected Share Option
   var $gotoShareSelections= array();// Available Shares for this account in Listbox format
   var $gotoAvailableShares= array();// Available Shares for this account
@@ -47,6 +50,7 @@ class environment extends plugin
   /* Kiosk profile */
   var $gotoKioskProfile   = "";     // The selected Kiosk Profile
   var $gotoKioskProfiles  = array();// All available Kiosk profiles
+  VAR $newKioskProfiles   = array();
 
   /* Hotplug Devices */
   var $gotoHotplugDevice  = "";     // Selected hotplug
@@ -55,33 +59,46 @@ class environment extends plugin
 
   /* general settings */
   // Sets the attributes which will kept on page reload, which will be saved, ...
-  var $attributes         = array("uid","useProfile","gotoProfileServer","gotoProfileServers","gotoProfileFlags","gotoProfileFlag_C",
-                                    "gotoXResolution","gotoXResolutions","gotoProfileFlag_L","gotoProfileQuota",
+  var $attributes         = array("uid","gotoProfileServer","gotoProfileFlags",
+                                    "gotoXResolution","gotoProfileQuota",
                                     "gotoLogonScripts","gotoLogonScript",
-                                    "gotoPrinters","gotoPrinter",
-                                    "gotoShares","gotoShare","gotoShareSelections",
+                                    "gotoPrinter",
+                                    "gotoShares","gotoShare",
                                     "gotoKioskProfile","gotoKioskProfiles",
-                                    "gotoHotplugDevice","gotoHotplugDevices");
+                                    "gotoHotplugDevices");
   var $objectclasses      = array("gotoEnvironment"); // Specifies the objectClass which contains the attributes edited here 
+  var $cn;
 
   function environment ($config, $dn= NULL)
   {
     plugin::plugin ($config, $dn);
 
+   
+    /* Check : Are we currently editing a group or user dialog */
+    if((isset($this->attrs['cn'][0]))&&(!isset($this->attrs['uid'][0]))){
+      $suffix="Group";
+      $this->uid          = $this->attrs['cn'][0];
+      $this->attrs['uid'] = $this->attrs['cn'][0];
+    }else{
+      $suffix="User";
+    }
+
+    $this->gotoKioskProfile= preg_replace("/^.*\//i","",$this->gotoKioskProfile);
+
     /* Get all Printer assignments */
     $ldap = $this->config->get_ldap_link();
     $ldap->cd($this->config->current['BASE']);
-    $ldap->search("(&(objectClass=gotoPrinter)(gotoUserPrinter=".$this->uid."))",array("*"));
+    $ldap->search("(&(objectClass=gotoPrinter)(goto".$suffix."Printer=".$this->uid."))",array("*"));
     while($printer = $ldap->fetch()){
-      $this->gotoPrinters[$printer['cn'][0]]=$printer;
-      $this->gotoPrinters[$printer['cn'][0]]['mode']="user";
+      $this->gotoPrinter[$printer['cn'][0]]=$printer;
+      $this->gotoPrinter[$printer['cn'][0]]['mode']="user";
     }
-    $ldap->search("(&(objectClass=gotoPrinter)(gotoAdminPrinter=".$this->uid."))",array("*"));
+    $ldap->search("(&(objectClass=gotoPrinter)(goto".$suffix."AdminPrinter=".$this->uid."))",array("*"));
     while($printer = $ldap->fetch()){
-      $this->gotoPrinters[$printer['cn'][0]]=$printer;
-      $this->gotoPrinters[$printer['cn'][0]]['mode']="admin";
+      $this->gotoPrinter[$printer['cn'][0]]=$printer;
+      $this->gotoPrinter[$printer['cn'][0]]['mode']="admin";
     }
-    
+  
     /* prepare hotplugs */
     if((isset($this->attrs['gotoHotplugDevice']))&&(is_array($this->attrs['gotoHotplugDevice']))){
       unset($this->attrs['gotoHotplugDevice']['count']);
@@ -125,9 +142,15 @@ class environment extends plugin
       foreach($this->attrs['gotoShare'] as $share){
         $tmp = $tmp2 = array();
         $tmp = split("\|",$share);
-        $tmp2['name']      =$tmp[0];
-        $tmp2['mountPoint']=$tmp[1];
-        $this->gotoShares[$tmp[0]]=$tmp2;
+        $tmp2['server']      =$tmp[0];
+        $tmp2['name']        =$tmp[1];
+        $tmp2['mountPoint']  =$tmp[2];
+        if(isset($tmp[3])){
+          $tmp2['OtherStuff']  =$tmp[3];
+        }else{
+          $tmp2['OtherStuff']  ="";
+        }
+        $this->gotoShares[$tmp[1]."|".$tmp[0]]=$tmp2;
       }
     }
 
@@ -143,18 +166,55 @@ class environment extends plugin
       $this->useProfile = false;
     }
 
+    /* Set to group environment if we editing a group */
+    if(!isset($this->parent)){
+      $this->is_group = true;
+    }
+
+    $this->gotoProfileServers= $config->getShareServerList() ;
+    $this->gotoShareSelections= $config->getShareList(true);
+    $this->gotoAvailableShares= $config->getShareList(false);  
+
   }
 
   function execute()
   {
-    /* Fill templating stuff */
-    $smarty= get_smarty();
-    $display= "";
-    /* Is accout enabled | are we editing from usermenu or admin menu 
-       All these tab management is done here
-    */
+       /* Call parent execute */
+       plugin::execute();
+
+  /* Fill templating stuff */
+  $smarty= get_smarty();
+  $display= "";
+
+  /* Prepare all variables for smarty */
+  foreach($this->attributes as $s_attr){
+    /* Set value*/
+    $smarty->assign($s_attr,$this->$s_attr);
+
+    /* Set checkbox state*/
+    if(empty($this->$s_attr)){
+      $smarty->assign($s_attr."CHK","");
+    }else{
+      $smarty->assign($s_attr."CHK"," checked ");
+    }
 
-    // 1. Account disabled . Editing from usermenu
+    /* Prepare ACL settings*/
+    if(chkacl($this->acl,$s_attr)=="") {
+      $smarty->assign($s_attr."ACL","");
+    }else{
+      $smarty->assign($s_attr."ACL"," disabled ");
+    }
+
+  }
+
+  /* Is accout enabled | are we editing from usermenu or admin menu 
+     All these tab management is done here
+   */
+
+  /* Working from Usermenu an the Account is currently disbled
+     * this->parent :  is only set if we are working in a list of tabs
+     * is_account   :  is only true if the needed objectClass is given
+    */
     if((!isset($this->parent))&&(!$this->is_account)){
       /* We are currently editing this tab from usermenu, but this account is not enabled */
       $smarty->assign("is_account",$this->is_account);
@@ -164,34 +224,75 @@ class environment extends plugin
       $display .= back_to_main(); 
       /* Display our message to the user */
       return $display;
+    
 
-    // Account is enabled
+    /* We are currently editing from group tabs, because 
+     * $this->parent is set
+     * posixAccount is not set, so we are not in usertabs.
+     */
+    }elseif((isset($this->parent))&&(!isset($this->parent->by_object['posixAccount']))){
+      $smarty->assign("is_account","true");
+      $this->is_group     = true;
+      $this->uid          = $this->cn;
+      $this->attrs['uid'] = $this->cn;
+
+      /* Change state if needed */
+      if (isset($_POST['modify_state'])){
+        $this->is_account= !$this->is_account;
+      }
+
+      /* Group Dialog with enabled environment options */
+      if ($this->is_account){
+        $display= $this->show_header(_("Remove environment extension"),
+            _("Environment extension enabled. You can disable it by clicking below."));
+      } else {
+  
+      /* Environment is disabled 
+         If theres is no posixAccount enabled, you won't be able to enable 
+         environment extensions
+       */
+        if((isset($this->attrs['objectClass']))&&((in_array("posixAccount",$this->attrs['objectClass'])))){
+          // 4. There is a PosixAccount
+          $display= $this->show_header(_("Add environment extension"),
+              _("Environment extension disabled. You can enable it by clicking below."));
+          return $display;
+        }else{
+          // 4. There is no PosixAccount
+          $display= $this->show_header(_("Add environment extension"),
+              _("Environment extension disabled. You have to setup a posix account before you can enable this feature."));
+          return $display;
+        }
+      }
     }else{
-      /* Tell smarty that this accoutn is enabled */
+      /* Editing from Usermenu 
+       *  Tell smarty that this accoutn is enabled 
+       */
       $smarty->assign("is_account","true");
 
+      $this->is_group = false;
+
       /* Do we need to flip is_account state? */
       if (isset($_POST['modify_state'])){
         $this->is_account= !$this->is_account;
       }
-      if(!isset($this->parent)){
-          // 3. ? Account Enabled . Editing from usermenu        
 
-      }else{
+      if(isset($this->parent)){
         // 3. Account enabled . Editing from adminmenu
         if ($this->is_account){
           $display= $this->show_header(_("Remove environment extension"),
-              _("This server has environment extension enabled. You can disable it by clicking below."));
+              _("Environment extension enabled. You can disable it by clicking below."));
         } else {
-          if((in_array("posixAccount",$this->attrs['objectClass']))||($this->parent->by_object['posixAccount']->is_account==true)){
+          if((isset($this->attrs['objectClass']))
+                &&((in_array("posixAccount",$this->attrs['objectClass'])))
+                  ||($this->parent->by_object['posixAccount']->is_account==true)){
             // 4. There is a PosixAccount
             $display= $this->show_header(_("Add environment extension"),
-                _("This server has environment extension disabled. You can enable it by clicking below."));
+                _("Environment extension disabled. You can enable it by clicking below."));
             return $display;
           }else{
             // 4. There is a PosixAccount
             $display= $this->show_header(_("Add environment extension"),
-                _("This server has environment extension disabled. You have to setup a posix account before you can enable this feature."),TRUE,TRUE);
+                _("Environment extension disabled. You have to setup a posix account before you can enable this feature."),TRUE,TRUE);
             return $display;
           }
         }
@@ -221,11 +322,21 @@ class environment extends plugin
       }
  
     }
+
+    foreach(array("gotoHotplugDevice","gotoPrinterSel") as $s_attr){
+      if(chkacl($this->acl,$s_attr)=="") {
+        $smarty->assign($s_attr."ACL","");
+      }else{
+        $smarty->assign($s_attr."ACL"," disabled ");
+      }
+    }
  
+    $smarty->assign("useProfileACL","");
     if($this->acl != "#none#"){
       $smarty->assign("useProfileACL","");
     }else{
       $smarty->assign("gotoProfileFlag_CACL"," disabled ");
+      $smarty->assign("useProfileACL","disabled");
       $smarty->assign("gotoProfileServer"," disabled ");
     }
 
@@ -236,6 +347,18 @@ class environment extends plugin
      * Assign this all to Smarty 
      */
 
+    if(empty($this->gotoProfileFlag_L)){
+      $smarty->assign("gotoProfileFlag_LCHK"," ");
+    }else{
+      $smarty->assign("gotoProfileFlag_LCHK"," checked ");
+    }
+
+    if(empty($this->gotoProfileFlag_C)){
+      $smarty->assign("gotoProfileFlag_CCHK"," ");
+    }else{
+      $smarty->assign("gotoProfileFlag_CCHK"," checked ");
+    }
+
     if(empty($this->useProfile)){
       $smarty->assign("gotoProfileACL","disabled"); 
       $smarty->assign("useProfileCHK",""); 
@@ -245,33 +368,42 @@ class environment extends plugin
     }
 
     $this->gotoXResolutions = array("640x480","800x600","1024x768","1280x768","1280x1024");
-    $this->gotoProfileServers =$this->getShareServerList() ;
 
     $smarty->assign("gotoXResolutions",$this->gotoXResolutions);
     $smarty->assign("gotoProfileServers",$this->gotoProfileServers);
+    if(!is_array($this->gotoProfileServers)){
+      $this->gotoProfileServers =array();
+    }
+    $smarty->assign("gotoProfileServerKeys",array_flip($this->gotoProfileServers));
 
     /* Handle kiosk profiles 
      * Read available from filesystem
      * Open management if post is transmitted
      */
-    $tmp = new kioskManagementDialog($this->config,$this->dn);
-    $list = $tmp->getKioskProfiles();
-
-    $smarty->assign("gotoKioskProfiles",$list);
-    $smarty->assign("gotoKioskProfileKeys",array_flip($list));
-
-    /* Open Management Dialog */
-    if(isset($_POST['KioskManagementDialog'])){
-      $this->dialog = new kioskManagementDialog($this->config,$this->dn); 
-      $this->is_dialog = true;
-    }
 
     /* Save */
     if(isset($_POST['KioskClose'])){
+      $this->newKioskProfiles = array_merge($this->newKioskProfiles,$this->dialog->save());
+  
       unset($this->dialog);
       $this->dialog=NULL;
       $this->is_dialog = false;
     }
+    $tmp = new kioskManagementDialog($this->config,$this->dn);
+    $list = $tmp->getKioskProfiles($this->newKioskProfiles);
+
+    /* Reassign help class */
+    $_SESSION['current_class_for_help'] = get_class($this);
+    /* Open Management Dialog */
+    if(isset($_POST['KioskManagementDialog'])){
+      $this->dialog = new kioskManagementDialog($this->config,$this->dn,$this->newKioskProfiles); 
+      $this->dialog->parent= $this;
+      $this->is_dialog = true;
+    }
+
+    $smarty->assign("gotoKioskProfiles",$list);
+    $smarty->assign("gotoKioskProfileKeys",array_flip($list));
 
     /* Logonscript Management
      * Get available LogonScripts (possibly grey out (or mark) these script that are defined for the group) 
@@ -338,10 +470,11 @@ class environment extends plugin
      *  and attach this combination to his setup.
      */
     
-    $this->gotoShareSelections    = $this->getShareList(true);
-    $this->gotoAvailableShares    = $this->getShareList(false);  
-
     $smarty->assign("gotoShareSelections",    $this->gotoShareSelections);
+    if(!is_array($this->gotoShareSelections)){
+      print $this->gotoShareSelections;
+      $this->gotoShareSelections = array();
+    }
     $smarty->assign("gotoShareSelectionKeys", array_flip($this->gotoShareSelections));
 
     /* if $_POST['gotoShareAdd'] is set, we will try to add a new entry 
@@ -349,14 +482,16 @@ class environment extends plugin
      */
     if(isset($_POST['gotoShareAdd'])){
       /* We assign a share to this user, if we don't know where to mount the share */
-      if((!isset($_POST['gotoShareMountPoint']))||(empty($_POST['gotoShareMountPoint']))){
-        print_red("You must specify a valid mount point.");
+      if((!isset($_POST['gotoShareMountPoint']))||(empty($_POST['gotoShareMountPoint']))||(preg_match("/[\|]/i",$_POST['gotoShareMountPoint']))){
+        print_red(_("You must specify a valid mount point."));
+      }elseif($_POST['gotoShareMountPoint'][0] !="/" ){
+        print_red(_("You must specify a valid mount point.")); 
       }else{
         $a_share = $this->gotoAvailableShares[$_POST['gotoShareSelection']];
         $s_mount = $_POST['gotoShareMountPoint'];
-
         /* Preparing the new assignment */ 
         $this->gotoShares[$a_share['name']]=$a_share;
+        $this->gotoShares[$a_share['name']]['OtherStuff']="";
         $this->gotoShares[$a_share['name']]['mountPoint']=$s_mount;
       }
     }  
@@ -453,29 +588,29 @@ class environment extends plugin
         $this->dialog->save_object();
         $tmp = $this->dialog->save();
         $tmp2= $this->dialog->getPrinter(true);
-        $this->gotoPrinters[$tmp]=$tmp2[$tmp];
-        $this->gotoPrinters[$tmp]['mode']="user";
+        $this->gotoPrinter[$tmp]=$tmp2[$tmp];
+        $this->gotoPrinter[$tmp]['mode']="user";
         $this->is_dialog=false;
         unset($this->dialog);
         $this->dialog   =NULL;
       }
     }
   
-    if((isset($_POST['gotoPrinterDel']))&&(isset($_POST['gotoPrinter']))&&(!empty($_POST['gotoPrinter']))){
-      $printer = $_POST['gotoPrinter'];
-      unset($this->gotoPrinters[$printer]);
+    if((isset($_POST['gotoPrinterDel']))&&(isset($_POST['gotoPrinterSel']))&&(!empty($_POST['gotoPrinterSel']))){
+      $printer = $_POST['gotoPrinterSel'];
+      unset($this->gotoPrinter[$printer]);
     }
 
-    if((isset($_POST['gotoPrinterEdit']))&&(isset($_POST['gotoPrinter']))&&(!empty($_POST['gotoPrinter']))){
-      $printer = $_POST['gotoPrinter'];
-      if($this->gotoPrinters[$printer]['mode']=="user"){
-        $this->gotoPrinters[$printer]['mode']="admin";
+    if((isset($_POST['gotoPrinterEdit']))&&(isset($_POST['gotoPrinterSel']))&&(!empty($_POST['gotoPrinterSel']))){
+      $printer = $_POST['gotoPrinterSel'];
+      if($this->gotoPrinter[$printer]['mode']=="user"){
+        $this->gotoPrinter[$printer]['mode']="admin";
       }else{
-        $this->gotoPrinters[$printer]['mode']="user";
+        $this->gotoPrinter[$printer]['mode']="user";
       }
     }
 
-    $smarty->assign("gotoPrinters",$this->printOutPrinterDevices());
+    $smarty->assign("gotoPrinter",$this->printOutPrinterDevices());
     $smarty->assign("gotoPrinterKeys",array_flip($this->printOutPrinterDevices()));
  
     /* General behavior */
@@ -515,9 +650,8 @@ class environment extends plugin
     plugin::remove_from_parent();
 
     /* Don't save our template variables */
-    $skip = array("useProfile","uid","gotoProfileServers","gotoProfileFlag_C","gotoXResolutions","gotoProfileFlag_L",
-                  "gotoLogonScripts","gotoPrinters","gotoShares","gotoShareSelections","gotoKioskProfiles","gotoHotplugDevices",
-                  "gotoPrinter");
+    $skip = array("uid",
+                  "gotoLogonScripts","gotoPrinter","gotoShares","gotoKioskProfiles","gotoHotplugDevices" );
     /* Skip all these attributes */
     foreach($skip as $del){
       unset($this->attrs[$del]);
@@ -541,11 +675,29 @@ class environment extends plugin
     /* Get all Posted vars 
      * Setup checkboxes 
      */
+   
     
     if(isset($_POST['iamposted'])){
+      if(isset($_POST['useProfile'])){
+        $this->useProfile = $_POST['useProfile'];
+      }else{
+        $this->useProfile = true;
+      }
+      if(isset($_POST['gotoProfileFlag_C'])){
+        $this->gotoProfileFlag_C = $_POST['gotoProfileFlag_C'];
+      }else{
+        $this->gotoProfileFlag_C = false;
+      }
+      if(isset($_POST['gotoProfileFlag_L'])){
+        $this->gotoProfileFlag_L = $_POST['gotoProfileFlag_L'];
+      }else{
+        $this->gotoProfileFlag_L = false;
+      }
+
       plugin::save_object();
       foreach($this->attributes as $s_attr){
-        if(in_array($s_attr,array("gotoShares","gotoHotplugDevices","gotoPrinters","gotoLogonScripts","uid"))) continue;
+        if(in_array($s_attr,array("gotoShares","gotoHotplugDevices","gotoPrinter","gotoLogonScripts","uid"))) continue;
         if(isset($_POST[$s_attr])){
           $this->$s_attr = $_POST[$s_attr];
         }else{
@@ -560,6 +712,18 @@ class environment extends plugin
   function check()
   {
     $message= array();
+  
+    if(preg_match("/[^0-9]/",$this->gotoProfileQuota)) {
+      $message[]=_("Please set a valid profile quota size.");
+    } 
+    if(!isset($this->attrs['objectClass'])){
+      $this->attrs['objectClass']=array();
+    } 
+    if(!$this->is_group){
+      if((!((in_array("posixAccount",$this->attrs['objectClass']))||($this->parent->by_object['posixAccount']->is_account==true)))&&(!$this->is_group)){
+        $message[]=(_("You need to setup a valid posix extension in order to enable evironment features."));  
+      }
+    }
     return ($message);
   }
 
@@ -571,7 +735,21 @@ class environment extends plugin
     $ldap= $this->config->get_ldap_link();
 
     $realyUsedAttrs= array();
-  
+
+
+    $path = search_config($this->config->data,"environment", "KIOSKPATH"); 
+    /* Creating Kiosk Profiles */
+    foreach($this->newKioskProfiles as $file){
+      $contents = $file['contents'];
+      $fp = @fopen($path."/".$file['name'],"w");
+      if(!$fp){
+        print_red(_("Can't save new kiosk profiles, possibly permission denied for folder")." : ",$path);
+      }else{
+        fwrite($fp,$contents,strlen($contents));
+      }
+      @unlink($file['tmp_name']);
+    }
     /* Save already used objectClasses */
     $ocs        = $this->attrs['objectClass'];
     unset($ocs['count']);
@@ -587,57 +765,86 @@ class environment extends plugin
     /* Save usersettings to Printer */
 
     if(chkacl($this->acl,"gotoPrinter")!=""){
-      $this->gotoPrinters = array();
+      $this->gotoPrinter = array();
     }  
+    
+    if($this->is_group){
+      $s_suffix = "Group";
+    }else{
+      $s_suffix = "User";
+    }
+  
 
-    if(count($this->gotoPrinters)==0){
-      $ldap->search("(&(objectClass=gotoPrinter)(|(gotoUserPrinter=".$this->uid.")(gotoAdminPrinter=".$this->uid.")))",array("*"));
-      while($attr = $ldap->fetch()){
-      
-        foreach($attr['gotoUserPrinter'] as $key => $user){
+    /* 1. Search all printers that have our uid/cn as member 
+     * 2. Delete this uid/cn from every single entry and save it again.
+     * 2.1 There are different types of members: Users / Groups, this will be defined in $suffix
+     * 2.2 And each type has two modes, Admin (e.g. 'gotoUserAdminPrinter') and Normal 
+     */
+    $ldap->search("(&(objectClass=gotoPrinter)(|(goto".$s_suffix."Printer=".$this->uid.")(goto".$s_suffix."AdminPrinter=".$this->uid.")))",array("*"));
+    while($attr = $ldap->fetch()){
+
+      /* Walk trough all printers and check if our user id used, if so remove it.
+       * Later we will insert our uid at the right place.
+       */
+
+      /* Remove normal entries (User)*/
+      if(isset($attr['goto'.$s_suffix.'Printer'])) {
+        foreach($attr['goto'.$s_suffix.'Printer'] as $key => $user){
           if($this->uid==$user){
-            unset($attr['gotoUserPrinter'][$key]);
+            unset($attr['goto'.$s_suffix.'Printer'][$key]);
           }
         }    
+      }
 
-        foreach($attr['gotoAdminPrinter'] as $key => $user){
+      /* Remove administrational entries (Admin)*/
+      if(isset($attr['goto'.$s_suffix.'AdminPrinter'])){
+        foreach($attr['goto'.$s_suffix.'AdminPrinter'] as $key => $user){
           if($this->uid==$user){
-            unset($attr['gotoAdminPrinter'][$key]);
+            unset($attr['goto'.$s_suffix.'AdminPrinter'][$key]);
           }
         }    
+      }
+
+      /* Extract useable tags, to be able to save all changes 
+       */
+      $attrs_used = array();
+      foreach($attr as $key=>$val){
 
-        $attrs_used = array();
-        foreach($attr as $key=>$val){
-          if((!is_numeric($key))&&($key!="count")){
-            if(is_array($val)&&isset($val['count'])){
-              unset($val['count']);
-            }
-            $attrs_used[$key]=$val;
+        /* If index is numeric, skip it ...*/
+        if((!is_numeric($key))&&($key!="count")){
+
+          /* If entry contains 'count' remove it */
+          if(is_array($val)&&isset($val['count'])){
+            unset($val['count']);
           }
+          $attrs_used[$key]=$val;
         }
-        $attr= $attrs_used;
-        $tmp =array();
-        foreach($attr['gotoAdminPrinter'] as $print){
-          $tmp[]=$print;
-        }
-        $attr['gotoAdminPrinter'] = $tmp;
+      }
+      /* the result of cleaning the entry is 
+       *  to be able to directly save this again,
+       *  if all changes are made 
+       */
+      $attr= $attrs_used;
 
-        $tmp =array();
-        foreach($attr['gotoUserPrinter'] as $print){
-          $tmp[]=$print;
-        }
-        $attr['gotoUserPrinter'] = $tmp;
+#fix : Id don't know why such an entry was set ... 
+      if(isset($attr['GOTOADMINPRINTER'])){
+        unset($attr['GOTOADMINPRINTER']);
+      }
 
-        $ldap->cd($attr['dn']);
-        unset($attr['dn']);
-        $ldap->modify($attr);
-        if($ldap->get_error()!="Success"){
-          print_red("Printer saving ".$ldap->get_error());
-        }
+      /* Save changes */
+      $ldap->cd($attr['dn']);
+      unset($attr['dn']);
+      $ldap->modify($attr);
+      if($ldap->get_error()!="Success"){
+        print_red(_("Error while writing printer")." : ".$ldap->get_error());
       }
-    }    
+    }
 
-    foreach($this->gotoPrinters as $printer) {
+    /* All printers are cleaned, (our cn/uid removed) 
+     *  now we must add our uid / cn 
+     *  to the new configured printers.
+     */
+    foreach($this->gotoPrinter as $printer) {
       $ldap->cd($printer['dn']);
       $ldap->cat($printer['dn']);
       $attrs= $ldap->fetch();
@@ -650,16 +857,20 @@ class environment extends plugin
           $attrs_used[$key]=$val;
         }
       }
+      /* $attrs contains all values 
+       * we need, to save the entry lateron 
+      */
       $attrs= $attrs_used;
 
-      /* Filter entries */
+      /* Depending on the type (User/Admin) 
+       *  switch these attributes, that makes it easier
+       */
       if($printer['mode'] == "user"){
-        $attribute  = "gotoUserPrinter";
-        $attribute2 = "gotoAdminPrinter";
+        $attribute  = "goto".$s_suffix."Printer";
+        $attribute2 = "goto".$s_suffix."AdminPrinter";
       }else{
-        $attribute  = "gotoAdminPrinter";
-        $attribute2 = "gotoUserPrinter";
+        $attribute  = "goto".$s_suffix."AdminPrinter";
+        $attribute2 = "goto".$s_suffix."Printer";
       }
 
       /* If this user is already assigned to $attribute2 
@@ -670,7 +881,7 @@ class environment extends plugin
         $printer[$attribute2]=array();
       }else{
         if(in_array($this->uid,$printer[$attribute2])){ 
-          $tmp = array_flip($attrs[$attribute2]);
+          $tmp = array_flip($printer[$attribute2]);
           unset($tmp[$this->uid]);
           $attrs[$attribute2]=array_flip($tmp);
         }
@@ -691,34 +902,14 @@ class environment extends plugin
         }
       }
 
-      $tmp =array();
-      if(isset($attrs['gotoAdminPrinter'])){
-        foreach($attrs['gotoAdminPrinter'] as $print){
-          $tmp[]=$print;
-        }
-      }
-      $attrs['gotoAdminPrinter'] = $tmp;
-
-      $tmp =array();
-      if(isset($attrs['gotoUserPrinter'])){
-        foreach($attrs['gotoUserPrinter'] as $print){
-          $tmp[]=$print;
-        }
-      }
-      $attrs['gotoUserPrinter'] = $tmp;
-
       $ldap->cd($attrs['dn']);
       unset($attrs['dn']);
       $ldap->modify($attrs);
       if($ldap->get_error()!="Success"){
-        print_red("Printer saving ".$ldap->get_error());
+        print_red(_("Error while writing printer settings")." : ".$ldap->get_error());
       }
     }
   
-    if(isset($this->attrs['gotoPrinter'])){
-      unset($this->attrs['gotoPrinter']);
-    }
     /* Prepare HotPlug devices */
     $this->attrs['gotoHotplugDevice'] = array();
     foreach($this->gotoHotplugDevices as $name => $device){
@@ -738,16 +929,23 @@ class environment extends plugin
     /* Prepare Shares */
     $this->attrs['gotoShare']=array();
     foreach($this->gotoShares as $name => $share){
-      $this->attrs['gotoShare'][] = $share['name']."|".$share['mountPoint'];
+      $this->attrs['gotoShare'][] =$share['server']."|".$share['name']."|".$share['mountPoint']."|".$share['OtherStuff'];
     }
 
     if(!empty($this->gotoKioskProfile)){
-      $this->attrs['gotoKioskProfile']= $this->gotoKioskProfile;
+      if(preg_match("/https/i",$_SERVER['HTTP_REFERER'])){
+        $method="https://";
+      }else{
+        $method="http://";
+      }
+
+      $str = $method.str_replace("//","/",$_SERVER['SERVER_NAME']."/kiosk/");
+      $this->attrs['gotoKioskProfile']= $str.$this->gotoKioskProfile;
     }else{
       $this->attrs['gotoKioskProfile']= array();
     }
 
-    $saveThis = array("gotoKioskProfile","gotoProfileQuota","gotoXResolution","gotoProfileServer");
+    $saveThis = array("gotoProfileQuota","gotoXResolution","gotoProfileServer");
   
     foreach($saveThis as $tosave){
       if(!empty($this->$tosave)){
@@ -793,49 +991,6 @@ class environment extends plugin
     $this->handle_post_events($mode);
   }
 
-/* This function returns all available Shares defined in this ldap 
- * There are two ways to call this function, if listboxEntry is true
- *  only name and path are attached to the array, in it is false, the whole 
- *  entry will be parsed an atached to the result.
- */
-  function getShareList($listboxEntry = false)
-  {
-    $ldap= $this->config->get_ldap_link();
-    $a_res = $ldap->search("(objectClass=goShareServer)",array("goExportEntry"));
-    $return= array();
-    while($entry = $ldap->fetch($a_res)){
-      unset($entry['goExportEntry']['count']);
-      foreach($entry['goExportEntry'] as $export){
-        $shareAttrs = split("\|",$export);
-        if($listboxEntry) { 
-          $return[$shareAttrs[0]] = $shareAttrs[0]." - ".$shareAttrs[4];
-        }else{
-          $return[$shareAttrs[0]]['name']         = $shareAttrs[0]; 
-          $return[$shareAttrs[0]]['description']  = $shareAttrs[1]; 
-          $return[$shareAttrs[0]]['type']         = $shareAttrs[2]; 
-          $return[$shareAttrs[0]]['charset']      = $shareAttrs[3]; 
-          $return[$shareAttrs[0]]['path']         = $shareAttrs[4]; 
-          $return[$shareAttrs[0]]['option']       = $shareAttrs[5]; 
-        }
-      }
-    }
-    return($return);
-  }
-
-
-/* This function returns all available ShareServer  
- */
-  function getShareServerList()
-  {
-    $ldap= $this->config->get_ldap_link();
-    $a_res = $ldap->search("(objectClass=goShareServer)",array("goExportEntry","cn"));
-    $return= array();
-    while($entry = $ldap->fetch($a_res)){
-      $return[$entry['cn'][0]] = $entry['cn'][0];
-    }
-    return($return);
-  }
-
 /* Generate ListBox frindly output for the defined shares 
  * Possibly Add or remove an attribute here, 
  */
@@ -844,7 +999,7 @@ class environment extends plugin
     $a_return = array();
     if(is_array($this->gotoShares)){
       foreach($this->gotoShares as $share){
-        $a_return[$share['name']]= $share['name']." ".$share['mountPoint'];
+        $a_return[$share['name']."|".$share['server']]= $share['server']."://".$share['name']." on ".$share['mountPoint'];
       }
     }
     return($a_return);
@@ -870,8 +1025,8 @@ function printOutHotPlugDevices()
   function printOutPrinterDevices()
   {
     $a_return = array();
-    if(is_array($this->gotoPrinters)){
-      foreach($this->gotoPrinters as $printer){
+    if(is_array($this->gotoPrinter)){
+      foreach($this->gotoPrinter as $printer){
         if($printer['mode'] == "admin"){
           $a_return[$printer['cn'][0]]= $printer['cn'][0]." - "._("Admin");
         }else{