New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closes #2491: Set Owner of Task when Transferring #2548
base: v7.0.X
Are you sure you want to change the base?
Conversation
999b007
to
171399d
Compare
80c00dd
to
537fa89
Compare
The discussions took place in the closed PR for the same issue: #2495 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You introduced new Sonarcloud issues in the new code. Please fix them!
...y-provider/src/test/java/acceptance/events/task/CreateHistoryEventOnTaskTransferAccTest.java
Outdated
Show resolved
Hide resolved
lib/taskana-core/src/main/java/pro/taskana/task/api/TaskService.java
Outdated
Show resolved
Hide resolved
lib/taskana-core/src/test/java/acceptance/task/transfer/TransferTaskAccTest.java
Show resolved
Hide resolved
rest/taskana-rest-spring/src/main/java/pro/taskana/task/rest/TaskController.java
Outdated
Show resolved
Hide resolved
...n/java/pro/taskana/task/rest/assembler/BulkOperationResultsRepresentationModelAssembler.java
Outdated
Show resolved
Hide resolved
rest/taskana-rest-spring/src/test/java/pro/taskana/task/rest/TaskControllerIntTest.java
Show resolved
Hide resolved
rest/taskana-rest-spring/src/test/java/pro/taskana/task/rest/TaskControllerIntTest.java
Outdated
Show resolved
Hide resolved
rest/taskana-rest-spring/src/test/java/pro/taskana/task/rest/TaskControllerIntTest.java
Show resolved
Hide resolved
...va/pro/taskana/task/rest/assembler/BulkOperationResultsRepresentationModelAssemblerTest.java
Outdated
Show resolved
Hide resolved
...va/pro/taskana/task/rest/assembler/BulkOperationResultsRepresentationModelAssemblerTest.java
Outdated
Show resolved
Hide resolved
I checked it again and cannot find the issues introduced by new code |
51d6779
to
c0df871
Compare
...va/pro/taskana/task/rest/assembler/BulkOperationResultsRepresentationModelAssemblerTest.java
Outdated
Show resolved
Hide resolved
You are right! |
d2ec2d3
to
cca8a7a
Compare
...va/pro/taskana/task/rest/assembler/BulkOperationResultsRepresentationModelAssemblerTest.java
Outdated
Show resolved
Hide resolved
cca8a7a
to
79d8d68
Compare
* transferred | ||
* @param destinationWorkbasketId the {@linkplain Workbasket#getId() id} of the target {@linkplain | ||
* Workbasket} | ||
* @return the transferred {@linkplain Task} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't the boolean parameter missing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This transfer() function does not have the setTransferFlag() boolean parameter, but it calls another transfer() function that has the boolean parameter. It does not make sense to include the boolean parameter here, since it does not directly have the boolean parameter.
* transferred | ||
* @param workbasketKey the {@linkplain Workbasket#getKey() key} of the target {@linkplain | ||
* Workbasket} | ||
* @param domain the {@linkplain Workbasket#getDomain() domain} of the target {@linkplain |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see above.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above
/** | ||
* Transfers a List of {@linkplain Task Tasks} to another {@linkplain Workbasket} while always | ||
* setting {@linkplain Task#isTransferred isTransferred} to true. | ||
* | ||
* @param destinationWorkbasketId {@linkplain Workbasket#getId() id} of the target {@linkplain | ||
* Workbasket} | ||
* @param taskIds List of source {@linkplain Task Tasks} which will be moved |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above
* @param destinationWorkbasketKey target {@linkplain Workbasket#getKey() key} | ||
* @param destinationWorkbasketDomain target {@linkplain Workbasket#getDomain() domain} | ||
* @param taskIds List of source {@linkplain Task Tasks} which will be moved | ||
* @return Bulkresult with {@linkplain Task#getId() ids} and Error for each failed transactions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above
79d8d68
to
d6f2590
Compare
https://sonarcloud.io/summary/new_code?id=jamesrdi_taskana&branch=TSK-2491-BackupNew
Release Notes:
For the submitter:
Verified by the reviewer: