moved double setter from accessItem to WorkbasketService

This commit is contained in:
Mustapha Zorgati 2020-01-08 13:41:51 +01:00
parent b47610b62e
commit 2fbb88d25f
2 changed files with 6 additions and 9 deletions

View File

@ -407,14 +407,6 @@ public class WorkbasketAccessItemImpl implements WorkbasketAccessItem {
this.permCustom12 = permCustom12; this.permCustom12 = permCustom12;
} }
public void setAccessIdWithSanitizing(String accessId) {
if (TaskanaEngineConfiguration.shouldUseLowerCaseForAccessIds()) {
setAccessId(accessId != null ? accessId.toLowerCase() : null);
} else {
setAccessId(accessId);
}
}
@Override @Override
public int hashCode() { public int hashCode() {
return Objects.hash( return Objects.hash(

View File

@ -18,6 +18,7 @@ import pro.taskana.WorkbasketPermission;
import pro.taskana.WorkbasketQuery; import pro.taskana.WorkbasketQuery;
import pro.taskana.WorkbasketService; import pro.taskana.WorkbasketService;
import pro.taskana.WorkbasketSummary; import pro.taskana.WorkbasketSummary;
import pro.taskana.configuration.TaskanaEngineConfiguration;
import pro.taskana.exceptions.DomainNotFoundException; import pro.taskana.exceptions.DomainNotFoundException;
import pro.taskana.exceptions.InvalidArgumentException; import pro.taskana.exceptions.InvalidArgumentException;
import pro.taskana.exceptions.InvalidWorkbasketException; import pro.taskana.exceptions.InvalidWorkbasketException;
@ -160,7 +161,11 @@ public class WorkbasketServiceImpl implements WorkbasketService {
public WorkbasketAccessItem newWorkbasketAccessItem(String workbasketId, String accessId) { public WorkbasketAccessItem newWorkbasketAccessItem(String workbasketId, String accessId) {
WorkbasketAccessItemImpl accessItem = new WorkbasketAccessItemImpl(); WorkbasketAccessItemImpl accessItem = new WorkbasketAccessItemImpl();
accessItem.setWorkbasketId(workbasketId); accessItem.setWorkbasketId(workbasketId);
accessItem.setAccessIdWithSanitizing(accessId); if (TaskanaEngineConfiguration.shouldUseLowerCaseForAccessIds()) {
accessItem.setAccessId(accessId != null ? accessId.toLowerCase() : null);
} else {
accessItem.setAccessId(accessId);
}
return accessItem; return accessItem;
} }