Leon Messerschmidt wrote:
> I added a new service named AssemblerBrokerService to the cvs. I also added
> some settings to the TR.properties file to support AssemblerBrokerService.
Looks good!
I would suggest defining constants in BaseJavaFactory for the strings that
the specific Java factories are using.
I would also suggest leaving only the Service interface, implementation
and facade class (yet to be written) in this package, and moving
the other classes to the turbine.util hierarchy.
turbine.util.assemblerbroker.AssemblerFactory
turbine.util.assemblerbroker.java.BaseFactory
turbine.util.assemblerbroker.java.ActionFactory
turbine.util.assemblerbroker.java.ScreenFactory
turbine.util.assemblerbroker.java.NavigationFactory
and then adding your classes under
turbine.util.assemblerbroker.jpython
We have a long tradition of crazy package partitioning :)
Rafal
--
Rafal Krzewski
Senior Internet Developer
mailto:[EMAIL PROTECTED]
+48 22 8534830 http://e-point.pl
------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?: [EMAIL PROTECTED]
