nosu
/
taskana
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

TSK-1441: Enable Admin to complete tasks already claimed by other users

This commit is contained in:
Joerg Heffner 2020-11-05 09:11:12 +01:00 committed by gitgoodjhe
parent 5ddb81a618
commit 81b89e1c08
2 changed files with 17 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/taskana-core/src/main/java/pro/taskana/task/internal/TaskServiceImpl.java
View File

@ -1247,7 +1247,8 @@ public class TaskServiceImpl implements TaskService {
.getEngine() .getEngine()
.getCurrentUserContext() .getCurrentUserContext()
.getAccessIds() .getAccessIds()
.contains(task.getOwner())) { .contains(task.getOwner())
&& !taskanaEngine.getEngine().isUserInRole(TaskanaRole.ADMIN)) {
throw new InvalidOwnerException( throw new InvalidOwnerException(
String.format( String.format(
"Owner of task %s is %s, but current user is %s ", "Owner of task %s is %s, but current user is %s ",

12
lib/taskana-core/src/test/java/acceptance/task/CompleteTaskAccTest.java
View File

@ -45,6 +45,18 @@ class CompleteTaskAccTest extends AbstractAccTest {
assertThat(completedTask.getModified()).isNotEqualTo(completedTask.getCreated()); assertThat(completedTask.getModified()).isNotEqualTo(completedTask.getCreated());
} }
@WithAccessId(user = "admin")
@Test
void should_completeClaimedTaskByAnotherUser_When_UserIsAdmin() throws Exception {
assertThat(TASK_SERVICE.getTask("TKI:000000000000000000000000000000000029").getState())
.isEqualTo(TaskState.CLAIMED);
Task completedTask = TASK_SERVICE.completeTask("TKI:000000000000000000000000000000000029");
assertThat(completedTask).isNotNull();
assertThat(completedTask.getCompleted()).isNotNull();
assertThat(completedTask.getState()).isEqualTo(TaskState.COMPLETED);
assertThat(completedTask.getModified()).isNotEqualTo(completedTask.getCreated());
}
@WithAccessId(user = "admin") @WithAccessId(user = "admin")
@WithAccessId(user = "taskadmin") @WithAccessId(user = "taskadmin")
@TestTemplate @TestTemplate