From: cajus <cajus@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Fri, 28 Mar 2008 14:33:12 +0000 (+0000)
Subject: Dialog unifying
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=1dc6e38d7cc2d20f66971a19d7b038639bd2780b;p=gosa.git

Dialog unifying


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

diff --git a/gosa-plugins/samba/personal/samba/class_sambaAccount.inc b/gosa-plugins/samba/personal/samba/class_sambaAccount.inc
index 7c197fa1f..65830c6d5 100644
--- a/gosa-plugins/samba/personal/samba/class_sambaAccount.inc
+++ b/gosa-plugins/samba/personal/samba/class_sambaAccount.inc
@@ -338,27 +338,34 @@ class sambaAccount extends plugin
     }
 
     /* Add user workstation finished? */
-    if (isset($_POST["add_ws_finish"]) || isset($_POST["add_ws_cancel"])){
+    if (isset($_POST["add_ws_cancel"])){
       $this->show_ws_dialog= FALSE;
       $this->dialog= FALSE;
     }
 
     /* Add user workstation? */
-    if (isset($_POST["add_ws_finish"]) && isset($_POST['wslist'])){
+    if (isset($_POST["add_ws_finish"])){
 
-      if($this->multiple_support_active){
-        foreach($_POST['wslist'] as $ws){
-          $this->multiple_sambaUserWorkstations[trim($we)] = array("Name" => trim($ws), "UsedByAllUsers" => TRUE);
-        }
-      }else{
-        $tmp= $this->sambaUserWorkstations;
-        foreach($_POST['wslist'] as $ws){
-          $tmp.= ",$ws";
+      if (isset($_POST['wslist'])){
+        if($this->multiple_support_active){
+          foreach($_POST['wslist'] as $ws){
+            $this->multiple_sambaUserWorkstations[trim($we)] = array("Name" => trim($ws), "UsedByAllUsers" => TRUE);
+          }
+        }else{
+          $tmp= $this->sambaUserWorkstations;
+          foreach($_POST['wslist'] as $ws){
+            $tmp.= ",$ws";
+          }
+          $tmp= preg_replace('/,+/', ',', $tmp);
+          $this->sambaUserWorkstations= trim($tmp, ',');
         }
-        $tmp= preg_replace('/,+/', ',', $tmp);
-        $this->sambaUserWorkstations= trim($tmp, ',');
+        $this->is_modified= TRUE;
+
+        $this->show_ws_dialog= FALSE;
+        $this->dialog= FALSE;
+      } else {
+        msg_dialog::display(_("Error"), _("Please select an entry!"), ERROR_DIALOG);
       }
-      $this->is_modified= TRUE;
     }
 
     /* Show ws dialog */
diff --git a/gosa-plugins/samba/personal/samba/samba3_workstations.tpl b/gosa-plugins/samba/personal/samba/samba3_workstations.tpl
index f403a434a..c44f17c3a 100644
--- a/gosa-plugins/samba/personal/samba/samba3_workstations.tpl
+++ b/gosa-plugins/samba/personal/samba/samba3_workstations.tpl
@@ -1,6 +1,6 @@
 <table summary="" style="width:100%; vertical-align:top; text-align:left;" cellpadding="4">
 <tr>
-  <td style="vertical-align:top;width:600px">
+  <td style="vertical-align:top;">
    <div class="contentboxh" style="height:20px;">
     <p class="contentboxh" style="font-size:12px">
      <b><label for="wslist">{t}Select workstations to add{/t}</label> {$hint}</b><br />
@@ -8,13 +8,13 @@
    </div>
    <div class="contentboxb">
     <p class="contentboxb" style="border-top:1px solid #B0B0B0;background-color:#F8F8F8">
-     <select id="wslist" style="width:600px; margin-top:4px; height:450px;" name="wslist[]" size="15" multiple>
+     <select id="wslist" style="width:100%; margin-top:4px; height:450px;" name="wslist[]" size="15" multiple>
 	{html_options values=$wslist output=$wslist}
      </select>
     </p>
    </div>
   </td>
-  <td style="vertical-align:top;">
+  <td style="vertical-align:top;width:350px">
    <div class="contentboxh" style="height:20px;">
     <p class="contentboxh" style="font-size:12px"><img src="{$launchimage}" align="right" alt="[F]">
 	<b>{t}Filters{/t}</b></p>
diff --git a/gosa-plugins/samba/personal/samba/sambaLogonHours.tpl b/gosa-plugins/samba/personal/samba/sambaLogonHours.tpl
index 24d2199b9..3f5ab3029 100644
--- a/gosa-plugins/samba/personal/samba/sambaLogonHours.tpl
+++ b/gosa-plugins/samba/personal/samba/sambaLogonHours.tpl
@@ -91,7 +91,7 @@
 <input type='hidden' name='sambaLogonHoursPosted' value='1'> 
 <br>
 <p class="plugbottom">
-  <input type=submit name="save_logonHours" value="{msgPool type=applyButton}">
+  <input type=submit name="save_logonHours" value="{msgPool type=saveButton}">
   &nbsp;
   <input type=submit name="cancel_logonHours" value="{msgPool type=cancelButton}">
 </p>