From 0e8909350b1cc6b44ee9b98c4bf051bf1b15509b Mon Sep 17 00:00:00 2001
From: cajus <cajus@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Mon, 25 Jan 2010 13:58:37 +0000
Subject: [PATCH] Reverted sortable lists

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@15281 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 .../personal/posix/class_posixAccount.inc     | 59 ++++---------------
 gosa-core/plugins/personal/posix/generic.tpl  | 27 ++++++---
 2 files changed, 33 insertions(+), 53 deletions(-)

diff --git a/gosa-core/plugins/personal/posix/class_posixAccount.inc b/gosa-core/plugins/personal/posix/class_posixAccount.inc
index 015c1af48..cdf74fe9e 100644
--- a/gosa-core/plugins/personal/posix/class_posixAccount.inc
+++ b/gosa-core/plugins/personal/posix/class_posixAccount.inc
@@ -100,7 +100,6 @@ class posixAccount extends plugin
   var $uid= "";
   var $multiple_support = TRUE;
   var $groupMembership_some = array();
-  var $groupMembershipList;
 
   /* constructor, if 'dn' is set, the node loads the given
      'dn' from LDAP */
@@ -263,15 +262,6 @@ class posixAccount extends plugin
       session::set("sysfilter", $sysfilter);
     }
     $this->ui = get_userinfo();
-
-    /* Create list widget */
-    $this->groupMembershipList= new sortableListing($this->groupMembership, $this->generateGroupInfos());
-    $this->groupMembershipList->setDeleteable(true);
-    $this->groupMembershipList->setHeader(array(_("Group"),_("Description")));
-    $this->groupMembershipList->setColspecs(array('*','*'));
-    $this->groupMembershipList->setWidth("100%");
-    $this->groupMembershipList->setHeight("130px");
-    $this->groupMembershipList->setInstantDelete(false);
   }
 
 
@@ -282,10 +272,6 @@ class posixAccount extends plugin
     plugin::execute();
     $display= "";
 
-    /* Assign list acl */
-    $this->groupMembershipList->setAcl($this->getacl('memberUid', (!is_object($this->parent) && !session::is_set('edit'))));
-    $this->groupMembershipList->update();
-
     /* Log view */
     if($this->is_account && !$this->view_logged){
       $this->view_logged = TRUE;
@@ -373,12 +359,6 @@ class posixAccount extends plugin
       $this->delGroup ($_POST['group_list']);
     }
 
-    /* Delete selected groups */
-    $action= $this->groupMembershipList->getAction();
-    if ($action['action'] == 'delete') {
-      $this->delGroup (array($this->groupMembershipList->getKey($action['targets'][0])));
-    }
-
     /* Add user workstation? */
     if (isset($_POST["add_ws"])){
       $this->show_ws_dialog= TRUE;
@@ -522,7 +502,6 @@ class posixAccount extends plugin
     /* Show main page */
     $smarty= get_smarty();
     $smarty->assign("usePrototype", "true");
-    $smarty->assign("groupMembershipList", $this->groupMembershipList->render());
 
     /* In 'MyAccount' mode, we must remove write acls if we are not in editing mode. */ 
     $SkipWrite = (!isset($this->parent) || !$this->parent) && !session::is_set('edit');
@@ -569,6 +548,16 @@ class posixAccount extends plugin
     $smarty->assign("shells", $this->loginShellList);
     $smarty->assign("secondaryGroups", $this->secondaryGroups);
     $smarty->assign("primaryGroup", $this->primaryGroup);
+    if(!$this->multiple_support_active){
+      if (!count($this->groupMembership)){
+        $smarty->assign("groupMembership", array("&nbsp;"));
+      } else {
+        $smarty->assign("groupMembership", $this->groupMembership);
+      }
+    }else{
+      $smarty->assign("groupMembership", $this->groupMembership);
+      $smarty->assign("groupMembership_some", $this->groupMembership_some);
+    }
     if (count($this->groupMembership) > 16){
       $smarty->assign("groups", "too_many_for_nfs");
     } else {
@@ -738,8 +727,6 @@ class posixAccount extends plugin
       /* Save values to object */
       plugin::save_object();
 
-      /* Maintain list */
-      $this->groupMembershipList->save_object();
 
       /* Save force GID checkbox */
       if($this->acl_is_writeable("gidNumber") || $this->acl_is_writeable("uidNumber")){
@@ -1265,8 +1252,9 @@ class posixAccount extends plugin
       }
     }
 
-    // Sync new list
-    $this->groupMembershipList->setListData($this->groupMembership, $this->generateGroupInfos());
+    /* Sort groups */
+    asort ($this->groupMembership);
+    reset ($this->groupMembership);
   }
 
 
@@ -1284,9 +1272,6 @@ class posixAccount extends plugin
         }
       }
     }
-
-    // Sync new list
-    $this->groupMembershipList->setListData($this->groupMembership, $this->generateGroupInfos());
   }
 
 
@@ -1819,25 +1804,7 @@ class posixAccount extends plugin
     plugin::set_multi_edit_values($attrs);
     $this->groupMembership = $groups;
   }
-
-
-  function generateGroupInfos()
-  {
-    $tmp= array();
-
-    foreach ($this->groupMembership as $key => $value) {
-      if (!preg_match('/^([^[]+)\[(.*)\]$/', $value, $m)) {
-        $tmp[$key]= array("data" => array($value, ""));
-      } else {
-        $tmp[$key]= array("data" => array(rtrim($m[1]), $m[2]));
-      }
-    }
-
-    return ($tmp);
-  }
-
 }
 
-
 // vim:tabstop=2:expandtab:shiftwidth=2:filetype=php:syntax:ruler:
 ?>
diff --git a/gosa-core/plugins/personal/posix/generic.tpl b/gosa-core/plugins/personal/posix/generic.tpl
index 5fe78a567..ceb6fa86d 100644
--- a/gosa-core/plugins/personal/posix/generic.tpl
+++ b/gosa-core/plugins/personal/posix/generic.tpl
@@ -91,15 +91,28 @@
     <b style="color:red">{t}(Warning: more than 16 groups are not supported by NFS!){/t}</b>
     <br>
    {/if}
-   {render acl=$groupMembershipACL}
-     {$groupMembershipList}
-   {/render}
-   {render acl=$groupMembershipACL}
+{render acl=$groupMembershipACL}
+   <select style="width:100%; height:130px;" name="group_list[]" size=16 multiple >
+{if !$multiple_support}
+    {html_options options=$groupMembership}
+{else}
+	{foreach from=$groupMembership item=group key=key}
+		<option value="{$key}">{$group}&nbsp;({t}Common group{/t})</option>
+	{/foreach}
+	{foreach from=$groupMembership_some item=group key=key}
+		<option value="{$key}" 
+			style='color: #888888; background: #DDDDDD;background-color: #DDDDDD;'>{$group}&nbsp;({t}Groups differ{/t})</option>
+	{/foreach}
+{/if}
+   </select>
+{/render}
+   <br>
+{render acl=$groupMembershipACL}
    <input type=submit value="{msgPool type=addButton}" name="edit_groupmembership" >&nbsp;
-   {/render}
-<!--{render acl=$groupMembershipACL}
+{/render}
+{render acl=$groupMembershipACL}
    <input type=submit value="{msgPool type=delButton}" name="delete_groupmembership" >
-{/render}-->
+{/render}
   </td>
  </tr>
 </table>
-- 
2.30.2