From a423c38e42c171806e5e94fbb9487ed4cd19af4c Mon Sep 17 00:00:00 2001
From: Stian Thorgersen <stianst@gmail.com>
Date: Fri, 6 Dec 2013 19:08:50 +0000
Subject: [PATCH] KEYCLOAK-214 view all users in a realm through admin console

---
 .../resources/admin/js/controllers/users.js   | 15 ++++++++-------
 .../resources/admin/partials/user-list.html   |  4 ++--
 .../java/org/keycloak/models/RealmModel.java  |  2 ++
 .../org/keycloak/models/jpa/RealmAdapter.java |  9 +++++++++
 .../models/picketlink/RealmAdapter.java       | 19 ++++++++++++-------
 .../resources/admin/UsersResource.java        | 19 +++++++++++--------
 6 files changed, 44 insertions(+), 24 deletions(-)

diff --git a/admin-ui/src/main/resources/META-INF/resources/admin/js/controllers/users.js b/admin-ui/src/main/resources/META-INF/resources/admin/js/controllers/users.js
index de73f6333a..f2c6b7c1fc 100755
--- a/admin-ui/src/main/resources/META-INF/resources/admin/js/controllers/users.js
+++ b/admin-ui/src/main/resources/META-INF/resources/admin/js/controllers/users.js
@@ -120,15 +120,16 @@ module.controller('UserListCtrl', function($scope, realm, User) {
     $scope.realm = realm;
     $scope.searchQuery = function() {
         $scope.searchLoaded = false;
+        $scope.currentSearch = $scope.search;
+
+        var params = { realm: realm.id };
         if ($scope.search) {
-            $scope.currentSearch = $scope.search;
-            $scope.users = User.query({ realm: realm.id, search : $scope.search }, function() {
-                $scope.searchLoaded = true;
-            });
-        } else {
-            $scope.currentSearch = null;
-            $scope.users = null;
+            params.search = $scope.search;
         }
+
+        $scope.users = User.query(params, function() {
+            $scope.searchLoaded = true;
+        });
     };
 });
 
diff --git a/admin-ui/src/main/resources/META-INF/resources/admin/partials/user-list.html b/admin-ui/src/main/resources/META-INF/resources/admin/partials/user-list.html
index c79b243125..b29f3ebcc3 100755
--- a/admin-ui/src/main/resources/META-INF/resources/admin/partials/user-list.html
+++ b/admin-ui/src/main/resources/META-INF/resources/admin/partials/user-list.html
@@ -14,7 +14,7 @@
                 </ol>
                 <h2><span>{{realm.realm}}</span> Users</h2>
                 <table>
-                    <caption data-ng-show="users && search" class="hidden">Table of realm users</caption>
+                    <caption data-ng-show="users" class="hidden">Table of realm users</caption>
                     <thead>
                         <tr>
                             <th class="rcue-table-actions" colspan="4">
@@ -66,7 +66,7 @@
                 <div class="feedback warning inline" data-ng-show="searchLoaded && users.length == 0">
                     <p><strong>Your search returned no results.</strong><br>Try modifying the query and try again.</p>
                 </div>
-                <div class="feedback info inline" data-ng-show="!currentSearch">
+                <div class="feedback info inline" data-ng-show="!users">
                     <br/>
                     <p><strong>Search for users.</strong><br>Please enter a search to display users.</p>
                 </div>
diff --git a/model/api/src/main/java/org/keycloak/models/RealmModel.java b/model/api/src/main/java/org/keycloak/models/RealmModel.java
index a4527ae1aa..ce76dcb45a 100755
--- a/model/api/src/main/java/org/keycloak/models/RealmModel.java
+++ b/model/api/src/main/java/org/keycloak/models/RealmModel.java
@@ -137,6 +137,8 @@ public interface RealmModel extends RoleContainerModel, RoleMapperModel, ScopeMa
 
     public void setAutomaticRegistrationAfterSocialLogin(boolean automaticRegistrationAfterSocialLogin);
 
+    List<UserModel> getUsers();
+
     List<UserModel> searchForUser(String search);
 
     List<UserModel> searchForUserByAttributes(Map<String, String> attributes);
diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/RealmAdapter.java b/model/jpa/src/main/java/org/keycloak/models/jpa/RealmAdapter.java
index 6e6c8f1186..0e8f879739 100755
--- a/model/jpa/src/main/java/org/keycloak/models/jpa/RealmAdapter.java
+++ b/model/jpa/src/main/java/org/keycloak/models/jpa/RealmAdapter.java
@@ -670,6 +670,15 @@ public class RealmAdapter implements RealmModel {
         em.flush();
     }
 
