TSK-2003: minor refactoring
This commit is contained in:
parent
3af17ece6a
commit
e40417f1fd
|
|
@ -422,8 +422,9 @@ public class WorkbasketServiceImpl implements WorkbasketService {
|
||||||
taskanaEngine.getEngine().getCurrentUserContext().getAccessIds()))
|
taskanaEngine.getEngine().getCurrentUserContext().getAccessIds()))
|
||||||
.map(this::getPermissionsFromWorkbasketAccessItem);
|
.map(this::getPermissionsFromWorkbasketAccessItem);
|
||||||
|
|
||||||
if (!grantedPermissions.isPresent()
|
if (grantedPermissions.isEmpty()
|
||||||
|| !grantedPermissions.get().containsAll(Arrays.asList(requestedPermissions))) {
|
|| !new HashSet<>(grantedPermissions.get())
|
||||||
|
.containsAll(Arrays.asList(requestedPermissions))) {
|
||||||
throw new MismatchedWorkbasketPermissionException(
|
throw new MismatchedWorkbasketPermissionException(
|
||||||
taskanaEngine.getEngine().getCurrentUserContext().getUserid(),
|
taskanaEngine.getEngine().getCurrentUserContext().getUserid(),
|
||||||
workbasketId,
|
workbasketId,
|
||||||
|
|
@ -456,8 +457,9 @@ public class WorkbasketServiceImpl implements WorkbasketService {
|
||||||
taskanaEngine.getEngine().getCurrentUserContext().getAccessIds()))
|
taskanaEngine.getEngine().getCurrentUserContext().getAccessIds()))
|
||||||
.map(this::getPermissionsFromWorkbasketAccessItem);
|
.map(this::getPermissionsFromWorkbasketAccessItem);
|
||||||
|
|
||||||
if (!grantedPermissions.isPresent()
|
if (grantedPermissions.isEmpty()
|
||||||
|| !grantedPermissions.get().containsAll(Arrays.asList(requestedPermissions))) {
|
|| !new HashSet<>(grantedPermissions.get())
|
||||||
|
.containsAll(Arrays.asList(requestedPermissions))) {
|
||||||
throw new MismatchedWorkbasketPermissionException(
|
throw new MismatchedWorkbasketPermissionException(
|
||||||
taskanaEngine.getEngine().getCurrentUserContext().getUserid(),
|
taskanaEngine.getEngine().getCurrentUserContext().getUserid(),
|
||||||
workbasketKey,
|
workbasketKey,
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue