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 2014/12/01 09:09:13 UTC

[2/2] isis git commit: ISIS-960: fixing logging statements interpolation in EventBusServiceDefault's exception handler

ISIS-960: fixing logging statements interpolation in EventBusServiceDefault's exception handler


Project: http://git-wip-us.apache.org/repos/asf/isis/repo
Commit: http://git-wip-us.apache.org/repos/asf/isis/commit/434ab61b
Tree: http://git-wip-us.apache.org/repos/asf/isis/tree/434ab61b
Diff: http://git-wip-us.apache.org/repos/asf/isis/diff/434ab61b

Branch: refs/heads/master
Commit: 434ab61bb4b08af36a4d8409e6ad8a996d2ff2b1
Parents: 333c472
Author: Dan Haywood <da...@haywood-associates.co.uk>
Authored: Mon Dec 1 08:09:03 2014 +0000
Committer: Dan Haywood <da...@haywood-associates.co.uk>
Committed: Mon Dec 1 08:09:03 2014 +0000

----------------------------------------------------------------------
 .../runtime/services/eventbus/EventBusServiceDefault.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/isis/blob/434ab61b/core/runtime/src/main/java/org/apache/isis/core/runtime/services/eventbus/EventBusServiceDefault.java
----------------------------------------------------------------------
diff --git a/core/runtime/src/main/java/org/apache/isis/core/runtime/services/eventbus/EventBusServiceDefault.java b/core/runtime/src/main/java/org/apache/isis/core/runtime/services/eventbus/EventBusServiceDefault.java
index 1b3c826..ff3835b 100644
--- a/core/runtime/src/main/java/org/apache/isis/core/runtime/services/eventbus/EventBusServiceDefault.java
+++ b/core/runtime/src/main/java/org/apache/isis/core/runtime/services/eventbus/EventBusServiceDefault.java
@@ -91,19 +91,19 @@ public class EventBusServiceDefault extends EventBusService {
                 final Phase phase = interactionEvent.getPhase();
                 switch (phase) {
                 case HIDE:
-                    LOG.warn("Exception '%s' (%s) thrown during HIDE phase, to be safe will veto (hide) the interaction event");
+                    LOG.warn("Exception thrown during HIDE phase, to be safe will veto (hide) the interaction event, msg='{}', class='{}'", exception.getMessage(), exception.getClass().getName());
                     interactionEvent.hide();
                     break;
                 case DISABLE:
-                    LOG.warn("Exception '%s' (%s) thrown during DISABLE phase, to be safe will veto (disable) the interaction event");
+                    LOG.warn("Exception thrown during DISABLE phase, to be safe will veto (disable) the interaction event, msg='{}', class='{}'", exception.getMessage(), exception.getClass().getName());
                     interactionEvent.disable(exception.getMessage()!=null?exception.getMessage(): exception.getClass().getName() + " thrown.");
                     break;
                 case VALIDATE:
-                    LOG.warn("Exception '%s' (%s) thrown during VALIDATE phase, to be safe will veto (invalidate) the interaction event");
+                    LOG.warn("Exception thrown during VALIDATE phase, to be safe will veto (invalidate) the interaction event, msg='{}', class='{}'", exception.getMessage(), exception.getClass().getName());
                     interactionEvent.invalidate(exception.getMessage()!=null?exception.getMessage(): exception.getClass().getName() + " thrown.");
                     break;
                 case EXECUTING:
-                    LOG.warn("Exception '%s' (%s) thrown during EXECUTING phase, to be safe will abort the transaction");
+                    LOG.warn("Exception thrown during EXECUTING phase, to be safe will abort the transaction, msg='{}', class='{}'", exception.getMessage(), exception.getClass().getName());
                     abortTransaction(exception);
                     break;
                 case EXECUTED: