Mailing-List: contact jetspeed-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Jetspeed Developers List" <jetspeed-dev@jakarta.apache.org>
Message-ID: <BD961D4D4634D511BB990050DA1CE12CB2884A@winexch02.rippe.com>
From: "Weaver, Scott" <Sweaver@rippe.com>
To: 'Jetspeed Developers List' <jetspeed-dev@jakarta.apache.org>
Subject: RE: Question on Muliple portlet instance behavior
Date: Fri, 28 Jun 2002 15:12:16 -0400
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----_=_NextPart_001_01C21ED7.BA3E6B50"

------_=_NextPart_001_01C21ED7.BA3E6B50
Content-Type: text/plain;
	charset="ISO-8859-1"
Content-Transfer-Encoding: quoted-printable

David,

> 1. change interface name and package to
> org.apache.jetspeed.portal.PortletInstance

No problem.  I can swing that.

> 2. clean up the code a bit

Go ahead, but could you give me a little detail on what you feel is
currently out of whack.  It would mean less work for you in the future =
;) =20

> 3. write the unit tests

As embarrassing as it is, I have to admit I have not had a chance to do
anything with JUnit.  If you write a test case, I will use to learn to =
write
them in the future.

> 4. remove old persistence code
> 5. modify all existing code to use the new service
> 6. check it in

It might be best if you handle these also.

I do have some more changes I'm working on.  I will get it to you =
within the
hour (before 4PM EST).

Scott
=20
> -----Original Message-----
> From: David Sean Taylor [mailto:david@bluesunrise.com]
> Sent: Friday, June 28, 2002 1:44 PM
> To: 'Jetspeed Developers List'
> Subject: RE: Question on Muliple portlet instance behavior
>=20
> Scott,
>=20
> I don't like the naming of PersistentPor(t)let.
> I think what we are really working with are portlet instances. Why =
not
> call it 'PortletInstance'.
> Would also recommend moving PortletInstance interface into
> org.apache.jetspeed.portal package.
>=20
> Then given a portlet, you could get back an instance:
>=20
> PortletInstance pi =3D portlet.getInstance();
> String attribute =3D pi.getAttribute(name);
> etc
>=20
>=20
> Do you have any unit tests for the service?
> If not, I would be glad to write one.
>=20
> So do you want me to take it from here?
> I can do this:
>=20
> 1. change interface name and package to
> org.apache.jetspeed.portal.PortletInstance
> 2. clean up the code a bit
> 3. write the unit tests
> 4. remove old persistence code
> 5. modify all existing code to use the new service
> 6. check it in
>=20
> This is great. This refactoring has been long overdue. Thanks Scott!
>=20
> David
>=20
> -----Original Message-----
> From: Weaver, Scott [mailto:Sweaver@rippe.com]
> Sent: Friday, June 28, 2002 8:45 AM
> To: 'Jetspeed Developers List'
> Subject: RE: Question on Muliple portlet instance behavior
>=20
>=20
> Hi David,
> I re-designed the persistence mechanism to be more object oriented =
and
> easier to use than my first iteration.
> There is now PersistentPortlet interface and a concrete =
implementation,
> JetspeedPersistentPortlet.  The PersistenceService now only has 3
> methods two to retrieve a PersistentPortlet and one to store it.
> All operations are performed through the PersistentPortlet as opposed =
to
> the PersistenceService itself, save for retrieval and storage.  This =
has
> allowed me to simplify the methods used to set, get and remove =
portlet
> attributes.
> Scott
> > -----Original Message-----
> > From: David Sean Taylor [mailto:david@bluesunrise.com]
> > Sent: Thursday, June 27, 2002 6:02 PM
> > To: 'Jetspeed Developers List'
> > Subject: RE: Question on Muliple portlet instance behavior
> >
> > Scott,
> >
> > Had a quick look.
> > Few comments:
> >
> > jData.getProfile();                 -- this will always get you the
> > current profile, good!
> > doc.getEntryById(portlet.getID())   -- great, that will get the
> correct
> > instance!
> >
> > Looking good, I will review it in more detail tomorrow.
> >
> > Thanks,
> >
> > David
> >
> > -----Original Message-----
> > From: Weaver, Scott [mailto:Sweaver@rippe.com]
> > Sent: Thursday, June 27, 2002 2:31 PM
> > To: 'Jetspeed Developers List'
> > Subject: RE: Question on Muliple portlet instance behavior
> >
> >
> > David,
> > Here is what I finished today.  There is a service interface, an
> > implementation and a fa=E7ade class to access the service plus an
> > exception.
> > I have tested the get and set and they are working and are aware of
> > individual portlet instances.  I have yet change AbstractPortlet =
and
> > test it there.  Hopefully, I will get to that tomorrow.  I also =
still
> > need to add the Apache license to the classes and better
> documentation.
> > I will be adding more methods as I get to them/think of them:
> > getPortletAttributeList(),getPortletAttributeNames(), etc.
> > Let me know of another other important functionality I may not have
> > thought of/been aware of.
> > Thanks,
> > Scott
> >
> >
> >
> >
> > --
> > To unsubscribe, e-mail:   <mailto:jetspeed-dev-
> > unsubscribe@jakarta.apache.org>
> > For additional commands, e-mail: <mailto:jetspeed-dev-
> > help@jakarta.apache.org>
>=20
>=20
>=20
>=20
> --
> To unsubscribe, e-mail:   <mailto:jetspeed-dev-
> unsubscribe@jakarta.apache.org>
> For additional commands, e-mail: <mailto:jetspeed-dev-
> help@jakarta.apache.org>

------_=_NextPart_001_01C21ED7.BA3E6B50--