From 1f0deac3ccaa66a8206bccbe0b5cc713ae136f26 Mon Sep 17 00:00:00 2001 From: Steffen Jacobs Date: Thu, 8 Aug 2019 18:37:30 +0200 Subject: [PATCH] Bug 5085: Added TODOs to SharedSailingDataImpl + formatting --- .../com/sap/sailing/server/impl/SharedSailingDataImpl.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/java/com.sap.sailing.server/src/com/sap/sailing/server/impl/SharedSailingDataImpl.java b/java/com.sap.sailing.server/src/com/sap/sailing/server/impl/SharedSailingDataImpl.java index 0b3eaf78c92..47a98ed25d8 100644 --- a/java/com.sap.sailing.server/src/com/sap/sailing/server/impl/SharedSailingDataImpl.java +++ b/java/com.sap.sailing.server/src/com/sap/sailing/server/impl/SharedSailingDataImpl.java @@ -31,7 +31,6 @@ public class SharedSailingDataImpl implements SharedSailingData { private final Map markPropertiesById = new ConcurrentHashMap<>(); private final TypeBasedServiceFinder deviceIdentifierServiceFinder; - public SharedSailingDataImpl(final DomainObjectFactory domainObjectFactory, final MongoObjectFactory mongoObjectFactory, TypeBasedServiceFinder deviceIdentifierServiceFinder) { @@ -98,6 +97,7 @@ public class SharedSailingDataImpl implements SharedSailingData { "Did not find a mark properties with ID %s for setting a fixed position, creating one.", markProperties.getId().toString())); } + // TODO: synchronization markProperties.setFixedPosition(position); markPropertiesById.put(markProperties.getId(), markProperties); mongoObjectFactory.storeMarkProperties(deviceIdentifierServiceFinder, markProperties); @@ -116,6 +116,7 @@ public class SharedSailingDataImpl implements SharedSailingData { "Did not find a mark properties with ID %s for setting a tracking device, creating one.", markProperties.getId().toString())); } + // TODO: synchronization markProperties.setTrackingDeviceIdentifier(deviceIdentifier); markPropertiesById.put(markProperties.getId(), markProperties); mongoObjectFactory.storeMarkProperties(deviceIdentifierServiceFinder, markProperties); @@ -151,6 +152,7 @@ public class SharedSailingDataImpl implements SharedSailingData { @Override public void deleteMarkProperties(MarkProperties markProperties) { + // TODO: synchronization if (this.markPropertiesById.remove(markProperties.getId()) != null) { mongoObjectFactory.removeMarkProperties(markProperties.getId()); } else {