From 6a53d0ce0d2678a0a567af41fd7a8659492ca711 Mon Sep 17 00:00:00 2001 From: Marcel Lengl <52546181+LenglBoy@users.noreply.github.com> Date: Thu, 23 Nov 2017 09:48:42 +0100 Subject: [PATCH 1/2] Throwing 'IllegalArgumentException: 1' corrected by changing some mappings. --- .../pro/taskana/TaskanaProducersTest.java | 3 --- .../impl/ClassificationServiceImpl.java | 6 ----- .../impl/persistence/TaskQueryImpl.java | 4 ++- .../model/mappings/ClassificationMapper.java | 27 ++++++++++++++++++- .../taskana/model/mappings/QueryMapper.java | 3 ++- .../taskana/model/mappings/TaskMapper.java | 2 +- .../impl/ClassificationServiceImplTest.java | 1 - .../taskana/impl/configuration/DBCleaner.java | 2 -- .../rest/ClassificationController.java | 4 +-- .../java/pro/taskana/rest/TaskController.java | 2 +- .../taskana/rest/WorkbasketController.java | 1 - .../sql/sample-data/classification.sql | 4 +-- workplace/src/app/app-routing.module.ts | 10 +++---- workplace/src/app/model/task.ts | 12 ++++----- workplace/src/app/model/workbasket.ts | 3 +-- workplace/src/app/services/data.service.ts | 3 +-- .../app/services/rest-connector.service.ts | 18 +++++++------ workplace/src/app/task/task.component.html | 8 +++--- workplace/src/app/task/task.component.ts | 18 +++++++------ .../taskdetails/taskdetails.component.html | 2 +- .../app/taskdetails/taskdetails.component.ts | 3 +-- .../src/app/tasklist/tasklist.component.html | 14 +++++----- .../src/app/tasklist/tasklist.component.ts | 6 ++--- workplace/src/app/tasks/tasks.component.html | 6 ++--- workplace/src/app/tasks/tasks.component.ts | 4 +-- workplace/src/app/util/orderTasksBy.pipe.ts | 19 +++++++------ .../workbasket-selector.component.ts | 8 +++--- 27 files changed, 108 insertions(+), 85 deletions(-) diff --git a/lib/taskana-cdi/src/test/java/pro/taskana/TaskanaProducersTest.java b/lib/taskana-cdi/src/test/java/pro/taskana/TaskanaProducersTest.java index 926d447be..c98a39b8b 100644 --- a/lib/taskana-cdi/src/test/java/pro/taskana/TaskanaProducersTest.java +++ b/lib/taskana-cdi/src/test/java/pro/taskana/TaskanaProducersTest.java @@ -66,7 +66,6 @@ public class TaskanaProducersTest { } Assert.assertEquals(1, resultCount); - } @@ -86,7 +85,5 @@ public class TaskanaProducersTest { } Assert.assertEquals(0, resultCount); - } - } diff --git a/lib/taskana-core/src/main/java/pro/taskana/impl/ClassificationServiceImpl.java b/lib/taskana-core/src/main/java/pro/taskana/impl/ClassificationServiceImpl.java index ef1d55a1c..da9631cb1 100644 --- a/lib/taskana-core/src/main/java/pro/taskana/impl/ClassificationServiceImpl.java +++ b/lib/taskana-core/src/main/java/pro/taskana/impl/ClassificationServiceImpl.java @@ -21,9 +21,7 @@ import java.util.List; public class ClassificationServiceImpl implements ClassificationService { private static final String ID_PREFIX_CLASSIFICATION = "CLI"; - public static final Date CURRENT_CLASSIFICATIONS_VALID_UNTIL = Date.valueOf("9999-12-31"); - private ClassificationMapper classificationMapper; private TaskanaEngine taskanaEngine; @@ -50,8 +48,6 @@ public class ClassificationServiceImpl implements ClassificationService { return classifications; } - - @Override public void addClassification(Classification classification) { classification.setId(IdGenerator.generateWithPrefix(ID_PREFIX_CLASSIFICATION)); @@ -111,7 +107,6 @@ public class ClassificationServiceImpl implements ClassificationService { } } - @Override public List getAllClassificationsWithId(String id, String domain) { return classificationMapper.getAllClassificationsWithId(id, domain); @@ -132,5 +127,4 @@ public class ClassificationServiceImpl implements ClassificationService { public ClassificationQuery createClassificationQuery() { return new ClassificationQueryImpl(taskanaEngine); } - } diff --git a/lib/taskana-core/src/main/java/pro/taskana/impl/persistence/TaskQueryImpl.java b/lib/taskana-core/src/main/java/pro/taskana/impl/persistence/TaskQueryImpl.java index 2e9257a72..745f0e37e 100644 --- a/lib/taskana-core/src/main/java/pro/taskana/impl/persistence/TaskQueryImpl.java +++ b/lib/taskana-core/src/main/java/pro/taskana/impl/persistence/TaskQueryImpl.java @@ -3,6 +3,7 @@ package pro.taskana.impl.persistence; import java.util.List; import org.apache.ibatis.session.RowBounds; + import pro.taskana.TaskanaEngine; import pro.taskana.exceptions.NotAuthorizedException; import pro.taskana.impl.TaskanaEngineImpl; @@ -112,7 +113,8 @@ public class TaskQueryImpl implements TaskQuery { @Override public List list() throws NotAuthorizedException { checkAuthorization(); - return taskanaEngine.getSession().selectList(LINK_TO_MAPPER, this); + List tempTaskList = taskanaEngine.getSession().selectList(LINK_TO_MAPPER, this); + return tempTaskList; } @Override diff --git a/lib/taskana-core/src/main/java/pro/taskana/model/mappings/ClassificationMapper.java b/lib/taskana-core/src/main/java/pro/taskana/model/mappings/ClassificationMapper.java index cd29af8e6..e31e08c4c 100644 --- a/lib/taskana-core/src/main/java/pro/taskana/model/mappings/ClassificationMapper.java +++ b/lib/taskana-core/src/main/java/pro/taskana/model/mappings/ClassificationMapper.java @@ -39,6 +39,32 @@ public interface ClassificationMapper { @Result(property = "validUntil", column = "VALID_UNTIL")}) Classification findByIdAndDomain(@Param("id") String id, @Param("domain") String domain, @Param("valid_until")Date validUntil); + @Select("SELECT ID, PARENT_CLASSIFICATION_ID, CATEGORY, TYPE, DOMAIN, VALID_IN_DOMAIN, CREATED, NAME, DESCRIPTION, PRIORITY, SERVICE_LEVEL, CUSTOM_1, CUSTOM_2, CUSTOM_3, CUSTOM_4, CUSTOM_5, CUSTOM_6, CUSTOM_7, CUSTOM_8, VALID_FROM, VALID_UNTIL " + + "FROM CLASSIFICATION " + + "WHERE ID = #{id}") + @Results({@Result(property = "id", column = "ID"), + @Result(property = "parentClassificationId", column = "PARENT_CLASSIFICATION_ID"), + @Result(property = "category", column = "CATEGORY"), + @Result(property = "type", column = "TYPE"), + @Result(property = "domain", column = "DOMAIN"), + @Result(property = "isValidInDomain", column = "VALID_IN_DOMAIN"), + @Result(property = "created", column = "CREATED"), + @Result(property = "name", column = "NAME"), + @Result(property = "description", column = "DESCRIPTION"), + @Result(property = "priority", column = "PRIORITY"), + @Result(property = "serviceLevel", column = "SERVICE_LEVEL"), + @Result(property = "custom1", column = "CUSTOM_1"), + @Result(property = "custom2", column = "CUSTOM_2"), + @Result(property = "custom3", column = "CUSTOM_3"), + @Result(property = "custom4", column = "CUSTOM_4"), + @Result(property = "custom5", column = "CUSTOM_5"), + @Result(property = "custom6", column = "CUSTOM_6"), + @Result(property = "custom7", column = "CUSTOM_7"), + @Result(property = "custom8", column = "CUSTOM_8"), + @Result(property = "validFrom", column = "VALID_FROM"), + @Result(property = "validUntil", column = "VALID_UNTIL")}) + Classification findById(@Param("id") String id); + @Insert("INSERT INTO CLASSIFICATION (ID, PARENT_CLASSIFICATION_ID, CATEGORY, TYPE, DOMAIN, VALID_IN_DOMAIN, CREATED, NAME, DESCRIPTION, PRIORITY, SERVICE_LEVEL, CUSTOM_1, CUSTOM_2, CUSTOM_3, CUSTOM_4, CUSTOM_5, CUSTOM_6, CUSTOM_7, CUSTOM_8, VALID_FROM, VALID_UNTIL) VALUES (#{classification.id}, #{classification.parentClassificationId}, #{classification.category}, #{classification.type}, #{classification.domain}, #{classification.isValidInDomain}, #{classification.created}, #{classification.name}, #{classification.description}, #{classification.priority}, #{classification.serviceLevel}, #{classification.custom1}, #{classification.custom2}, #{classification.custom3}, #{classification.custom4}, #{classification.custom5}, #{classification.custom6}, #{classification.custom7}, #{classification.custom8}, #{classification.validFrom}, #{classification.validUntil})") void insert(@Param("classification") Classification classification); @@ -74,6 +100,5 @@ public interface ClassificationMapper { @Result(property = "validFrom", column = "VALID_FROM"), @Result(property = "validUntil", column = "VALID_UNTIL")}) List getAllClassificationsWithId(@Param("id") String id, @Param("domain") String domain); - } diff --git a/lib/taskana-core/src/main/java/pro/taskana/model/mappings/QueryMapper.java b/lib/taskana-core/src/main/java/pro/taskana/model/mappings/QueryMapper.java index 18125e0bc..ea1d22c95 100644 --- a/lib/taskana-core/src/main/java/pro/taskana/model/mappings/QueryMapper.java +++ b/lib/taskana-core/src/main/java/pro/taskana/model/mappings/QueryMapper.java @@ -19,7 +19,8 @@ import java.util.List; public interface QueryMapper { String OBJECTREFERENCEMAPPER_FINDBYID = "pro.taskana.model.mappings.ObjectReferenceMapper.findById"; - String CLASSIFICATION_FINDBYID = "pro.taskana.model.mappings.ClassificationMapper.findByIdAndDomain"; + String CLASSIFICATION_FINDBYIDANDDOMAIN = "pro.taskana.model.mappings.ClassificationMapper.findByIdAndDomain"; + String CLASSIFICATION_FINDBYID = "pro.taskana.model.mappings.ClassificationMapper.findById"; @Select("