From 43646dbf8de86dfa6408e064e6af0d8bc0b4b51c Mon Sep 17 00:00:00 2001
From: Jakob Sack <kde@jakobsack.de>
Date: Mon, 18 Apr 2011 09:30:37 +0200
Subject: [PATCH] User management works, now we need some usability expert ...

---
 admin/js/users.js         | 29 ++++++++++++++++++++++-------
 admin/templates/users.php |  4 ++--
 help/appinfo/app.php      |  2 +-
 3 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/admin/js/users.js b/admin/js/users.js
index ad1555c95a..6a9ae9fc11 100644
--- a/admin/js/users.js
+++ b/admin/js/users.js
@@ -23,7 +23,12 @@ $(document).ready(function(){
 						groups.push($(this).val());
 					}
 				});
-				$("#changegroups").prev().html( groups.join(", "));
+				if( groups.length == 0 ){
+					$("#changegroups").prev().html( '&nbsp;' );
+				}
+				else{
+					$("#changegroups").prev().html( groups.join(", "));
+				}
 			}
 			else{
 				alert( "something went wrong! sorry!" );
@@ -38,10 +43,15 @@ $(document).ready(function(){
 
 	// Manipulating the page after crteating a user
 	function userCreated( username, groups ){
+		// We need at least a space for showing the div
+		if( groups == "" ){
+			groups = '&nbps;';
+		}
+
 		// Add user to table
-		var newrow = '<tr x-uid="'+username+'"><td x-use="username"><span x-use="usernamespan">'+username+'</span></td>';
-		newrow = newrow+'<td x-use="usergroups">'+groups+'</td>';
-		newrow = newrow+'<td><a href="" class="edituserbutton">edit</a> | <a  class="removeuserbutton" href="">remove</a></td></tr>';
+		var newrow = '<tr x-uid="'+username+'"><td x-use="username"><div x-use="usernamediv">'+username+'</div></td>';
+		newrow = newrow+'<td x-use="usergroups"><div x-use="usergroupsdiv">'+groups+'</td>';
+		newrow = newrow+'<td><a class="removeuserbutton" href="">remove</a></td></tr>';
 		$("#usertable").append( newrow  );
 
 		// Clear forms
@@ -96,7 +106,7 @@ $(document).ready(function(){
 	//#########################################################################
 
 	// Password (clicking on user name)
-	$("span[x-use='usernamespan']").live( "click", function(){
+	$("div[x-use='usernamediv']").live( "click", function(){
 		if( togglepassword == "" || $(this).parent().parent().attr("x-uid") != togglepassword ){
 			togglepassword = $(this).parent().parent().attr("x-uid");
 			// Set the username!
@@ -128,7 +138,7 @@ $(document).ready(function(){
 	});
 
 	// Groups
-	$("span[x-use='usergroupsspan']").live( "click", function(){
+	$("div[x-use='usergroupsdiv']").live( "click", function(){
 		if( togglegroup == "" || $(this).parent().parent().attr("x-uid") != togglegroup){
 			togglegroup = $(this).parent().parent().attr("x-uid");
 			var groups = $(this).text();
@@ -161,7 +171,12 @@ $(document).ready(function(){
 					groups.push($(this).val());
 				}
 			});
-			$(this).html( groups.join(", "));
+			if( groups.length == 0 ){
+				$("#changegroups").prev().html( '&nbsp;' );
+			}
+			else{
+				$("#changegroups").prev().html( groups.join(", "));
+			}
 			$('#changegroups').hide();
 			togglegroup = "";
 		}
diff --git a/admin/templates/users.php b/admin/templates/users.php
index 509fca6410..e769dcd4fc 100644
--- a/admin/templates/users.php
+++ b/admin/templates/users.php
@@ -35,8 +35,8 @@
 	<tbody>
 		<?php foreach($_["users"] as $user): ?>
 			<tr x-uid="<?php echo $user["name"] ?>">
-				<td x-use="username"><span x-use="usernamespan"><?php echo $user["name"]; ?></span></td>
-				<td x-use="usergroups"><span x-use="usergroupsspan"><?php echo $user["groups"]; ?></span></td>
+				<td x-use="username"><div x-use="usernamediv"><?php echo $user["name"]; ?></div></td>
+				<td x-use="usergroups"><div x-use="usergroupsdiv"><?php if( $user["groups"] ){ echo $user["groups"]; }else{echo "&nbsp";} ?></div></td>
 				<td><a  class="removeuserbutton" href="">remove</a></td>
 			</tr>
 		<?php endforeach; ?>
diff --git a/help/appinfo/app.php b/help/appinfo/app.php
index ee308624aa..fd46085ef7 100644
--- a/help/appinfo/app.php
+++ b/help/appinfo/app.php
@@ -4,7 +4,7 @@ OC_APP::register( array( "order" => 1, "id" => "help", "name" => "Help" ));
 
 // Workaround for having help as the last entry always
 $entry = array( "id" => "help", "order" => 1000, "href" => OC_HELPER::linkTo( "help", "index.php" ), "name" => "Help", "icon" => OC_HELPER::imagePath( "help", "help.png" ));
-if( OC_GROUP::inGroup( $_SESSION["user_id"], "admin" )){
+if( isset( $_SESSION["user_id"] ) && OC_GROUP::inGroup( $_SESSION["user_id"], "admin" )){
 	OC_APP::addAdminPage( $entry );
 }
 else{
-- 
GitLab