You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by ra...@apache.org on 2001/06/04 20:57:43 UTC

cvs commit: jakarta-jetspeed/src/java/org/apache/jetspeed/services/portaltoolkit JetspeedPortalToolkitService.java

raphael     01/06/04 11:57:43

  Modified:    src/java/org/apache/jetspeed/services/portaltoolkit
                        JetspeedPortalToolkitService.java
  Log:
  [fix] incorrect inheritance of skin because service did not send back a
  null when expected
  
  Revision  Changes    Path
  1.2       +7 -10     jakarta-jetspeed/src/java/org/apache/jetspeed/services/portaltoolkit/JetspeedPortalToolkitService.java
  
  Index: JetspeedPortalToolkitService.java
  ===================================================================
  RCS file: /home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/services/portaltoolkit/JetspeedPortalToolkitService.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- JetspeedPortalToolkitService.java	2001/06/04 17:30:35	1.1
  +++ JetspeedPortalToolkitService.java	2001/06/04 18:57:43	1.2
  @@ -81,7 +81,7 @@
    * Simple implementation of the PortalFactoryService.
    * 
    * @author <a href="mailto:raphael@apache.org">Rapha�l Luta</a>
  - * @version $Id: JetspeedPortalToolkitService.java,v 1.1 2001/06/04 17:30:35 raphael Exp $
  + * @version $Id: JetspeedPortalToolkitService.java,v 1.2 2001/06/04 18:57:43 raphael Exp $
    */
   public class JetspeedPortalToolkitService extends TurbineBaseService
       implements PortalToolkitService
  @@ -332,18 +332,15 @@
           if (skin != null)
           {
               name = skin.getName();
  -        }
  -        else
  -        {
  -            name = defaultSkin;
  -        }
   
  -        // create the PortletSkin corresponding to this entry
  -        result = getSkin( name );
  +            // create the PortletSkin corresponding to this entry
  +            result = getSkin( name );
           
  -        // override the values with the locally defined properties
  -        result.putAll(getParameters(skin));
  +            // override the values with the locally defined properties
  +            result.putAll(getParameters(skin));
           
  +        }
  +
           return result;
       }
   
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org