[EMAIL PROTECTED] writes:

>   User: jmcnally
>   Date: 02/03/13 17:13:15
>
>   Modified:    src/test/org/tigris/scarab/test BaseTestCase.java
>   Added:       src/java/org/tigris/scarab/om AbstractScarabModule.java
>                         Module.java ModuleManager.java
>                         ScarabUserManager.java
>   Removed:     src/java/org/tigris/scarab/om ScarabModuleManager.java
>                         ScarabUserImplManager.java
>                src/java/org/tigris/scarab/services/module
>                         AbstractModuleService.java
>                         AbstractScarabModule.java DefaultModuleService.java
>                         ModuleEntity.java ModuleManager.java
>                         ModuleService.java
>                src/java/org/tigris/scarab/services/user
>                         AbstractUserService.java TurbineUserService.java
>                         UserManager.java UserService.java
>   Log:
>   Using managers to get instances of om's.  Application code should not
>   call FooPeer.retrieveByPk.

How about deprecating the public retrieveByPk() methods in favor of
protected versions?  This does make the assumption that the object
managers are always in the same package, but this is a reasonable
trade-off because it gives pluggable object factories for the OM
classes.

- Dan

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to