Skip to content

Commit b69cf7e

Browse files
committed
Changed class and method names for clarity
1 parent cb463f2 commit b69cf7e

12 files changed

Lines changed: 447 additions & 421 deletions

File tree

FROST-Server.SQLjooq/src/main/java/de/fraunhofer/iosb/ilt/frostserver/persistence/pgjooq/ExpressionHelper.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ private void handleEntityProperty(PathState state, Path path, EntityPropertyMain
216216
if (state.parentPropFields == null) {
217217
state.parentPropFields = state.pathTableRef.getTable()
218218
.getPropertyFieldRegistry()
219-
.getSelectFieldsForProperty(element);
219+
.getPropertyFieldsForProperty(element);
220220
} else {
221221
state.parentPropFields = state.parentPropFields.getSubField(element);
222222
}
@@ -231,11 +231,11 @@ private void handleEntityProperty(PathState state, Path path, EntityPropertyMain
231231
}
232232
pathExpressions = state.pathTableRef.getTable()
233233
.getPropertyFieldRegistry()
234-
.resolveFieldsForProperty(state.parentPropFields, new LinkedHashMap<>());
234+
.resolveAllFieldsForProperty(state.parentPropFields, new LinkedHashMap<>());
235235
} else {
236236
pathExpressions = state.pathTableRef.getTable()
237237
.getPropertyFieldRegistry()
238-
.resolveFieldsForProperty(element, new LinkedHashMap<>());
238+
.resolveAllFieldsForProperty(element, new LinkedHashMap<>());
239239
}
240240
if (pathExpressions.size() == 1) {
241241
final Field field = pathExpressions.values().stream().iterator().next();

FROST-Server.SQLjooq/src/main/java/de/fraunhofer/iosb/ilt/frostserver/persistence/pgjooq/JooqAbsPersistenceManager.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -809,7 +809,7 @@ private void validateMappings() {
809809

810810
private void validateMappings(final StaMainTable<?> tableForType, PropertyFieldRegistry<?> pfReg, ContainerType<?> type) {
811811
for (var property : type.getEntityProperties()) {
812-
var pf = pfReg.getSelectFieldsForProperty(property);
812+
var pf = pfReg.getPropertyFieldsForProperty(property);
813813
validateMapping(pf, "", tableForType);
814814
}
815815
}

FROST-Server.SQLjooq/src/main/java/de/fraunhofer/iosb/ilt/frostserver/persistence/pgjooq/tables/StaTableAbstract.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -305,7 +305,7 @@ public Entity insertIntoDatabase(JooqPersistenceManager pm, Entity entity, EditF
305305
if (entity.isSetProperty(np)) {
306306
Entity ne = entity.getProperty(np);
307307
entityFactories.entityExistsOrCreate(pm, ne, updateMode);
308-
PropertyFields<T> registry = pfReg.getSelectFieldsForProperty(np);
308+
PropertyFields<T> registry = pfReg.getPropertyFieldsForProperty(np);
309309
registry.convert(thisTable, entity, insertFields);
310310
}
311311
}
@@ -329,15 +329,15 @@ public Entity insertIntoDatabase(JooqPersistenceManager pm, Entity entity, EditF
329329
continue;
330330
}
331331
if (entity.isSetProperty(ep)) {
332-
pfReg.getSelectFieldsForProperty(ep).convert(thisTable, entity, insertFields);
332+
pfReg.getPropertyFieldsForProperty(ep).convert(thisTable, entity, insertFields);
333333
}
334334
}
335335

336336
// Sixth, do the actual insert.
337337
// This returns the entire inserted row. It may or may not be changed by databse rules or before-triggers.
338338
DSLContext dslContext = pm.getDslContext();
339-
final Set<PropertyFields<T>> propertyFields = getPropertyFieldRegistry().getSelectFields(new HashSet<>());
340-
final Set<Field> selectFields = QueryState.propertiesToFields(thisTable, propertyFields);
339+
final Set<PropertyFields<T>> propertyFields = getPropertyFieldRegistry().getPropertyFields(new HashSet<>());
340+
final Set<Field> selectFields = QueryState.propertiesToSelectFields(thisTable, propertyFields);
341341
Record result = pm.timeFetchAny(
342342
dslContext.insertInto(thisTable)
343343
.set(insertFields)
@@ -448,7 +448,7 @@ public EntityChangedMessage updateInDatabase(JooqPersistenceManager pm, Entity e
448448
if (!entityFactories.entityExists(pm, ne, PrincipalExtended.getLocalPrincipal().isAdmin())) {
449449
throw new NoSuchEntityException("Linked " + ne.getType() + " not found.");
450450
}
451-
PropertyFields<T> registry = pfReg.getSelectFieldsForProperty(np);
451+
PropertyFields<T> registry = pfReg.getPropertyFieldsForProperty(np);
452452
registry.convert(thisTable, entity, updateFields, message);
453453
}
454454
}
@@ -461,7 +461,7 @@ public EntityChangedMessage updateInDatabase(JooqPersistenceManager pm, Entity e
461461
continue;
462462
}
463463
if (entity.isSetProperty(ep)) {
464-
pfReg.getSelectFieldsForProperty(ep).convert(thisTable, entity, updateFields, message);
464+
pfReg.getPropertyFieldsForProperty(ep).convert(thisTable, entity, updateFields, message);
465465
}
466466
}
467467

@@ -473,8 +473,8 @@ public EntityChangedMessage updateInDatabase(JooqPersistenceManager pm, Entity e
473473
}
474474

475475
DSLContext dslContext = pm.getDslContext();
476-
final Set<PropertyFields<T>> propertyFields = getPropertyFieldRegistry().getSelectFields(new HashSet<>());
477-
final Set<Field> selectFields = QueryState.propertiesToFields(thisTable, propertyFields);
476+
final Set<PropertyFields<T>> propertyFields = getPropertyFieldRegistry().getPropertyFields(new HashSet<>());
477+
final Set<Field> selectFields = QueryState.propertiesToSelectFields(thisTable, propertyFields);
478478
Record result = null;
479479
if (!updateFields.isEmpty()) {
480480
result = executeUpdate(pm, dslContext, thisTable, updateFields, where, selectFields);

0 commit comments

Comments
 (0)