Discussion :

[Ho(c)ine.]

Bonjour,
J'ai un objet mappé sur une table avec une collection attachée.
(Collection contenant des objets eux meme avec des collections d'autres objets).

Mon projet : Spring + Hibernate 3 (objet : HibernateDaoSupport).

Quand j'essaie de mettre à jour mon objet avec le saveOrUpdate() de l'objet HibernateTemplate j'ai l'erreur suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
Illegal attempt to associate a collection with two open sessions

En faisant des recherches sur l'ami google j'ai bien compris qu'il y avait un pb de session ? mais je n'ai pas trouvé la solution au pb ???
A noter que si je ne rappatrie pas les données de la collection (lazy) la mise à jour fonctionne correctement.

Merci d'avance pour une piste....

[OButterlin]

Tu n'aurais pas oublié de fermer la session qui a été utilisée pour la lecture ?

[Ho(c)ine.]

Tout d'abord merci pour ta réponse ultra rapide !

J'ai suivi ton conseil et rajouter dans ma classe DAO la fermeture de la session

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
 
// this = HibernateDaoSupport
 
Session s = this.getSession();
s.close();
s = null;

Malheureusement cela ne fonctionne toujours pas ?
Je me demande si ma session est bien fermée ?
Est ce la bonne méthode pour fermer une session... ? Je fais une recherche...

[OButterlin]

Il n'y a pas de problème sur la méthode, c'est bien session.close()...

Peux-tu montrer le fichier hibernate.cfg.xml, les fichiers de mapping concernés par ton problème et le code des DAO ?

A+

[Ho(c)ine.]

Le fichier de mapping :

Fichier de mapping de mon objet (test), ici l'implémentation de test : TestImpl

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
<class name="objet.process.TestImpl" table="test" >
 
	<id name="id" type="int">
		<column name="id"  />
		<generator class="native"/>
	</id>		
	<property name="nom" type="string">
		<column name="nom" length="100" not-null="true" />
	</property>
 
	...
	<bag name="questions" table="compose" lazy="true"   order-by="numeroQuestion" cascade="all">
		<key column="idTest" />
		<many-to-many class="objet.process.QuestionImpl" column="numeroQuestion" />
	</bag>
 
</class>

la query qui charge mon objet...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
	<sql-query name="testNouveau">
		<return alias="test" class="objet.process.TestImpl"/>
		<![CDATA[
			Select test.*
			From test 
			Inner join compose On test.id = compose.idTest
			Inner join question On compose.numeroQuestion = question.numero
			Where codeSousCategorie = :codeSG
			order by nbrePassage desc
		]]>
	</sql-query>

La DAO abstraite

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public abstract class AbsctractDao extends HibernateDaoSupport {
 
  .....
 
  public void modifier(IObjetHcd bean) throws DAOException{
 
        try {	
            this.getHibernateTemplate().saveOrUpdate(bean);
 
        } catch (DataAccessException e) {
 
        	throw new DAOException(e.getMessage());
        } 
    }
 
    ....
 
}

Extrait de La DAO test qui étend la DAO abstraite

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
Query query = this.getSession().getNamedQuery(DaoTestImpl.QUERY_TEST_NOUVEAU);
			query.setInteger("codeSG", codeSousCategorie);
            listeTests = query.list();
            test = (ITest) listeTests.get(0);
 
            Session s = this.getSession();
 
            s.flush();
            s.close();
            s = null;
            this.getSessionFactory().close();

En gros juste aprés le chargement de l'objet, je close la session... !

PAR AVANCE, MERCI de TON AIDE.

[OButterlin]

Je pense que ton problème vient du fait que tu ouvres 2 sessions et que tu ne fermes pas celle du query.
En gros, il faudrait faire ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
Session s = this.getSession();

Query query = s.getNamedQuery(DaoTestImpl.QUERY_TEST_NOUVEAU);
			query.setInteger("codeSG", codeSousCategorie);
listeTests = query.list();
test = (ITest) listeTests.get(0);
s.close();

[Ho(c)ine.]

Non j'ai toujours l'erreur.
Je suis ok avec toi c'est un pb de double session.
Mais je crois que c'est au moment de la sauvegarde que ca ne va pas....
Bon, mille fois merci pour ton aide.....

[OButterlin]

Est ce que tu fais un traitement (qui rattacherait l'objet à une session) sur les éléments de la liste avant de passer dans ta méthode "modifier" ?

[Ho(c)ine.]

Et bé, je fais une lecture pour charger l'objet et puis je le mets tout de suite à jour en appelant le service "modifier".
En fait je mets à jour un indicateur (un champ) de la table (+1 dans le champ).

[OButterlin]

Ton problème vient très certainement de là.
Quand tu fais ta lecture, tu attaches l'objet à une session et quand ensuite tu exécutes la méthode "modifier", les objets liés restent attachés à la session si elle n'a pas été fermée...

Par contre, pourquoi faire une lecture ?
Soit tu mets lazy="false", soit après la lecture de l'objet parent, tu fais un appel au getter lié à la liste (pour charger)

[Ho(c)ine.]

OK.
Mais le chargement differé de ma collection posé un pb -> ???
Je suis obligé de charger complétement l'objet.
D'ou ce pb.....

[OButterlin]

Sinon, essaye de fermer la session qui a été utilisée pour lire avant de passer le Pojo à ta méthode "modifier", ça devrait passer aussi...