From: Hannu Niemistö Date: Tue, 9 Oct 2018 07:20:50 +0000 (+0300) Subject: Documented difference of RuntimeEnvironmentRequest and Runti...quest2 X-Git-Tag: v1.43.0~136^2~340^2 X-Git-Url: https://gerrit.simantics.org/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F86%2F2286%2F1;p=simantics%2Fplatform.git Documented difference of RuntimeEnvironmentRequest and Runti...quest2 Removed also all cosmetic differences between the classes to make comparison possible. Change-Id: I83e96d09a6e33594a9076df83de887b733d4de46 --- diff --git a/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/util/RuntimeEnvironmentRequest.java b/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/util/RuntimeEnvironmentRequest.java index cde4e0239..9f6b8fe53 100644 --- a/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/util/RuntimeEnvironmentRequest.java +++ b/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/util/RuntimeEnvironmentRequest.java @@ -26,36 +26,36 @@ public class RuntimeEnvironmentRequest extends UnaryRead callback; - - UpdateListenerImpl(EnvironmentSpecification environmentSpecification, Listener callback) { - this.environmentSpecification = environmentSpecification; - this.callback = callback; - } + + final EnvironmentSpecification environmentSpecification; + final Listener callback; + + UpdateListenerImpl(EnvironmentSpecification environmentSpecification, Listener callback) { + this.environmentSpecification = environmentSpecification; + this.callback = callback; + } @Override public void notifyAboutUpdate() { - if(callback.isDisposed()) { - stopListening(); - return; - } - getRuntimeEnvironment(environmentSpecification, callback, this); + if(callback.isDisposed()) { + stopListening(); + return; + } + getRuntimeEnvironment(environmentSpecification, callback, this); } - }; + }; public static void getRuntimeEnvironment(EnvironmentSpecification environmentSpecification, Listener callback, UpdateListenerImpl listener) { try { - + SCLContext context = SCLContext.getCurrent(); - + RuntimeEnvironment env; Object graph = context.get("graph"); if(graph == null) @@ -63,7 +63,7 @@ public class RuntimeEnvironmentRequest extends UnaryRead() { @Override public RuntimeEnvironment perform(ReadGraph graph) throws DatabaseException { - + SCLContext sclContext = SCLContext.getCurrent(); Object oldGraph = sclContext.get("graph"); try { @@ -94,7 +94,7 @@ public class RuntimeEnvironmentRequest extends UnaryRead(mainModuleUri) { - - UpdateListenerImpl sclListener; - - @Override - public void register(ReadGraph graph, Listener procedure) { - - SCLContext context = SCLContext.getCurrent(); - Object oldGraph = context.put("graph", graph); - try { - - if(procedure.isDisposed()) { - getRuntimeEnvironment(environmentSpecification, procedure, null); - } else { - sclListener = new UpdateListenerImpl(environmentSpecification, procedure); - sclListener.notifyAboutUpdate(); - } - - } finally { - context.put("graph", oldGraph); - } - - } - - @Override - public void unregistered() { - if(sclListener != null) - sclListener.stopListening(); + + return graph.syncRequest(new ParametrizedPrimitiveRead(mainModuleUri) { + + UpdateListenerImpl sclListener; + + @Override + public void register(ReadGraph graph, Listener procedure) { + + SCLContext context = SCLContext.getCurrent(); + Object oldGraph = context.put("graph", graph); + try { + + if(procedure.isDisposed()) { + getRuntimeEnvironment(environmentSpecification, procedure, null); + } else { + sclListener = new UpdateListenerImpl(environmentSpecification, procedure); + sclListener.notifyAboutUpdate(); + } + + } finally { + context.put("graph", oldGraph); } - - }); + + } + + @Override + public void unregistered() { + if(sclListener != null) + sclListener.stopListening(); + } + + }); } - + @Override public int hashCode() { return 31*getClass().hashCode() + super.hashCode(); diff --git a/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/util/RuntimeEnvironmentRequest2.java b/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/util/RuntimeEnvironmentRequest2.java index 4bbd48002..ca8bc07cd 100644 --- a/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/util/RuntimeEnvironmentRequest2.java +++ b/bundles/org.simantics.db.layer0/src/org/simantics/db/layer0/util/RuntimeEnvironmentRequest2.java @@ -1,8 +1,6 @@ package org.simantics.db.layer0.util; import java.util.Collection; -import java.util.HashMap; -import java.util.Map; import org.simantics.db.ReadGraph; import org.simantics.db.Resource; @@ -26,13 +24,17 @@ import org.simantics.scl.runtime.SCLContext; * * @author Hannu Niemistö * @author Antti Villberg + * + * Difference between this class and {@code RuntimeEnvironmentRequest} is an additional parameter + * that is typically some component type. All modules under this resource are added to the environment + * in addition to the SCLMain of the root resource. */ public class RuntimeEnvironmentRequest2 extends BinaryRead { - public RuntimeEnvironmentRequest2(Resource parameter, Resource parameter2) { - super(parameter, parameter2); + public RuntimeEnvironmentRequest2(Resource componentType, Resource indexRoot) { + super(componentType, indexRoot); } - + protected void fillEnvironmentSpecification(EnvironmentSpecification environmentSpecification) { } @@ -56,7 +58,7 @@ public class RuntimeEnvironmentRequest2 extends BinaryRead callback, UpdateListenerImpl listener) { + public static void getRuntimeEnvironment(EnvironmentSpecification environmentSpecification, Listener callback, UpdateListenerImpl listener) { try { @@ -120,7 +122,9 @@ public class RuntimeEnvironmentRequest2 extends BinaryRead(environmentSpecification) { + UpdateListenerImpl sclListener; + @Override public void register(ReadGraph graph, Listener procedure) { @@ -155,4 +159,4 @@ public class RuntimeEnvironmentRequest2 extends BinaryRead