J'ai besoin de cette modif, j'ai cependant remarquer quelque chose que je ne comprends pas en essayant, pour tester,  d'implémenter les commitTransaction() et rollbackTransaction() (sur TopiaDAOAbstract ou TopiaDAOHibernate ce qui ne change rien) par :

@Override
    public void commitTransaction() throws TopiaException {
        context.commitTransaction();
    }

Dans TopiaContextImpl ou se situe l'implémentation des commitTransaction et rollbackTransaction au niveau du context, se trouve ce bout de code dans chacune des deux méthodes (en remplacant la ligne qui va bien) :

for (TopiaDAO<? extends TopiaEntity> dao : daoCache.values()) {
                dao.commitTransaction();
 }

Donc forcement ca boucle !! Je me demandais l'intérêt de ce "for" étant donné que les commitTransaction et rollbackTransaction sur les dao sont vide !!

Y a t'il un intérêt particulier dans un certain cas ? Il me suffirait de supprimer la boucle for pour ne plus avoir de problème (notamment avec les tests).

Qu'en pensez-vous ?

Flo

Le 25 mai 2009 14:29, Eric Chatellier <chatellier@codelutin.com> a écrit :
Benjamin POUSSIN a écrit :
> J'avais commence a l'ecrire pour etre sur que le code que je
> produissait de l'autre cote (hibernate) ne soit pas trop specifique
> hibernate et donc que l'on puisse plus tard utiliser d'autres moyens de
> persistence. Mais est-ce vraiment utile ? ne gagnerions nous pas en
> simplicite a faire sauter ce flatfile qui n'est de toute facon pas
> fonctionnelle et pas maintenu ?
>
Il m'avais aussi posé problème lors du passage hibernate 3.2 -> 3.3.
J'avais fixé les erreurs de compilation, mais sans tester.

--
Éric <chatellier@codelutin.com>
Tel: 02 40 50 29 28
http://www.codelutin.com

_______________________________________________
Topia-devel mailing list
Topia-devel@lists.labs.libre-entreprise.org
http://lists.labs.libre-entreprise.org/mailman/listinfo/topia-devel