[jboss-cvs] jboss-seam/src/main/org/jboss/seam/theme ...

[Gavin King]

  User: gavin   
  Date: 07/07/10 04:57:37

  Modified:    src/main/org/jboss/seam/theme  Theme.java
  Log:
  JBSEAM-1606, JBSEAM 1632
  
  Revision  Changes    Path
  1.8       +4 -6      jboss-seam/src/main/org/jboss/seam/theme/Theme.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: Theme.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/theme/Theme.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -b -r1.7 -r1.8
  --- Theme.java	8 Jul 2007 21:13:37 -0000	1.7
  +++ Theme.java	10 Jul 2007 08:57:37 -0000	1.8
  @@ -1,6 +1,6 @@
   package org.jboss.seam.theme;
   
  -import static org.jboss.seam.ScopeType.SESSION;
  +import static org.jboss.seam.ScopeType.EVENT;
   import static org.jboss.seam.annotations.Install.BUILT_IN;
   
   import java.util.AbstractMap;
  @@ -104,13 +104,11 @@
      }
      
      /**
  -    * Create a Map in the session scope. The session scope is used because
  -    * creating the bundles is somewhat expensive, so it can be cached there because
  -    * the session theme changes infrequently. When the theme is changed, ThemeSelector
  -    * is responsible for removing the Map from the session context.
  +    * Create a Map in the event scope. When the theme is changed, ThemeSelector
  +    * is responsible for removing the Map from the event context.
       * 
       */
  -   @Factory(value="org.jboss.seam.theme.theme", autoCreate=true, scope=SESSION)
  +   @Factory(value="org.jboss.seam.theme.theme", autoCreate=true, scope=EVENT)
      public java.util.Map getTheme()
      {
         return createMap();