From d01cf8af884e44c3ddc6482a9dca4ac73bb8e93a Mon Sep 17 00:00:00 2001
From: Bruno Sinou <bsinou@argeo.org>
Date: Fri, 1 Jul 2016 20:01:55 +0000
Subject: [PATCH] Reduce log amount when getting an invalid thread exception
 during user management

git-svn-id: https://svn.argeo.org/commons/trunk@9025 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc
---
 .../providers/UserTransactionProvider.java       | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/providers/UserTransactionProvider.java b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/providers/UserTransactionProvider.java
index 43c71ddff..cf1dd5dac 100644
--- a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/providers/UserTransactionProvider.java
+++ b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/providers/UserTransactionProvider.java
@@ -6,13 +6,19 @@ import java.util.Map;
 import javax.transaction.Status;
 import javax.transaction.UserTransaction;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.argeo.ArgeoException;
 import org.argeo.security.ui.admin.SecurityAdminPlugin;
+import org.eclipse.swt.SWTException;
 import org.eclipse.ui.AbstractSourceProvider;
 import org.eclipse.ui.ISources;
 
 /** Observe and notify UI on UserTransaction state changes */
 public class UserTransactionProvider extends AbstractSourceProvider {
+	private final static Log log = LogFactory
+			.getLog(UserTransactionProvider.class);
+
 	public final static String TRANSACTION_STATE = SecurityAdminPlugin.PLUGIN_ID
 			+ ".userTransactionState";
 	public final static String STATUS_ACTIVE = "status.active";
@@ -53,8 +59,14 @@ public class UserTransactionProvider extends AbstractSourceProvider {
 
 	/** Publishes the ability to notify a state change */
 	public void fireTransactionStateChange() {
-		fireSourceChanged(ISources.WORKBENCH, TRANSACTION_STATE,
-				getInternalCurrentState());
+		try {
+			fireSourceChanged(ISources.WORKBENCH, TRANSACTION_STATE,
+					getInternalCurrentState());
+		} catch (SWTException e) {
+			// FIXME
+			log.warn("Cannot fire transaction state change event: "
+					+ e.getMessage());
+		}
 	}
 
 	/* DEPENDENCY INJECTION */
-- 
2.30.2