Juan Hernandez has uploaded a new change for review. Change subject: restapi: Don't add external id to domain users ......................................................................
restapi: Don't add external id to domain users Domain users only have an identifier, the one assigned by the external directory. Change-Id: I3f1e84f1615718faa25332daa54d9ba0a9cfdbee Signed-off-by: Juan Hernandez <[email protected]> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GroupMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/UserMapper.java 2 files changed, 0 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/92/23292/1 diff --git a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GroupMapper.java b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GroupMapper.java index 7e8658d..5e38e53 100644 --- a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GroupMapper.java +++ b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GroupMapper.java @@ -28,7 +28,6 @@ Group model = template != null ? template : new Group(); model.setName(entity.getName()); model.setId(entity.getId().toHex()); - model.setExternalId(entity.getId().toString()); if (!StringUtils.isEmpty(entity.getDirectory().getName())) { Domain dom = new Domain(); dom.setId(new Guid(entity.getDirectory().getName().getBytes(), true).toString()); diff --git a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/UserMapper.java b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/UserMapper.java index b99124e..c36002e 100644 --- a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/UserMapper.java +++ b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/UserMapper.java @@ -42,7 +42,6 @@ public static User map(DirectoryUser entity, User template) { User model = template != null ? template : new User(); model.setName(entity.getFirstName()); - model.setExternalId(entity.getId().toString()); model.setUserName(entity.getName()); model.setId(entity.getId().toHex()); model.setLastName(entity.getLastName()); -- To view, visit http://gerrit.ovirt.org/23292 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3f1e84f1615718faa25332daa54d9ba0a9cfdbee Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
