[
https://issues.apache.org/jira/browse/PLUTO-537?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ate Douma reopened PLUTO-537:
-----------------------------
Reopening this issue for a small adjustment to move base implementations of the
PortletContext and PortletConfig interfaces back from the o.a.p.c.driver.impl
back to o.a.p.c.impl
These implementations can be made almost 100% generic, except for the
PortletConfig.getResourceBundle(Locale) method which is tricky and really
depends on how the portal needs to handle that.
The few Pluto Portal Driver specifics I'll derive from the current
implementations and move them to extended DriverPortletContextImpl and
DriverPortletConfigImpl classes in o.a.p.c.driver.impl.
I'm already running the TCK again against these changes and expect no surprises
so will commit these as soon as the test passes.
> Cleaning up Pluto packages
> --------------------------
>
> Key: PLUTO-537
> URL: https://issues.apache.org/jira/browse/PLUTO-537
> Project: Pluto
> Issue Type: Task
> Components: portlet container
> Affects Versions: 2.0.0
> Reporter: Ate Douma
> Assignee: Ate Douma
> Fix For: 2.0.0
>
>
> A cleaning up of the current Pluto packages was proposed initially by Casten
> on the dev list, see:
> http://www.nabble.com/-RT--Cleaning-up-our-packages-p22288945.html
> A follow up proposal was accepted on the dev list, see:
> http://www.nabble.com/Re%3A--RT--Cleaning-up-our-packages-p22414927.html
> defining the following changes which will be done:
> container-api:
> - o.a.p
> leave as is
> - o.a.p.core
> rename to o.a.p.driver
> move NullPortlet as inner class to PortletServlet
> - o.a.p.services
> move PortletAppDescriptorServices to o.a.p.spi
> move ContainerServices to o.a.p
> move PlutoServices to o.a.p.driver (side-by-side PortletServlet)
> - o.a.p.om.portlet
> leave as is
> - o.a.p.internal
> rename Internal* to Container* (I really don't like the "Internal"
> prefix)
> move * to o.a.p
> - o.a.p.spi.optional
> move * to o.a.p.spi
> the distinction between spi and spi.optional (while somewhat useful
> configuration wise) really is too artificial technical wise imo.
> container (impl):
> - o.a.p
> move PortletContainerFactory to o.a.p.driver.impl
> - o.a.p.core
> move the following to new package o.a.p.impl
> ContainerInvocation* (note: I've some ideas to completely get rid of
> these all together)
> PlutoContainerServices
> PortletContainerImpl
> move all the others to either
> - o.a.p.driver.impl (those which really are used only by/for a
> standalone/pure embedded driver)
> - o.a.p.spi.impl (those which are of more generic usage and/or
> embedding in a larger portal)
> drop package o.a.p.core
> - o.a.p.descriptors.portlet, o.a.p.descriptors.portlet10
> rename these to o.a.p.om.portlet.impl and o.a.p.om.portlet10.impl
> - o.a.p.descriptors.services.jaxb
> move PortletAppDescriptorServiceImpl to o.a.p.spi.impl
> - o.a.p.internal.impl
> move * to o.a.p.impl
> - o.a.p.util
> move to o.a.p.impl.util
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.