Ravi Nori has uploaded a new change for review.

Change subject: restapi : Create user API fails in REST & Java
......................................................................

restapi : Create user API fails in REST & Java

Unable to add new user using rest api
getting class cast exception

Change-Id: Ifed7f39b5b706482198afd74163791105b1d956a
Bug-Url: https://bugzilla.redhat.com/1127623
Signed-off-by: Ravi Nori <rn...@redhat.com>
---
M 
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupsResource.java
M 
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUsersResource.java
2 files changed, 20 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/26/31226/1

diff --git 
a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupsResource.java
 
b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupsResource.java
index 6936fc0..f75473b 100644
--- 
a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupsResource.java
+++ 
b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupsResource.java
@@ -22,7 +22,9 @@
 import org.ovirt.engine.core.common.businessentities.aaa.DbGroup;
 import org.ovirt.engine.core.common.interfaces.SearchType;
 import org.ovirt.engine.core.common.queries.DirectoryIdQueryParameters;
+import org.ovirt.engine.core.common.queries.DirectorySearchParameters;
 import org.ovirt.engine.core.common.queries.IdQueryParameters;
+import org.ovirt.engine.core.common.queries.SearchParameters;
 import org.ovirt.engine.core.common.queries.VdcQueryParametersBase;
 import org.ovirt.engine.core.common.queries.VdcQueryType;
 import org.ovirt.engine.core.compat.Guid;
@@ -209,6 +211,14 @@
     }
 
     @Override
+    protected SearchParameters createSearchParameters(SearchType searchType, 
String constraint) {
+        if (SearchType.DirectoryGroup.equals(searchType)) {
+            return new DirectorySearchParameters(constraint, searchType);
+        }
+        return super.createSearchParameters(searchType, constraint);
+    }
+
+    @Override
     public Response performRemove(String id) {
         return performAction(VdcActionType.RemoveGroup, new 
IdParameters(asGuid(id)));
     }
diff --git 
a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUsersResource.java
 
b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUsersResource.java
index 422501c..172f75d 100644
--- 
a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUsersResource.java
+++ 
b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUsersResource.java
@@ -23,7 +23,9 @@
 import org.ovirt.engine.core.common.action.VdcActionType;
 import org.ovirt.engine.core.common.businessentities.aaa.DbUser;
 import org.ovirt.engine.core.common.interfaces.SearchType;
+import org.ovirt.engine.core.common.queries.DirectorySearchParameters;
 import org.ovirt.engine.core.common.queries.IdQueryParameters;
+import org.ovirt.engine.core.common.queries.SearchParameters;
 import org.ovirt.engine.core.common.queries.VdcQueryParametersBase;
 import org.ovirt.engine.core.common.queries.VdcQueryType;
 import org.ovirt.engine.core.compat.Guid;
@@ -194,6 +196,14 @@
         return performCreate(VdcActionType.AddUser, parameters, resolver, 
BaseResource.class);
     }
 
+    @Override
+    protected SearchParameters createSearchParameters(SearchType searchType, 
String constraint) {
+        if (SearchType.DirectoryUser.equals(searchType)) {
+            return new DirectorySearchParameters(constraint, searchType);
+        }
+        return super.createSearchParameters(searchType, constraint);
+    }
+
     private boolean isNameContainsDomain(User user) {
         return ((user.getUserName().contains("@")) && 
(user.getUserName().indexOf('@') != user.getUserName().length() - 1));
     }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifed7f39b5b706482198afd74163791105b1d956a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
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