From: Bruno Sinou <bsinou@argeo.org>
Date: Mon, 20 Jan 2014 16:52:33 +0000 (+0000)
Subject: Fix a trivial null pointer exception while working on user management for connect... 
X-Git-Tag: argeo-commons-2.1.30~620
X-Git-Url: http://git.argeo.org/?a=commitdiff_plain;h=527fc5178f8b8fc31832cf05e913a58a66e604f8;p=lgpl%2Fargeo-commons.git

Fix a trivial null pointer exception while working on user management for connect People

git-svn-id: https://svn.argeo.org/commons/trunk@6731 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc
---

diff --git a/security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/RefreshUsersList.java b/security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/RefreshUsersList.java
index 0904ed2fb..e4c14ab21 100644
--- a/security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/RefreshUsersList.java
+++ b/security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/RefreshUsersList.java
@@ -33,6 +33,7 @@ import org.argeo.security.ui.admin.views.UsersView;
 import org.eclipse.core.commands.AbstractHandler;
 import org.eclipse.core.commands.ExecutionEvent;
 import org.eclipse.core.commands.ExecutionException;
+import org.eclipse.ui.IWorkbenchPart;
 import org.eclipse.ui.handlers.HandlerUtil;
 
 /**
@@ -69,12 +70,22 @@ public class RefreshUsersList extends AbstractHandler {
 		} finally {
 			JcrUtils.logoutQuietly(session);
 		}
-
 		userAdminService.synchronize();
+
+		// FIXME try to refresh views that extend the UsersView and have another
+		// ID
+		IWorkbenchPart part = HandlerUtil.getActiveWorkbenchWindow(event)
+				.getActivePage().getActivePart();
+		if (part instanceof UsersView)
+			((UsersView) part).refresh();
+
+		// Try to refresh UsersView if opened
 		UsersView view = (UsersView) HandlerUtil
 				.getActiveWorkbenchWindow(event).getActivePage()
 				.findView(UsersView.ID);
-		view.refresh();
+		if (view != null)
+			view.refresh();
+
 		return null;
 	}