incubator-graffito-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Christophe Lombart <christophe.lomb...@gmail.com>
Subject Re: VOTE : PersistenceManager & Mapper interface API
Date Tue, 06 Sep 2005 13:31:11 GMT
2005/9/6, Martin Koci <martin.koci@aura.cz>:
> 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.

Ooops - correct thanks ! Hum, you seem to be very motivate today :-)
Are you agree to send me a patch for this kind of fixs :-) 

Thanks !
Christophe

> 
> 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:  martin.koci@aura.cz
> internet: http://www.aura.cz
>          http://www.j2ee.cz
> ---------------------------------
> 
>
Mime
View raw message