From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Wed, 11 May 2011 10:43:34 +0000 (+0000)
Subject: Addec comments
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=8682e061e20333a6c5ff340a6f037df2835d7c2e;p=gosa.git

Addec comments


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

diff --git a/gosa-plugins/goto/admin/systems/goto/Device/class_AddPartitionDialog.inc b/gosa-plugins/goto/admin/systems/goto/Device/class_AddPartitionDialog.inc
index e36708de5..b76db637c 100644
--- a/gosa-plugins/goto/admin/systems/goto/Device/class_AddPartitionDialog.inc
+++ b/gosa-plugins/goto/admin/systems/goto/Device/class_AddPartitionDialog.inc
@@ -5,6 +5,7 @@ class AddPartitionDialog
     public $partitionObject;
     public $disks = array();
 
+    // Partition properties
     public $size = 1000;
     public $fsType = 'ext3';
     public $mountPoint = '';
@@ -14,10 +15,16 @@ class AddPartitionDialog
     public $size_max_value = 1000;
     public $size_options = 0;
     
-
+    // Attributes managed by this plugin.
     public $attributes = array("size", "fsType", "mountPoint", "forcePrimary", "encrypt", 
             "size_options", "size_max_value");
 
+
+    /*! \brief  Constructs the Dialog and loads all required informations
+     *          to be able to add partitions, raid devices, volumes groups ...
+     *  @param  Config          The GOsa configuration object.
+     *  @param  remoteObject    The remote partition object.
+     */
     function __construct($config, $partitionObject)
     {
         $this->partitionObject = &$partitionObject;
@@ -43,9 +50,12 @@ class AddPartitionDialog
         if(!count($this->usedDisks)){
             $this->usedDisks[] = key($this->disks);
         }
-
     }
     
+
+    /*! \brief     Generates the HTML output for this plugin. 
+     *  @return    String   HTML content of the plugin. 
+     */
     function execute()
     {
         $smarty = get_smarty();
@@ -57,17 +67,18 @@ class AddPartitionDialog
             $used[$disk] = in_array($disk, $this->usedDisks);
         }
         $smarty->assign("disk_selected", $used);
-
         foreach(array("forcePrimary", "encrypt") as $attr){
             $smarty->assign("{$attr}_selected", $this->$attr != FALSE);
         }
         $smarty->assign('fsTypes', $this->fsTypes);
         $smarty->assign('size', $this->size);
         $smarty->assign('disks', $this->disks);
-
         return($smarty->fetch(get_template_path("goto/Device/AddPartitionDialog.tpl", TRUE)));
     }
 
+
+    /*! \brief     Saves posted values. 
+     */
     function save_object()
     {
         // Get posted string values 
@@ -91,6 +102,10 @@ class AddPartitionDialog
         }
     }
 
+
+    /*! \brief     Stores the changes back to the remote table model. 
+     *  @return    TRUE on success else false.
+     */
     function save()
     {
         // Get all currently used partitions
@@ -104,6 +119,22 @@ class AddPartitionDialog
             $usedTargets[] = $part['target'];
         }
 
+        // Collect options
+        $size = $this->size;
+        $maxSize = NULL;
+        if($this->size_options == 2){
+            $maxSize = $this->size_max_value;
+        }
+        $grow = $this->size_options == 1;
+        $format = FALSE;
+        $boot = FALSE;
+        $primary = $this->forcePrimary;
+        $fsType = $this->fsType;
+        $fsOptions = "";
+        $encrypt = $this->encrypt;
+        $passphrase = "";
+        $disk = array_shift($this->usedDisks);
+
         // We've to create a raid disk
         if($this->fsType == "raid"){
             $target="raid.";
@@ -112,12 +143,7 @@ class AddPartitionDialog
                 $id ++;
             }
             $target = $target.str_pad($id, 2, '0', STR_PAD_LEFT);
-            
-            // Get first selected disk
-            $disk = array_shift($this->usedDisks);
-            $this->partitionObject->addPartition($target,$this->size +0, NULL, FALSE, TRUE, FALSE, FALSE, 
-                    NULL, NULL , FALSE, NULL, $disk);  
-            return($this->partitionObject->success());
+            $fsType = $fsOptions = NULL;
         }else
 
         // We've to create a raid disk
@@ -128,21 +154,20 @@ class AddPartitionDialog
                 $id ++;
             }
             $target = $target.str_pad($id, 2, '0', STR_PAD_LEFT);
-            
-            // Get first selected disk
-            $disk = array_shift($this->usedDisks);
-            $this->partitionObject->addPartition($target,$this->size +0, NULL, FALSE, TRUE, FALSE, FALSE, 
-                    NULL, NULL , FALSE, NULL, $disk);  
-            return($this->partitionObject->success());
+            $fsType = $fsOptions = NULL;
         }else{
         
             // Add normal-physical partition
             $target = $this->mountPoint;
-            $disk = array_shift($this->usedDisks);
-            $this->partitionObject->addPartition($target,$this->size +0, NULL, FALSE, TRUE, FALSE, FALSE, 
-                    NULL, NULL , FALSE, NULL, $disk);  
-            return($this->partitionObject->success());
+            if($fsType == "swap"){
+                $target = "swap";
+            }
         }
+
+        // Add partition to remote model
+        $this->partitionObject->addPartition($target,$size, $maxSize, $grow, $format, $boot, $primary,
+                $fsType, $fsOptions , $encrypt, $passphrase, $disk);
+        return($this->partitionObject->success());
     }
 }
 ?>
diff --git a/gosa-plugins/goto/admin/systems/goto/Device/class_DevicePartition.inc b/gosa-plugins/goto/admin/systems/goto/Device/class_DevicePartition.inc
index cc3c2d5ec..4bba04eed 100644
--- a/gosa-plugins/goto/admin/systems/goto/Device/class_DevicePartition.inc
+++ b/gosa-plugins/goto/admin/systems/goto/Device/class_DevicePartition.inc
@@ -48,6 +48,9 @@ class DevicePartition
 
     function execute()
     {
+        /*****
+         * Handle add-partition dialog
+         *****/
         if(isset($_POST['create_partition'])){
             $this->addDialog = new AddPartitionDialog($this->config, $this->object); 
         }
@@ -66,6 +69,9 @@ class DevicePartition
         }
 
 
+        /*****
+         * Fill listing
+         *****/
 
         // Receive list informations
         $disks = $this->object->getDisks();
@@ -255,7 +261,7 @@ class DevicePartition
 
     function save()
     {
-        return("");
+        return($this->object->dump());
     }
 
     function __convertPartSize($size)