From 527fc5178f8b8fc31832cf05e913a58a66e604f8 Mon Sep 17 00:00:00 2001 From: Bruno Sinou Date: Mon, 20 Jan 2014 16:52:33 +0000 Subject: [PATCH] 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 --- .../ui/admin/commands/RefreshUsersList.java | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) 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; } -- 2.30.2