Code

Corrected kioskpath parsing
[gosa.git] / plugins / personal / environment / class_environment.inc
index 8dac38155451c77815df349b87826e2395a7863e..6ec5f58fce0dafe4dca5246c2ea4080180bf1aef 100644 (file)
@@ -141,7 +141,7 @@ class environment extends plugin
         $tmp = $tmp2 = array();
         $tmp = split("\|",$share);
         $tmp2['name']      =$tmp[0];
-        $tmp2['mountPoint']=$tmp[1];
+        $tmp2['mountPoint']=base64_decode($tmp[1]);
         $this->gotoShares[$tmp[0]]=$tmp2;
       }
     }
@@ -158,6 +158,12 @@ class environment extends plugin
       $this->useProfile = false;
     }
 
+    /* Set to group environment if we editing a group */
+    if(!isset($this->parent)){
+      $this->is_group = true;
+    }
+
+
   }
 
   function execute()
@@ -169,7 +175,10 @@ class environment extends plugin
        All these tab management is done here
     */
 
-    // 1. Account disabled . Editing from usermenu
+    /* 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);
@@ -179,65 +188,73 @@ 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_Account = true;
-        $this->is_group = true;
-        $this->uid          = $this->cn;
-        $this->attrs['uid'] = $this->cn;
+      $smarty->assign("is_account","true");
+      $this->is_group     = true;
+      $this->uid          = $this->cn;
+      $this->attrs['uid'] = $this->cn;
 
-          if (isset($_POST['modify_state'])){
+      /* Change state if needed */
+      if (isset($_POST['modify_state'])){
         $this->is_account= !$this->is_account;
       }
-        // 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."));
-        } else {
-          if((in_array("posixAccount",$this->attrs['objectClass']))){
-            // 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."));
-            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."));
-            return $display;
-          }
-        }
-
-
 
+      /* 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");
 
       /* 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((isset($this->attrs['objectClass']))&&((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;
           }
         }
@@ -400,7 +417,7 @@ 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']))||(preg_match("/[\|]/i",$_POST['gotoShareMountPoint']))){
-        print_red("You must specify a valid mount point.");
+        print_red(_("You must specify a valid mount point."));
       }else{
         $a_share = $this->gotoAvailableShares[$_POST['gotoShareSelection']];
         $s_mount = $_POST['gotoShareMountPoint'];
@@ -635,13 +652,13 @@ class environment extends plugin
     $realyUsedAttrs= array();
 
 
-    $path = search_config($this->config->data['MENU'],"environment", "KIOSKPATH"); 
+    $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);
+        print_red(_("Can't save new kiosk profiles, possibly permission denied for folder")." : ",$path);
       }else{
         fwrite($fp,$contents,strlen($contents));
       }
@@ -715,7 +732,7 @@ class environment extends plugin
         unset($attr['dn']);
         $ldap->modify($attr);
         if($ldap->get_error()!="Success"){
-          print_red("Printer saving ".$ldap->get_error());
+          print_red(_("Error while writing printer")." : ".$ldap->get_error());
         }
       }
     }    
@@ -794,7 +811,7 @@ class environment extends plugin
       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());
       }
     }
   
@@ -821,7 +838,7 @@ 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['name']."|".base64_encode($share['mountPoint']);
     }
 
     if(!empty($this->gotoKioskProfile)){