[ROL-1750] PlanetURLStrategy needs to be moved out of ui package

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 5.0
• Component/s: Website and Documentation
• Level of effort: less than a day to move this class to the right package and change the references (1 class)
• Assignee: David Johnson
• Reporter: Ganesh Mathrerootham
• Created: 30/Sep/08 16:25
• Updated: 31/May/14 14:07
• Resolved: 14/Oct/08 03:45

Description

The location of PlanetURLStrategy.java 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 org.apache.roller.weblogger.planet.business

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/RollerPlanetModule.java (business project) as it is referring to PlanetURLStrategy.java, which is in web project because of the way inclusion/exclusion patterns are defined.

This problem can be easily resolved by moving PlanetURLStrategy.java from org.apache.roller.weblogger.planet.ui to org.apache.roller.weblogger.planet.business