[jboss-svn-commits] JBL Code SVN: r5331 - in labs/jbossforums/trunk: authz-jaae-plugin/src/main/org/jboss/forums/security forums/src/resources/portal-forums-war/WEB-INF

[jboss-svn-commits at lists.jboss.org]

Author: sohil.shah at jboss.com
Date: 2006-07-27 21:41:26 -0400 (Thu, 27 Jul 2006)
New Revision: 5331

Modified:
   labs/jbossforums/trunk/authz-jaae-plugin/src/main/org/jboss/forums/security/AuthorizationProvider.java
   labs/jbossforums/trunk/forums/src/resources/portal-forums-war/WEB-INF/forums-config.xml
Log:
couple of bug fixes (security related issue, and transaction related issue)

Modified: labs/jbossforums/trunk/authz-jaae-plugin/src/main/org/jboss/forums/security/AuthorizationProvider.java
===================================================================
--- labs/jbossforums/trunk/authz-jaae-plugin/src/main/org/jboss/forums/security/AuthorizationProvider.java	2006-07-27 23:35:02 UTC (rev 5330)
+++ labs/jbossforums/trunk/authz-jaae-plugin/src/main/org/jboss/forums/security/AuthorizationProvider.java	2006-07-28 01:41:26 UTC (rev 5331)
@@ -64,6 +64,11 @@
                 JBossPortalUtil.populateRoles(user);
                 UserThreadLocal.setUser(user);
             }
+            else
+            {
+                //set an anonymous user
+                UserThreadLocal.setUser(new User());
+            }
 	        
 	        if(context instanceof JSFActionContext)
 	        {

Modified: labs/jbossforums/trunk/forums/src/resources/portal-forums-war/WEB-INF/forums-config.xml
===================================================================
--- labs/jbossforums/trunk/forums/src/resources/portal-forums-war/WEB-INF/forums-config.xml	2006-07-27 23:35:02 UTC (rev 5330)
+++ labs/jbossforums/trunk/forums/src/resources/portal-forums-war/WEB-INF/forums-config.xml	2006-07-28 01:41:26 UTC (rev 5331)
@@ -27,11 +27,14 @@
     </application> 
     
     <!-- phase listeners -->
-    <!-- remove this in portal-environment -->
-    <lifecycle>    	
+    <!-- 
+    	remove this in portal-environment. This is needed only for managing a transaction in standalone mode.
+    	in portal mode, the portal takes care of this
+    -->
+    <!--lifecycle>    	
     	<phase-listener>org.jboss.portlet.forums.ui.event.BeginTransactionListener</phase-listener>    	
     	<phase-listener>org.jboss.portlet.forums.ui.event.EndTransactionListener</phase-listener>    	
-    </lifecycle>
+    </lifecycle-->
         
     	
 	<!-- configuration for the shared EmptyController -->