roller-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ganesh Mathrubootham (JIRA)" <>
Subject [jira] Commented: (ROL-1750) PlanetURLStrategy needs to be moved out of ui package
Date Tue, 30 Sep 2008 16:31:19 GMT


Ganesh Mathrubootham commented on ROL-1750:

And, I have tested this patch in my local machine.  Sources compile fine and the functionality
looks good too.

> PlanetURLStrategy needs to be moved out of ui package

> -------------------------------------------------------
>                 Key: ROL-1750
>                 URL:
>             Project: Roller
>          Issue Type: Improvement
>          Components: Build Process
>            Reporter: Ganesh Mathrubootham
>            Assignee: Roller Unassigned
>            Priority: Trivial
>         Attachments: move_PlanetURLStrategy.patch
> The location of is preventing a clear separation between web tier
and business tier.
> This class is currently packaged under org.apache.roller.weblogger.planet.ui.   Like
all other Strategy classes, this belongs to business layer and so should probably go under
> I discovered this issue when I tried to create separate projects for web and business
layers in my Eclipse IDE.  I followed the inclusion and exclusion patterns defined in the
ant script, removed roller-business.jar and roller-web.jar from classpath to avoid duplication
and made my web project dependent on business project.  At this point, I am getting compilation
error in org/apache/roller/weblogger/planet/business/jpa/ (business
project) as it is referring to, which is in web project because of
the way inclusion/exclusion patterns are defined.  
> This problem can be easily resolved by moving from org.apache.roller.weblogger.planet.ui

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message