You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@isis.apache.org by da...@apache.org on 2015/09/15 08:02:28 UTC
[04/28] isis git commit: ISIS-1194: moving methods from
DomainObjectServices to MessageBrokerService
ISIS-1194: moving methods from DomainObjectServices to MessageBrokerService
Project: http://git-wip-us.apache.org/repos/asf/isis/repo
Commit: http://git-wip-us.apache.org/repos/asf/isis/commit/4d355a3a
Tree: http://git-wip-us.apache.org/repos/asf/isis/tree/4d355a3a
Diff: http://git-wip-us.apache.org/repos/asf/isis/diff/4d355a3a
Branch: refs/heads/ISIS-1194
Commit: 4d355a3ab8d7dd71188e14a55ec6bf75b9929df9
Parents: d830713
Author: Dan Haywood <da...@haywood-associates.co.uk>
Authored: Mon Sep 14 08:58:02 2015 +0100
Committer: Dan Haywood <da...@haywood-associates.co.uk>
Committed: Mon Sep 14 08:58:02 2015 +0100
----------------------------------------------------------------------
.../runtimecontext/DomainObjectServices.java | 28 ----------------
.../runtimecontext/MessageBrokerService.java | 24 ++++++++++++++
.../MessageBrokerServiceAbstract.java | 2 +-
.../MessageBrokerServiceAware.java | 2 +-
.../noruntime/RuntimeContextNoRuntime.java | 35 ++++++++++----------
.../container/DomainObjectContainerDefault.java | 16 ++++++---
.../internal/RuntimeContextFromSession.java | 25 +++++++-------
.../system/persistence/PersistenceSession.java | 2 +-
8 files changed, 70 insertions(+), 64 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/isis/blob/4d355a3a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/DomainObjectServices.java
----------------------------------------------------------------------
diff --git a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/DomainObjectServices.java b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/DomainObjectServices.java
index bef89de..9c307db 100644
--- a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/DomainObjectServices.java
+++ b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/DomainObjectServices.java
@@ -24,34 +24,6 @@ import org.apache.isis.core.commons.config.IsisConfiguration;
public interface DomainObjectServices extends Injectable {
// //////////////////////////////////////////////////////////////////
- // info, warn, error messages
- // //////////////////////////////////////////////////////////////////
-
- /**
- * Provided by <tt>MessageBroker</tt> when used by framework.
- *
- * <p>
- * Called by <tt>DomainObjectContainerDefault</tt>.
- */
- void informUser(String message);
-
- /**
- * Provided by <tt>MessageBroker</tt> when used by framework.
- *
- * <p>
- * Called by <tt>DomainObjectContainerDefault</tt>.
- */
- void warnUser(String message);
-
- /**
- * Provided by <tt>MessageBroker</tt> when used by framework.
- *
- * <p>
- * Called by <tt>DomainObjectContainerDefault</tt>.
- */
- void raiseError(String message);
-
- // //////////////////////////////////////////////////////////////////
// properties
// //////////////////////////////////////////////////////////////////
http://git-wip-us.apache.org/repos/asf/isis/blob/4d355a3a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerService.java
----------------------------------------------------------------------
diff --git a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerService.java b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerService.java
index 5e16540..0f20b95 100644
--- a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerService.java
+++ b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerService.java
@@ -20,5 +20,29 @@ import org.apache.isis.core.commons.components.Injectable;
public interface MessageBrokerService extends Injectable {
+ /**
+ * Provided by <tt>MessageBroker</tt> when used by framework.
+ *
+ * <p>
+ * Called by <tt>DomainObjectContainerDefault</tt>.
+ */
+ void informUser(String message);
+
+ /**
+ * Provided by <tt>MessageBroker</tt> when used by framework.
+ *
+ * <p>
+ * Called by <tt>DomainObjectContainerDefault</tt>.
+ */
+ void warnUser(String message);
+
+ /**
+ * Provided by <tt>MessageBroker</tt> when used by framework.
+ *
+ * <p>
+ * Called by <tt>DomainObjectContainerDefault</tt>.
+ */
+ void raiseError(String message);
+
}
http://git-wip-us.apache.org/repos/asf/isis/blob/4d355a3a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAbstract.java
----------------------------------------------------------------------
diff --git a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAbstract.java b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAbstract.java
index 39d8c3b..c6d4a22 100644
--- a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAbstract.java
+++ b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAbstract.java
@@ -22,7 +22,7 @@ public abstract class MessageBrokerServiceAbstract implements MessageBrokerServi
public void injectInto(final Object candidate) {
if (MessageBrokerServiceAware.class.isAssignableFrom(candidate.getClass())) {
final MessageBrokerServiceAware cast = MessageBrokerServiceAware.class.cast(candidate);
- cast.setQuerySubmitter(this);
+ cast.setMessageBrokerService(this);
}
}
http://git-wip-us.apache.org/repos/asf/isis/blob/4d355a3a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAware.java
----------------------------------------------------------------------
diff --git a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAware.java b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAware.java
index 5b5551e..d1c9db0 100644
--- a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAware.java
+++ b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/MessageBrokerServiceAware.java
@@ -21,6 +21,6 @@ package org.apache.isis.core.metamodel.runtimecontext;
public interface MessageBrokerServiceAware {
- public void setQuerySubmitter(final MessageBrokerService messageBrokerService);
+ public void setMessageBrokerService(final MessageBrokerService messageBrokerService);
}
http://git-wip-us.apache.org/repos/asf/isis/blob/4d355a3a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java
----------------------------------------------------------------------
diff --git a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java
index efa1c74..102f22a 100644
--- a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java
+++ b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java
@@ -61,7 +61,7 @@ public class RuntimeContextNoRuntime extends RuntimeContextAbstract {
private final ObjectPersistorAbstract objectPersistor;
private final DomainObjectServicesAbstract domainObjectServices;
private final LocalizationProviderAbstract localizationProvider;
- private final MessageBrokerServiceAbstract querySubmitter;
+ private final MessageBrokerServiceAbstract messageBrokerService;
public RuntimeContextNoRuntime() {
this(DeploymentCategory.PRODUCTION);
@@ -228,40 +228,41 @@ public class RuntimeContextNoRuntime extends RuntimeContextAbstract {
@Override
- public void informUser(final String message) {
+ public List<String> getPropertyNames() {
throw new UnsupportedOperationException("Not supported by this implementation of RuntimeContext");
}
-
+
@Override
- public void warnUser(final String message) {
+ public String getProperty(final String name) {
throw new UnsupportedOperationException("Not supported by this implementation of RuntimeContext");
}
+ };
+ localizationProvider = new LocalizationProviderAbstract() {
+
+ private final Localization defaultLocalization = new LocalizationDefault();
@Override
- public void raiseError(final String message) {
- throw new UnsupportedOperationException("Not supported by this implementation of RuntimeContext");
+ public Localization getLocalization() {
+ return defaultLocalization;
}
+ };
+ messageBrokerService = new MessageBrokerServiceAbstract() {
@Override
- public List<String> getPropertyNames() {
+ public void informUser(final String message) {
throw new UnsupportedOperationException("Not supported by this implementation of RuntimeContext");
}
@Override
- public String getProperty(final String name) {
+ public void warnUser(final String message) {
throw new UnsupportedOperationException("Not supported by this implementation of RuntimeContext");
}
- };
- localizationProvider = new LocalizationProviderAbstract() {
-
- private final Localization defaultLocalization = new LocalizationDefault();
@Override
- public Localization getLocalization() {
- return defaultLocalization;
+ public void raiseError(final String message) {
+ throw new UnsupportedOperationException("Not supported by this implementation of RuntimeContext");
}
- };
- querySubmitter = new MessageBrokerServiceAbstract() {
+
};
}
@@ -314,7 +315,7 @@ public class RuntimeContextNoRuntime extends RuntimeContextAbstract {
@Override
public MessageBrokerService getMessageBrokerService() {
- return querySubmitter;
+ return messageBrokerService;
}
@Override
http://git-wip-us.apache.org/repos/asf/isis/blob/4d355a3a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java
----------------------------------------------------------------------
diff --git a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java
index e8d8205..926583a 100644
--- a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java
+++ b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java
@@ -65,6 +65,8 @@ import org.apache.isis.core.metamodel.runtimecontext.DomainObjectServices;
import org.apache.isis.core.metamodel.runtimecontext.DomainObjectServicesAware;
import org.apache.isis.core.metamodel.runtimecontext.LocalizationProvider;
import org.apache.isis.core.metamodel.runtimecontext.LocalizationProviderAware;
+import org.apache.isis.core.metamodel.runtimecontext.MessageBrokerService;
+import org.apache.isis.core.metamodel.runtimecontext.MessageBrokerServiceAware;
import org.apache.isis.core.metamodel.runtimecontext.ObjectPersistor;
import org.apache.isis.core.metamodel.runtimecontext.ObjectPersistorAware;
import org.apache.isis.core.metamodel.runtimecontext.ServicesInjector;
@@ -79,8 +81,9 @@ import org.apache.isis.core.metamodel.spec.SpecificationLoaderAware;
public class DomainObjectContainerDefault
implements DomainObjectContainer, DomainObjectServicesAware,
ObjectPersistorAware, SpecificationLoaderAware, AuthenticationSessionProviderAware, AdapterManagerAware,
- ServicesInjectorAware,
+ ServicesInjectorAware, MessageBrokerServiceAware,
LocalizationProviderAware, ExceptionRecognizer {
+ private MessageBrokerService messageBrokerService;
//region > titleOf
@@ -349,6 +352,11 @@ public class DomainObjectContainerDefault
persist(object);
}
+ @Override
+ public void setMessageBrokerService(final MessageBrokerService messageBrokerService) {
+ this.messageBrokerService = messageBrokerService;
+ }
+
//endregion
//region > security
@@ -464,7 +472,7 @@ public class DomainObjectContainerDefault
@Programmatic
@Override
public void informUser(final String message) {
- getDomainObjectServices().informUser(message);
+ messageBrokerService.informUser(message);
}
@Override
@@ -475,7 +483,7 @@ public class DomainObjectContainerDefault
@Programmatic
@Override
public void warnUser(final String message) {
- getDomainObjectServices().warnUser(message);
+ messageBrokerService.warnUser(message);
}
@Override
@@ -486,7 +494,7 @@ public class DomainObjectContainerDefault
@Programmatic
@Override
public void raiseError(final String message) {
- getDomainObjectServices().raiseError(message);
+ messageBrokerService.raiseError(message);
}
@Override
http://git-wip-us.apache.org/repos/asf/isis/blob/4d355a3a/core/runtime/src/main/java/org/apache/isis/core/runtime/persistence/internal/RuntimeContextFromSession.java
----------------------------------------------------------------------
diff --git a/core/runtime/src/main/java/org/apache/isis/core/runtime/persistence/internal/RuntimeContextFromSession.java b/core/runtime/src/main/java/org/apache/isis/core/runtime/persistence/internal/RuntimeContextFromSession.java
index 8ca005f..ddcd30c 100644
--- a/core/runtime/src/main/java/org/apache/isis/core/runtime/persistence/internal/RuntimeContextFromSession.java
+++ b/core/runtime/src/main/java/org/apache/isis/core/runtime/persistence/internal/RuntimeContextFromSession.java
@@ -222,32 +222,33 @@ public class RuntimeContextFromSession extends RuntimeContextAbstract {
this.domainObjectServices = new DomainObjectServicesAbstract() {
@Override
- public void informUser(final String message) {
- getMessageBroker().addMessage(message);
+ public String getProperty(final String name) {
+ return RuntimeContextFromSession.this.getProperty(name);
}
@Override
- public void warnUser(final String message) {
- getMessageBroker().addWarning(message);
+ public List<String> getPropertyNames() {
+ return RuntimeContextFromSession.this.getPropertyNames();
}
+ };
+ this.messageBrokerService = new MessageBrokerServiceAbstract() {
+
@Override
- public void raiseError(final String message) {
- throw new RecoverableException(message);
+ public void informUser(final String message) {
+ getMessageBroker().addMessage(message);
}
@Override
- public String getProperty(final String name) {
- return RuntimeContextFromSession.this.getProperty(name);
+ public void warnUser(final String message) {
+ getMessageBroker().addWarning(message);
}
@Override
- public List<String> getPropertyNames() {
- return RuntimeContextFromSession.this.getPropertyNames();
+ public void raiseError(final String message) {
+ throw new RecoverableException(message);
}
- };
- this.messageBrokerService = new MessageBrokerServiceAbstract() {
};
this.localizationProvider = new LocalizationProviderAbstract() {
http://git-wip-us.apache.org/repos/asf/isis/blob/4d355a3a/core/runtime/src/main/java/org/apache/isis/core/runtime/system/persistence/PersistenceSession.java
----------------------------------------------------------------------
diff --git a/core/runtime/src/main/java/org/apache/isis/core/runtime/system/persistence/PersistenceSession.java b/core/runtime/src/main/java/org/apache/isis/core/runtime/system/persistence/PersistenceSession.java
index 529c4a0..a0d46f0 100644
--- a/core/runtime/src/main/java/org/apache/isis/core/runtime/system/persistence/PersistenceSession.java
+++ b/core/runtime/src/main/java/org/apache/isis/core/runtime/system/persistence/PersistenceSession.java
@@ -367,7 +367,7 @@ public class PersistenceSession implements TransactionalResource, SessionScopedC
}
if (MessageBrokerServiceAware.class.isAssignableFrom(candidate.getClass())) {
final MessageBrokerServiceAware cast = MessageBrokerServiceAware.class.cast(candidate);
- cast.setQuerySubmitter(this);
+ cast.setMessageBrokerService(this);
}
if (ObjectPersistorAware.class.isAssignableFrom(candidate.getClass())) {
final ObjectPersistorAware cast = ObjectPersistorAware.class.cast(candidate);