X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Finternal%2FUserAdminWrapper.java;fp=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Finternal%2FUserAdminWrapper.java;h=0000000000000000000000000000000000000000;hb=e66b9893b0e511f8ab295e3cee42b7dc966f1597;hp=d443a3d47ce3c1a4b4e3c8109d7cc02334585796;hpb=8260f4470f514ea347ca53f5b4dfc632c4a4de66;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/UserAdminWrapper.java b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/UserAdminWrapper.java
deleted file mode 100644
index d443a3d47..000000000
--- a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/UserAdminWrapper.java
+++ /dev/null
@@ -1,77 +0,0 @@
-package org.argeo.security.ui.admin.internal;
-
-import java.util.ArrayList;
-import java.util.List;
-
-import javax.transaction.Status;
-import javax.transaction.UserTransaction;
-
-import org.argeo.cms.CmsException;
-import org.osgi.service.useradmin.UserAdminEvent;
-import org.osgi.service.useradmin.UserAdminListener;
-
-/** Centralise interaction with the UserAdmin in this bundle */
-public class UserAdminWrapper extends
-		org.argeo.cms.util.useradmin.UserAdminWrapper {
-
-	// First effort to simplify UX while managing users and groups
-	public final static boolean COMMIT_ON_SAVE = true;
-
-	// Registered listeners
-	List<UserAdminListener> listeners = new ArrayList<UserAdminListener>();
-
-	/**
-	 * Starts a transaction if necessary. Should always been called together
-	 * with {@link UserAdminWrapper#commitOrNotifyTransactionStateChange()} once
-	 * the security model changes have been performed.
-	 */
-	public UserTransaction beginTransactionIfNeeded() {
-		try {
-			UserTransaction userTransaction = getUserTransaction();
-			if (userTransaction.getStatus() == Status.STATUS_NO_TRANSACTION) {
-				userTransaction.begin();
-				// UiAdminUtils.notifyTransactionStateChange(userTransaction);
-			}
-			return userTransaction;
-		} catch (Exception e) {
-			throw new CmsException("Unable to begin transaction", e);
-		}
-	}
-
-	/**
-	 * Depending on the current application configuration, it will either commit
-	 * the current transaction or throw a notification that the transaction
-	 * state has changed (In the later case, it must be called from the UI
-	 * thread).
-	 */
-	public void commitOrNotifyTransactionStateChange() {
-		try {
-			UserTransaction userTransaction = getUserTransaction();
-			if (userTransaction.getStatus() == Status.STATUS_NO_TRANSACTION)
-				return;
-
-			if (UserAdminWrapper.COMMIT_ON_SAVE)
-				userTransaction.commit();
-			else
-				UiAdminUtils.notifyTransactionStateChange(userTransaction);
-		} catch (Exception e) {
-			throw new CmsException("Unable to clean transaction", e);
-		}
-	}
-
-	// TODO implement safer mechanism
-	public void addListener(UserAdminListener userAdminListener) {
-		if (!listeners.contains(userAdminListener))
-			listeners.add(userAdminListener);
-	}
-
-	public void removeListener(UserAdminListener userAdminListener) {
-		if (listeners.contains(userAdminListener))
-			listeners.remove(userAdminListener);
-	}
-
-	public void notifyListeners(UserAdminEvent event) {
-		for (UserAdminListener listener : listeners)
-			listener.roleChanged(event);
-	}
-}
\ No newline at end of file