From 515c3aa1d74365df93b1bf6fe3c17f5005267dcb Mon Sep 17 00:00:00 2001
From: hickert <hickert@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Wed, 11 May 2011 11:13:55 +0000
Subject: [PATCH] Updated partition handling

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@20804 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 .../goto/Device/AddPartitionDialog.tpl        | 155 ++++++++++--------
 .../goto/Device/class_AddPartitionDialog.inc  |  37 ++---
 2 files changed, 102 insertions(+), 90 deletions(-)

diff --git a/gosa-plugins/goto/admin/systems/goto/Device/AddPartitionDialog.tpl b/gosa-plugins/goto/admin/systems/goto/Device/AddPartitionDialog.tpl
index ae9cf1a43..315b60b1c 100644
--- a/gosa-plugins/goto/admin/systems/goto/Device/AddPartitionDialog.tpl
+++ b/gosa-plugins/goto/admin/systems/goto/Device/AddPartitionDialog.tpl
@@ -1,76 +1,93 @@
-<h3>{t}Add Partition{/t}</h3>
 
-<table>
-    <tr>
-        <td>{t}Mount point{/t}</td>
-        <td>
-            {if $fsType == "raid" || $fsType == "swap" || $fsType == "pv"}
-                <input disabled type="text" name="mountPoint" value=" - ">
-            {else}
-                <input type="text" name="mountPoint" value="{$mountPoint}">
-            {/if}
-        </td>
-    </tr>
-    <tr>
-        <td>{t}File system type{/t}</td>
-        <td>
-            <select name="fsType" onChange="document.mainform.submit();">
-                {html_options options=$fsTypes selected=$fsType}
-            </select>
-        </td>
-    </tr>
-    <tr>
-        <td>{t}Allowable drives{/t}</td>
-        <td>
-            {foreach from=$disks item=item key=key}
-                <input type="checkbox" {if $disk_selected[$item]} checked {/if} 
-                    name="disk_selected_{$item}">{$item}
-            {/foreach}
-        </td>
-    </tr>
-    <tr>
-        <td>{t}Size{/t}</td>
-        <td>
-            <input name="size" value="{$size}">
-        </td>
-    </tr>
-    <tr>
-        <td><input type="checkbox" name="forcePrimary" {if $forcePrimary_selected} checked {/if}></td>
-        <td>{t}Force to be primary partition{/t}</td>
-    </tr>
-    <tr>
-        <td><input type="checkbox" name="encrypt" {if $encrypt_selected} checked {/if}></td>
-        <td>{t}Encrypt{/t}</td>
-    </tr>
-</table>
+<h3>{t}Type{/t}</h3>
+
+<input {if $selected_type==1} checked {/if} onClick="document.mainform.submit();"
+        type="radio" value="1" name="selected_type">{t}Physical partition{/t}<br>
+<input {if $selected_type==2} checked {/if} onClick="document.mainform.submit();"
+        type="radio" value="2" name="selected_type">{t}Raid device{/t}<br>
+<input {if $selected_type==3} checked {/if} onClick="document.mainform.submit();"
+        type="radio" value="3" name="selected_type">{t}LVM Valume group{/t}<br>
+<input {if $selected_type==4} checked {/if} onClick="document.mainform.submit();"
+        type="radio" value="4" name="selected_type">{t}LVM Valume{/t}<br>
 
 <hr>
 
