diff --git a/symphony-bdk-core/src/main/java/com/symphony/bdk/core/service/user/UserService.java b/symphony-bdk-core/src/main/java/com/symphony/bdk/core/service/user/UserService.java index 80495a2c6..291e4ec63 100644 --- a/symphony-bdk-core/src/main/java/com/symphony/bdk/core/service/user/UserService.java +++ b/symphony-bdk-core/src/main/java/com/symphony/bdk/core/service/user/UserService.java @@ -382,10 +382,20 @@ public Stream listAllUsersDetail(@Nonnull UserFilter filter, * @param roleId Role Id * @see Add Role */ - public void addRole(@Nonnull Long userId, @Nonnull RoleId roleId) { - StringId stringId = new StringId().id(roleId.name()); + public void addRole(@Nonnull Long userId, @Nonnull StringId roleId) { executeAndRetry("addRole", - () -> userApi.v1AdminUserUidRolesAddPost(authSession.getSessionToken(), userId, stringId)); + () -> userApi.v1AdminUserUidRolesAddPost(authSession.getSessionToken(), userId, roleId)); + } + + /** + * Add a role to an user. + * + * @param userId User Id + * @param roleId Role Id + * @see Add Role + */ + public void addRole(@Nonnull Long userId, @Nonnull RoleId roleId) { + addRole(userId, new StringId().id(roleId.name())); } /** @@ -406,10 +416,20 @@ public List listRoles() { * @param roleId Role Id * @see Remove Role */ - public void removeRole(@Nonnull Long userId, @Nonnull RoleId roleId) { - StringId stringId = new StringId().id(roleId.name()); + public void removeRole(@Nonnull Long userId, @Nonnull StringId roleId) { executeAndRetry("removeRole", - () -> userApi.v1AdminUserUidRolesRemovePost(authSession.getSessionToken(), userId, stringId)); + () -> userApi.v1AdminUserUidRolesRemovePost(authSession.getSessionToken(), userId, roleId)); + } + + /** + * Remove a role from an user. + * + * @param userId User Id + * @param roleId Role Id + * @see Remove Role + */ + public void removeRole(@Nonnull Long userId, @Nonnull RoleId roleId) { + removeRole(userId, new StringId().id(roleId.name())); } /**