]> gerrit.simantics Code Review - simantics/platform.git/blobdiff - bundles/org.simantics.objmap2/src/org/simantics/objmap/structural/annotations/factories/UpdateMethodFactory.java
Thread safety changes in objmap2
[simantics/platform.git] / bundles / org.simantics.objmap2 / src / org / simantics / objmap / structural / annotations / factories / UpdateMethodFactory.java
index a5dc78f9120196e76dca9432ad3514f7d0799085..c0fee3da6e53c27ed6042c23e58ed881c5fa5564 100644 (file)
@@ -14,7 +14,8 @@ package org.simantics.objmap.structural.annotations.factories;
 import java.lang.annotation.Annotation;
 import java.lang.reflect.Method;
 
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.simantics.db.ReadGraph;
 import org.simantics.db.WriteGraph;
 import org.simantics.db.exception.DatabaseException;
@@ -26,7 +27,7 @@ import org.simantics.objmap.graph.rules.factory.IMethodRuleFactory;
 
 public class UpdateMethodFactory<Domain, Range> implements IMethodRuleFactory<Domain, Range> {
 
-    static Logger LOGGER = Logger.getLogger("org.simantics.objmap");
+    static Logger LOGGER = LoggerFactory.getLogger(UpdateMethodFactory.class);
     
     @Override
     public IBidirectionalMappingRule<Domain, Range> create(ReadGraph g, 
@@ -40,7 +41,7 @@ public class UpdateMethodFactory<Domain, Range> implements IMethodRuleFactory<Do
             public boolean updateRange(ReadGraph g, IForwardMapping<Domain, Range> map,
                     Domain domainElement, Range rangeElement)
                     throws MappingException {
-                LOGGER.info("    UpdateMethodFactory.updateRange");      
+                LOGGER.trace("    UpdateMethodFactory.updateRange");      
                 try {
                     return (Boolean)method.invoke(rangeElement, g, domainElement);
                 } catch (Exception e) {
@@ -57,6 +58,12 @@ public class UpdateMethodFactory<Domain, Range> implements IMethodRuleFactory<Do
                 return false;
             }
             
+            @Override
+            public boolean checkChanges(ReadGraph g, IForwardMapping<Domain, Range> map, Domain domainElement,
+                    Range rangeElement) throws MappingException {
+                return false;
+            }
+            
             public void createDomain(WriteGraph g, IBackwardMapping<Domain,Range> map, Domain domainElement, Range rangeElement) throws MappingException {
                updateDomain(g, map, domainElement, rangeElement);
             };