portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Sean Taylor" <da...@bluesunrise.com>
Subject RE: [PATCH] proposed for peid
Date Mon, 25 Mar 2002 15:50:37 GMT
-            // set Portlet #1 for wml
-            // Andreas Kempf, ICM S CP PE, Munich
-            //
---------------------------------------------------------------------
-            if ((editMediaType != null) &&
(editMediaType.equalsIgnoreCase("wml")))
-            {
-              name="1";
-            }
-            else  
-              name="0";
-            //
---------------------------------------------------------------------

Did we remove the WML support here?

Was also thinking that we need this more formalized.
The profiler already looks for this param:

   /MediaType/<mediatype_name>

> "forPortlet()" was introduced recently, in:

What does that mean, forPortlet?
Is it supposed to be like the terminology in the class loader?

Aren't we just getting a portlet given an id, i.e. getPortlet(String
id), which could be coded in vm as

$jlink.Portlet(id)   // im actually not sure if that will work or not,
since there is a param

Note: all the $jlink and other tools in Jetspeed need to be documented
on the website
   

> -----Original Message-----
> From: Glenn Golden [mailto:ggolden@umich.edu] 
> Sent: Monday, March 25, 2002 7:36 AM
> To: 'Jetspeed Developers List'
> Subject: RE: [PATCH] proposed for peid
> 
> 
> "forPortlet()" was introduced recently, in:
> 
> revision 1.6
> date: 2002/03/05 12:26:33;  author: sgala
> 
> I propose we change it, since there has not been enough time 
> for this to become a widely used feature.
> 
> - Glenn
> 
> > -----Original Message-----
> > From: Paul Spencer [mailto:paulsp@apache.org]
> > Sent: Monday, March 25, 2002 8:58 AM
> > To: Jetspeed Developers List
> > Subject: Re: [PATCH] proposed for peid
> > 
> > 
> > Glenn
> > 
> > Glenn Golden wrote:
> > 
> > > David -
> > > 
> > <snip>
> > 
> > > I still want a change in the JetspeedTemplateLink's
> > "forPortlet", so
> > > that it
> > > a) takes a peid not a name, and b) if given blank or null,
> > just ignores the
> > > call.  My changes for this are in the patch as well.
> > 
> > 
> > 
> > I applied and tested the patch locally.  Although it worked as you
> > described, I did not commit it because it changes a parameter 
> > passed to 
> > the existing method forPortlet() from Name to PEID.  Their 
> are sites 
> > using Jetspeed and we need to maintain backward compatibility 
> > whenever 
> > practical.  I have be working with David on the best way to 
> > resolve the 
> > root problem(s).
> > 
> > > 
> > > Can you review and apply these patches?  Thanks!
> > 
> >  >
> >  >
> > 
> > > - Glenn
> > 
> > 
> > 
> > Paul Spencer
> > 
> > 
> > --
> > To unsubscribe, e-mail:   
> > <mailto:jetspeed-dev-> unsubscribe@jakarta.apache.org>
> > For
> > additional commands, 
> > e-mail: <mailto:jetspeed-dev-help@jakarta.apache.org>
> > 
> 
> --
> To unsubscribe, e-mail:   
> <mailto:jetspeed-dev-> unsubscribe@jakarta.apache.org>
> For 
> additional commands, 
> e-mail: <mailto:jetspeed-dev-help@jakarta.apache.org>
> 
> 



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


Mime
View raw message