morciuch    2002/08/21 08:55:42

  Modified:    src/java/org/apache/jetspeed/util/template JetspeedTool.java
  Log:
  Check style + javadoc updates - no functional changes made
  
  Revision  Changes    Path
  1.22      +32 -21    
jakarta-jetspeed/src/java/org/apache/jetspeed/util/template/JetspeedTool.java
  
  Index: JetspeedTool.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/util/template/JetspeedTool.java,v
  retrieving revision 1.21
  retrieving revision 1.22
  diff -u -r1.21 -r1.22
  --- JetspeedTool.java 20 Aug 2002 17:38:31 -0000      1.21
  +++ JetspeedTool.java 21 Aug 2002 15:55:42 -0000      1.22
  @@ -63,7 +63,6 @@
   import org.apache.jetspeed.portal.PortletSet;
   import org.apache.jetspeed.portal.PortletControl;
   import org.apache.jetspeed.services.PortalToolkit;
  -import org.apache.jetspeed.services.PsmlManager;
   import org.apache.jetspeed.services.PortletFactory;
   import org.apache.jetspeed.services.resources.JetspeedResources;
   import org.apache.jetspeed.services.Profiler;
  @@ -563,12 +562,13 @@
        * <P>
        * Issues to resolve:
        * <UL>
  -     * <LI> is new portlet instance created everytime someone previews the same 
portlet?
  -     * <LI>should use the same skin as the current pane
  +     * <LI>is new portlet instance created everytime someone previews the same 
portlet?</LI>
  +     * <LI>should use the same skin as the current pane</LI>
        * <LI>if TitlePortletControl is used, the action icons (max, min, etc) are not 
functional.
  -     * Also, customize icon should not be present.
  +     * Also, customize icon should not be present.</LI>
        * <LI> interactive portlets (such as DatabaseBrowser) lose functionality (such 
as sorting
  -     * in DatabaseBrowser).
  +     * in DatabaseBrowser).</LI>
  +     * </UL>
        * 
        * @param portletName
        *               Name of the portlet as defined in registry
  @@ -582,22 +582,23 @@
           ConcreteElement result = null;
           Portlet p = null;
           
  -        try {
  +        try 
  +        {
               // Always set portlet id to "preview" so each preview request gets it 
from the cache.
               // At least, I think that's how it works.
               p = PortletFactory.getPortlet(portletName, "preview");
  -            PortletControl control = controlName == null ? 
PortalToolkit.getControl((String)null) :
  -                                                           
PortalToolkit.getControl(controlName);
  +            PortletControl control = controlName == null ? 
PortalToolkit.getControl((String) null) 
  +                                                         : 
PortalToolkit.getControl(controlName);
               if (control != null)
               {
  -                JetspeedRunData jdata = (JetspeedRunData)rundata;
  -                // Use the profile's skin
  -                
p.getPortletConfig().setSkin(PortalToolkit.getSkin(jdata.getProfile().getDocument().getPortlets().getSkin()));
  +                JetspeedRunData jdata = (JetspeedRunData) rundata;
  +                // Use the profile's skin - deprecated
  +                // 
p.getPortletConfig().setSkin(PortalToolkit.getSkin(jdata.getProfile().getDocument().getPortlets().getSkin()));
                   control.setPortlet(p);
                   control.init();
                   result = control.getContent(rundata);
               } 
  -            else if (p!=null)
  +            else if (p != null)
               {
                   result = p.getContent(rundata);
               }
  @@ -620,23 +621,33 @@
       /**
        * Returns a parameter in its defined parameter style
        * 
  -     * @param portlet
  -     * @param parmName
  -     * @return 
  +     * @param data     for this request
  +     * @param portlet  portlet instance
  +     * @param parmName parameter name
  +     * @return current parameter value using specified presentation style
        */
       public static String getPortletParameter(RunData data, Portlet portlet, String 
parmName)
       {
           String result = null;
  -        try {
  -            if (portlet != null && parmName != null) {
  +        try 
  +        {
  +            if (portlet != null && parmName != null) 
  +            {
                   Map portletParms = portlet.getPortletConfig().getInitParameters();
                   String parmValue = 
portlet.getPortletConfig().getInitParameter(parmName);
                   String parmStyle = 
portlet.getPortletConfig().getInitParameter(parmName + ".style");
  -                if (parmStyle != null) {
  -                    result = ParameterLoader.getInstance().eval(data, parmStyle, 
parmName, parmValue, portletParms);                
  +                if (parmStyle != null) 
  +                {
  +                    result = ParameterLoader.getInstance().eval(data, 
  +                                                                parmStyle, 
  +                                                                parmName, 
  +                                                                parmValue, 
  +                                                                portletParms);      
          
                   }
               }
  -        } catch (Exception e){
  +        } 
  +        catch (Exception e)
  +        {
               Log.error(e);
           }
   
  
  
  

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

Reply via email to