Hi All,
when I used the IdentitySession API, the typical template would be like:
try
{
IdentitySession.beginTransaction();
do operations;
identitySession.getTransaction().commit();
} catch (Exception e) {
identitySession.getTransaction.rollback();
} finally {
identitySession.close();
}
This is without considering catching the exceptions on rollback() and close() method.
Therefore, I was coming up with a Template class, named it JBossIdentityTemplate, the class is like:
public abstract class JBossIdentityTemplate<T> {
private static Logger logger = Logger.getLogger(JBossIdentityTemplate.class.toString());
private IdentitySession session;
public JBossIdentityTemplate(IdentitySession session) {
this.session = session;
}
public T execute() throws IdentityException {
T o = null;
try {
session.beginTransaction();
o = doExecute(session);
session.getTransaction().commit();
} catch (Exception e) {
session.getTransaction().rollback();
} finally {
closeSession();
}
return o;
}
private void closeSession() {
try {
session.close();
} catch (IdentityException e) {
logger.log(Level.SEVERE, "Error in closing the session");
}
}
public abstract T doExecute(IdentitySession session) throws Exception;
}
With this class, the implementation class would be like:
public Group createGroup(final Group group) throws Exception{
return new JBossIdentityTemplate<Group>(identitySession) {
@Override
public Group doExecute(IdentitySession session) throws Exception {
org.jboss.identity.idm.api.Group theGroup = identitySession.getPersistenceManager().createGroup(group.getName(), "GROUP");
group.setId(theGroup.getId());
AttributesManager attrManager = identitySession.getAttributesManager();
attrManager.addAttribute(theGroup, GROUP_DESCRIPTION, group.getDescription());
return group;
}
}.execute();
}
What do you think of this idea? Comments and feedback are welcome.
