modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/authz/AuthorizationManagerBean.java | 1 - 1 file changed, 1 deletion(-)
New commits: commit b660cd54392e1044f32424394e8e580edcffd55d Author: John Mazzitelli mazz@redhat.com Date: Fri May 28 16:06:53 2010 -0400
this can't be correct. we should not be flushing here - we don't know what happened previously nor do we know if the caller wants us to flush to the db. if we really did need this, the other get*Permissions methods would have/need this too.
diff --git a/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/authz/AuthorizationManagerBean.java b/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/authz/AuthorizationManagerBean.java index a6b3e99..77932d0 100644 --- a/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/authz/AuthorizationManagerBean.java +++ b/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/authz/AuthorizationManagerBean.java @@ -47,7 +47,6 @@ public class AuthorizationManagerBean implements AuthorizationManagerLocal {
@SuppressWarnings("unchecked") public Set<Permission> getExplicitGlobalPermissions(Subject subject) { - entityManager.flush(); Query query = entityManager.createNamedQuery(Subject.QUERY_GET_GLOBAL_PERMISSIONS); query.setParameter("subject", subject); List<Permission> results = query.getResultList();
rhq-commits@lists.fedorahosted.org