-<h3>{t}Additional size options{/t}</h3>
-<table>
-    <tr>
-        <td><input type="radio" name="size_options" value="0" 
-                onClick="document.mainform.submit();"
-                {if $size_options==0} checked {/if}></td>
-        <td>{t}Fixed size{/t}</td>
-    </tr>
-    <tr>
-        <td><input type="radio" name="size_options" value="1" 
-                onClick="document.mainform.submit();"
-                {if $size_options==1} checked {/if}></td>
-        <td>{t}Fill all space up to{/t} 
-            <input {if $size_options != 1} disabled {/if}
-                    id="size_max_value"
-                    type="text" value="{$size_max_value}">&nbsp;{t}MB{/t}
-        </td>
-    </tr>
-    <tr>
-        <td><input type="radio" name="size_options" value="2" 
-                onClick="document.mainform.submit();"
-                {if $size_options==2} checked {/if}></td>
-        <td>{t}Fill to maximum allowable size{/t}</td>
-    </tr>
-</table>
+
+{if $selected_type==1}
+    <h3>{t}Add Partition{/t}</h3>
+
+    <table>
+        <tr>
+            <td>{t}Mount point{/t}</td>
+            <td>
+                {if $fsType == "raid" || $fsType == "swap" || $fsType == "pv"}
+                    <input disabled type="text" name="mountPoint" value=" - ">
+                {else}
+                    <input type="text" name="mountPoint" value="{$mountPoint}">
+                {/if}
+            </td>
+        </tr>
+        <tr>
+            <td>{t}File system type{/t}</td>
+            <td>
+                <select name="fsType" onChange="document.mainform.submit();">
+                    {html_options options=$fsTypes selected=$fsType}
+                </select>
+            </td>
+        </tr>
+        <tr>
+            <td>{t}Allowable drives{/t}</td>
+            <td>
+                {foreach from=$disks item=item key=key}
+                    <input type="radio" {if $used_disk == $item} checked {/if} 
+                        name="used_disk" value="{$item}">{$item}
+                {/foreach}
+            </td>
+        </tr>
+        <tr>
+            <td>{t}Size{/t}</td>
+            <td>
+                <input name="size" value="{$size}">
+            </td>
+        </tr>
+        <tr>
+            <td><input type="checkbox" name="forcePrimary" {if $forcePrimary_selected} checked {/if}></td>
+            <td>{t}Force to be primary partition{/t}</td>
+        </tr>
+        <tr>
+            <td><input type="checkbox" name="encrypt" {if $encrypt_selected} checked {/if}></td>
+            <td>{t}Encrypt{/t}</td>
+        </tr>
+    </table>
+
+    <hr>
+
+    <h3>{t}Additional size options{/t}</h3>
+    <table>
+        <tr>
+            <td><input type="radio" name="size_options" value="0" 
+                    onClick="document.mainform.submit();"
+                    {if $size_options==0} checked {/if}></td>
+            <td>{t}Fixed size{/t}</td>
+        </tr>
+        <tr>
+            <td><input type="radio" name="size_options" value="1" 
+                    onClick="document.mainform.submit();"
+                    {if $size_options==1} checked {/if}></td>
+            <td>{t}Fill all space up to{/t} 
+                <input {if $size_options != 1} disabled {/if}
+                        id="size_max_value"
+                        type="text" value="{$size_max_value}">&nbsp;{t}MB{/t}
+            </td>
+        </tr>
+        <tr>
+            <td><input type="radio" name="size_options" value="2" 
+                    onClick="document.mainform.submit();"
+                    {if $size_options==2} checked {/if}></td>
+            <td>{t}Fill to maximum allowable size{/t}</td>
+        </tr>
+    </table>
+{/if}
 
 <hr>
 <div class="clear"></div>
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 b76db637c..b6bde4edc 100644
--- a/gosa-plugins/goto/admin/systems/goto/Device/class_AddPartitionDialog.inc
+++ b/gosa-plugins/goto/admin/systems/goto/Device/class_AddPartitionDialog.inc
@@ -1,24 +1,32 @@
 <?php
 
+
+define('PARTITION' , 1);
+define('RAID_DEVICE' , 2);
+define('VOLUME_GROUP' , 3);
+define('VOLUME' , 4);
+
+
+
 class AddPartitionDialog
 {
     public $partitionObject;
     public $disks = array();
+    public $selected_type = PARTITION;
 
     // Partition properties
     public $size = 1000;
     public $fsType = 'ext3';
     public $mountPoint = '';
-    public $usedDisks = array();
+    public $used_disk = array();
     public $forcePrimary = FALSE;
     public $encrypt = FALSE;
     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");
-
+    public $attributes = array("selected_type", "size", "fsType", "mountPoint", "forcePrimary", "encrypt", 
+            "size_options", "size_max_value", "used_disk");
 
     /*! \brief  Constructs the Dialog and loads all required informations
      *          to be able to add partitions, raid devices, volumes groups ...
@@ -47,9 +55,9 @@ class AddPartitionDialog
         foreach($disks as $disk){
             $this->disks[$disk['device']] = $disk['device'];
         }
-        if(!count($this->usedDisks)){
-            $this->usedDisks[] = key($this->disks);
-        }
+    
+        // Select first disk as default.    
+        $this->used_disk = key($this->disks);
     }
     
 
@@ -62,11 +70,6 @@ class AddPartitionDialog
         foreach($this->attributes as $attr){
             $smarty->assign($attr, $this->$attr);
         }
-        $used = array();
-        foreach($this->disks as $disk){
-            $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);
         }
@@ -92,14 +95,6 @@ class AddPartitionDialog
         foreach(array("forcePrimary", "encrypt") as $attr){
             $this->$attr = isset($_POST[$attr]);
         }
-
-        // Get selected disks
-        $this->usedDisks = array();
-        foreach($this->disks as $disk){
-            if(isset($_POST['disk_selected_'.$disk])){
-                $this->usedDisks[] = $disk;
-            }
-        }
     }
 
 
@@ -133,7 +128,7 @@ class AddPartitionDialog
         $fsOptions = "";
         $encrypt = $this->encrypt;
         $passphrase = "";
-        $disk = array_shift($this->usedDisks);
+        $disk = $this->used_disk;
 
         // We've to create a raid disk
         if($this->fsType == "raid"){
-- 
2.30.2