Ravi Nori has uploaded a new change for review.

Change subject: webadmin, aaa: Sorting by 'authorization provider' in 'users' 
tab don't sort
......................................................................

webadmin, aaa: Sorting by 'authorization provider' in 'users' tab don't sort

Sorting users by 'authorization provider' does not
work in webadmin users tab

Change-Id: Ia1597c2b31b23c4352a883ff75e4f06f06070791
Bug-Url: https://bugzilla.redhat.com/1194262
Signed-off-by: Ravi Nori <rn...@redhat.com>
---
M 
backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VdcUserConditionFieldAutoCompleter.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabUserView.java
2 files changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/89/37989/1

diff --git 
a/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VdcUserConditionFieldAutoCompleter.java
 
b/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VdcUserConditionFieldAutoCompleter.java
index a0244a6..da867c4 100644
--- 
a/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VdcUserConditionFieldAutoCompleter.java
+++ 
b/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VdcUserConditionFieldAutoCompleter.java
@@ -6,6 +6,7 @@
     public static final String LAST_NAME = "LASTNAME";
     public static final String USER_NAME = "USRNAME";
     public static final String GROUP = "GROUP";
+    public static final String DIRECTORY = "DIRECTORY";
 
     public enum UserOrGroup {
         User,
@@ -19,7 +20,7 @@
         mVerbs.add(LAST_NAME);
         mVerbs.add(USER_NAME);
         mVerbs.add("LOGIN");
-        mVerbs.add("DIRECTORY");
+        mVerbs.add(DIRECTORY);
         mVerbs.add("DEPARTMENT");
         mVerbs.add(GROUP);
         mVerbs.add("TITLE");
@@ -37,7 +38,7 @@
         getTypeDictionary().put(LAST_NAME, String.class);
         getTypeDictionary().put(USER_NAME, String.class);
         getTypeDictionary().put("LOGIN", String.class);
-        getTypeDictionary().put("DIRECTORY", String.class);
+        getTypeDictionary().put(DIRECTORY, String.class);
         getTypeDictionary().put("DEPARTMENT", String.class);
         getTypeDictionary().put("TITLE", String.class);
         getTypeDictionary().put(GROUP, String.class);
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabUserView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabUserView.java
index 5cfdaed..5b370b8 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabUserView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabUserView.java
@@ -69,7 +69,7 @@
                 return object.getDomain();
             }
         };
-        authzColumn.makeSortable();
+        authzColumn.makeSortable(VdcUserConditionFieldAutoCompleter.DIRECTORY);
         getTable().addColumn(authzColumn, constants.authz(), "150px"); 
//$NON-NLS-1$
 
         TextColumnWithTooltip<DbUser> namespaceColumn = new 
TextColumnWithTooltip<DbUser>() {


-- 
To view, visit http://gerrit.ovirt.org/37989
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia1597c2b31b23c4352a883ff75e4f06f06070791
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to