Rapha�l Luta wrote:
> 
> "Kevin A. Burton" wrote:
> >
> > ( Within Portlet Markup )
> >
> > So I was thinking that this looks cleaner:
> >
> >         <entry
> > classname="org.apache.jetspeed.portal.portlets.RSSPortlet"
> >                url="http://freshmeat.net/backend/fm.rdf"/>
> >
> > -- instead of:
> >
> >         <entry>
> >             <url>http://freshmeat.net/backend/fm.rdf</url>
> >
> > <classname>org.apache.jetspeed.portal.portlets.RSSPortlet</classname>
> >         </entry>
> >
> > 1.  This fixes a bug in Castor 0.8.0
> > 2.  I think it is nicer.
> >
> > Thoughts?
> >
> 
> -1 for the url attribute, because I can see some cases where multiple URLs may
> be specified in the same portlet (any broker type portlet actually).
> Also it makes the URL even more different than other parameters which IMO
> is not good. RSSPortlet may require one but a lot of possible portlets don't
> care about URLs.

OK.
 
> +0 for the classname attribute since it's necessarily unique and mandatory
> per entry so is elligible for attribute status but it don't see any compelling
> reason for the change.

Classname I think is a good idea I think.
 
> Is the Castor bug resolved by moving only classname as an attribute ?

BTW it is a Castor 0.8.0 bug and I am trying get us migrated to the
latest version.  It is getting fixed :)

-- 
Kevin A Burton ([EMAIL PROTECTED])
http://relativity.yi.org
Message to SUN:  "Please Open Source Java!"
"For evil to win is for good men to do nothing."


--
--------------------------------------------------------------
Please read the FAQ! <http://java.apache.org/faq/>
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Archives and Other:  <http://java.apache.org/main/mail.html>
Problems?:           [EMAIL PROTECTED]

Reply via email to