+    @Override
+    public List<UserModel> getUsers() {
+        TypedQuery<UserEntity> query = em.createQuery("select u from UserEntity u", UserEntity.class);
+        List<UserEntity> results = query.getResultList();
+        List<UserModel> users = new ArrayList<UserModel>();
+        for (UserEntity entity : results) users.add(new UserAdapter(entity));
+        return users;
+    }
+
     @Override
     public List<UserModel> searchForUser(String search) {
         TypedQuery<UserEntity> query = em.createQuery("select u from UserEntity u where u.realm = :realm and lower(u.loginName) like :search or lower(concat(u.firstName, ' ', u.lastName)) like :search or u.email like :search", UserEntity.class);
diff --git a/model/picketlink/src/main/java/org/keycloak/models/picketlink/RealmAdapter.java b/model/picketlink/src/main/java/org/keycloak/models/picketlink/RealmAdapter.java
index 798643e815..fb2dca68f1 100755
--- a/model/picketlink/src/main/java/org/keycloak/models/picketlink/RealmAdapter.java
+++ b/model/picketlink/src/main/java/org/keycloak/models/picketlink/RealmAdapter.java
@@ -914,23 +914,28 @@ public class RealmAdapter implements RealmModel {
         getRelationshipManager().remove(relationship);
     }
 
+    @Override
+    public List<UserModel> getUsers() {
+        List<UserModel> userModels = new ArrayList<UserModel>();
+        IdentityQuery<User> query = getIdm().createIdentityQuery(User.class);
+        for (User u : query.getResultList()) {
+            userModels.add(new UserAdapter(u, idm));
+        }
+        return userModels;
+    }
+
     @Override
     public List<UserModel> searchForUser(String search) {
         QueryParameter[] params = new QueryParameter[] { User.LOGIN_NAME, User.FIRST_NAME, User.LAST_NAME, User.EMAIL };
 
-        Map<String, User> users = new HashMap<String, User>();
+        List<UserModel> userModels = new ArrayList<UserModel>();
         for (QueryParameter p : params) {
             IdentityQuery<User> query = getIdm().createIdentityQuery(User.class);
             query.setParameter(p, search.toLowerCase());
             for (User u : query.getResultList()) {
-                users.put(u.getLoginName(), u);
+                userModels.add(new UserAdapter(u, idm));
             }
         }
-
-        List<UserModel> userModels = new ArrayList<UserModel>();
-        for (User user : users.values()) {
-            userModels.add(new UserAdapter(user, idm));
-        }
         return userModels;
     }
 
diff --git a/services/src/main/java/org/keycloak/services/resources/admin/UsersResource.java b/services/src/main/java/org/keycloak/services/resources/admin/UsersResource.java
index 61e70bf682..e8e34c76e1 100755
--- a/services/src/main/java/org/keycloak/services/resources/admin/UsersResource.java
+++ b/services/src/main/java/org/keycloak/services/resources/admin/UsersResource.java
@@ -135,14 +135,10 @@ public class UsersResource {
                                              @QueryParam("username") String username) {
         RealmManager manager = new RealmManager(session);
         List<UserRepresentation> results = new ArrayList<UserRepresentation>();
+        List<UserModel> userModels;
         if (search != null) {
-            List<UserModel> userModels = manager.searchUsers(search, realm);
-            for (UserModel user : userModels) {
-                if (isUser(user)) {
-                    results.add(manager.toRepresentation(user));
-                }
-            }
-        } else {
+            userModels = manager.searchUsers(search, realm);
+        } else if (last != null || first != null || email != null || username != null) {
             Map<String, String> attributes = new HashMap<String, String>();
             if (last != null) {
                 attributes.put(UserModel.LAST_NAME, last);
@@ -156,11 +152,18 @@ public class UsersResource {
             if (username != null) {
                 attributes.put(UserModel.LOGIN_NAME, username);
             }
-            List<UserModel> userModels = realm.searchForUserByAttributes(attributes);
+            userModels = realm.searchForUserByAttributes(attributes);
             for (UserModel user : userModels) {
                 results.add(manager.toRepresentation(user));
             }
+        } else {
+            userModels = realm.getUsers();
+        }
 
+        for (UserModel user : userModels) {
+            if (isUser(user)) {
+                results.add(manager.toRepresentation(user));
+            }
         }
         return results;
     }