and all try - catch construct should look like:
catch (Exception e)
       {
throw new JcrMappingException("Impossible to add the node : " + parentNode, e); // WITH ROOT EXCEPTION
       }

not 'throw new JcrMappingException("Impossible to add the node : " + parentNode); '. This masks trivial exceptions like NPE.

Martin

Martin Koci wrote:

Hi,
I think org.apache.portals.graffito.jcr.persistence.objectconverter.ObjectConverter methods should return new inserted Node (with method 'insert') and method 'update' updated javax.jcr.Node. Return values can be used for testing.

Christophe Lombart wrote:

Hi all,
I just commited some code refactoring. Can you vote for the PersistenceManager API & the Mapper API ? We can expect to see some modification on thoses components when we
will add new features.

See on :
* org.apache.portals.graffito.jcr.persistence.PersistenceManager
* org.apache.portals.graffito.jcr.mapper.Mapper.

You can also check the current impletation. Let me know what you want to do. I'm currently reviewing (for the secon time :-) the collection management.

Christophe






--
Mgr. Martin Kočí
---------------------------------
AURA, s.r.o.
Úvoz 499/56; 602 00 Brno
ISO 9001 certifikovaná společnost
tel./fax: +420 5 43 24 51 11
e-mail:  [EMAIL PROTECTED]
internet: http://www.aura.cz
        http://www.j2ee.cz
---------------------------------

Reply via email to