You are viewing a plain text version of this content. The canonical link for it is here.

Posted to graffito-dev@incubator.apache.org by "Christophe Lombart (JIRA)" <ji...@apache.org> on 2005/09/21 19:43:33 UTC

[jira] Closed: (GRFT-51) Review the PersistenceManager API

     [ http://issues.apache.org/jira/browse/GRFT-51?page=all ]
     
Christophe Lombart closed GRFT-51:
----------------------------------

    Resolution: Fixed

Done

> Review the PersistenceManager API
> ---------------------------------
>
>          Key: GRFT-51
>          URL: http://issues.apache.org/jira/browse/GRFT-51
>      Project: Graffito
>         Type: Improvement
>   Components: JCR-Mapping
>     Reporter: Christophe Lombart
>      Fix For: 1.0-a1-dev

>
> From Oliver : 
> First of all,the Persistence Manager looks fine to me except that I am not sure if it should contain the methods setupRepository(), getRepository(), getSession(), getUser(), setJcrCustomNodeTypeCreator() and getJcrCustomNodeTypeCreator() because they allow direct access to the underlying JCR API and the JCR repository itself. I believe the PersistenceManager should avoid allowing methods that circumvent the use of the graffito object persistence mechanism. These methods would be better off somewhere else.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira