Pull request #296: Adding child elements to a diagram uses the displa…

…y default position Merge in WALTZ/waltz from WALTZ/waltz-dw:CTCTOWALTZ-2831-diagram-elem-sort-6711 to db-feature/waltz-6711-diagram-elem-sort * commit 'f5e626242fbee3ef9b07cd74d52c90e6758d1d15': Adding child elements to a diagram uses the display default position
finos · Aug 21, 2023 · 1f14f34 · 1f14f34
2 parents 21835df + f5e6262
commit 1f14f34
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/waltz-data/src/main/java/org/finos/waltz/data/data_type/DataTypeDao.java b/waltz-data/src/main/java/org/finos/waltz/data/data_type/DataTypeDao.java
@@ -156,6 +156,7 @@ public Collection<DataType> findByParentId(long id) {
                 .selectDistinct(DATA_TYPE.fields())
                 .from(DATA_TYPE)
                 .where(DATA_TYPE.PARENT_ID.eq(id))
+                .orderBy(DATA_TYPE.NAME)
                 .fetchSet(TO_DOMAIN);
     }
 }
diff --git a/waltz-data/src/main/java/org/finos/waltz/data/measurable/MeasurableDao.java b/waltz-data/src/main/java/org/finos/waltz/data/measurable/MeasurableDao.java
@@ -280,6 +280,7 @@ public List<Measurable> findByParentId(Long parentId) {
                 .from(MEASURABLE)
                 .where(MEASURABLE.PARENT_ID.eq(parentId))
                 .and(MEASURABLE.ENTITY_LIFECYCLE_STATUS.eq(EntityLifecycleStatus.ACTIVE.name()))
+                .orderBy(MEASURABLE.POSITION, MEASURABLE.NAME)
                 .fetch(TO_DOMAIN_MAPPER);
     }
 

diff --git a/waltz-data/src/main/java/org/finos/waltz/data/person/PersonDao.java b/waltz-data/src/main/java/org/finos/waltz/data/person/PersonDao.java
@@ -260,7 +260,7 @@ public Set<Person> findActivePeopleByUserRole(String role) {
     }
 
 
-    public Set<Person> findDirectsForPersonIds(List<Long> personIds) {
+    public List<Person> findDirectsForPersonIds(List<Long> personIds) {
         Condition cond = PERSON.MANAGER_EMPLOYEE_ID.in(DSL
                             .select(PERSON.EMPLOYEE_ID)
                             .from(PERSON)
@@ -274,6 +274,6 @@ public Set<Person> findDirectsForPersonIds(List<Long> personIds) {
                 .orderBy(PERSON.DISPLAY_NAME);
 
         return qry
-                .fetchSet(personMapper);
+                .fetch(personMapper);
     }
 }
diff --git a/...lient/entity-diagrams/components/entity-overlay-diagrams/builder/diagram-builder-store.js b/...lient/entity-diagrams/components/entity-overlay-diagrams/builder/diagram-builder-store.js
@@ -8,7 +8,7 @@ export const DiagramModes = {
 }
 
 export function createInitialGroup() {
-    return mkGroup("Diagram Title", generateUUID(), null, 1);
+    return mkGroup("Initial Group", generateUUID(), null, 1);
 }
 
 export let movingGroup = writable(null);

diff --git a/waltz-service/src/main/java/org/finos/waltz/service/person/PersonService.java b/waltz-service/src/main/java/org/finos/waltz/service/person/PersonService.java
@@ -74,9 +74,9 @@ public List<Person> findDirectsByEmployeeId(String employeeId) {
     }
 
 
-    public Set<Person> findDirectsForPersonIds(List<Long> personIds) {
+    public List<Person> findDirectsForPersonIds(List<Long> personIds) {
         if (CollectionUtilities.isEmpty(personIds)) {
-            return Collections.emptySet();
+            return Collections.emptyList();
         }
         return time("PS.findDirectsByPersonIds", () -> personDao.findDirectsForPersonIds(personIds));
     }