cvs commit: jakarta-turbine/src/java/org/apache/turbine/services/pull TurbinePullService.java

[jon]

jon         01/06/07 18:21:32

  Modified:    src/java/org/apache/turbine/services/pull
                        TurbinePullService.java
  Log:
  excellent patch by Youngho Cho <[EMAIL PROTECTED]>
  
  I think the PersistentTools has no meaning if user doesn't logged in.
  
  Thanks
  
  youngho
  
  Revision  Changes    Path
  1.23      +6 -3      
jakarta-turbine/src/java/org/apache/turbine/services/pull/TurbinePullService.java
  
  Index: TurbinePullService.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-turbine/src/java/org/apache/turbine/services/pull/TurbinePullService.java,v
  retrieving revision 1.22
  retrieving revision 1.23
  diff -u -r1.22 -r1.23
  --- TurbinePullService.java   2001/05/28 15:22:01     1.22
  +++ TurbinePullService.java   2001/06/08 01:21:31     1.23
  @@ -147,7 +147,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Jason van Zyl</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Sean Legassick</a>
  - * @version $Id: TurbinePullService.java,v 1.22 2001/05/28 15:22:01 jvanzyl Exp $
  + * @version $Id: TurbinePullService.java,v 1.23 2001/06/08 01:21:31 jon Exp $
    */
   public class TurbinePullService extends TurbineBaseService 
       implements PullService
  @@ -419,7 +419,10 @@
           // boolean parameter indicates whether get/setPerm is to be used
           // rather than get/setTemp)
           populateWithSessionTools(sessionTools,    context, data, false);
  -        populateWithSessionTools(persistentTools, context, data, true);
  +        if( data.getUser().hasLoggedIn() )
  +        {
  +            populateWithSessionTools(persistentTools, context, data, true);
  +        }
       }
   
       /**
  @@ -448,7 +451,7 @@
               {
                   Log.error(
                           "Could not instantiate tool " + toolData.toolClassName
  -                        + " to add to the context");
  +                        + " to add to the context" , e );
               }
           }
       }
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]