Discussion :

[Fichman]

Bonjour,

Je travaille sur une appli Web qui utilise Hibernate pour la gestion de la persistance.

Certains de mes objets contiennent des listes d'éléments que l'utilisateur peut manipuler dans l'interface graphique (ajout, modif, suppression).

Tout marche bien sauf pour la suppression d'un élément d'une liste.
Dans ce cas précis, Hibernate me met bien à jour l'objet principal mais ne supprime pas l'élément qui n'existe plus dans une de ses listes.

Alors j'ai lu quelque part qu'en plus du "cascade='all-delete-orphan'" il fallait mettre à null le père dans l'élément de la collection à supprimer.
Le problème c'est que quand je récupère mon objet du côté serveur, ben cet objet n'existe déjà plus, puisque supprimé du côté serveur.

Pour moi c'est un problème "basique" qui devrait être gérer par Hibernate, mais apparemment c'est pas aussi simple que ça.

Quelqu'un a une solution ?

Voici le mapping de la liste en question :

Code :

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

1
2
3
4
5
 
<bag name="documents" table="DOCUMENTATION" lazy="false" inverse="true" cascade="all-delete-orphan">
			<key column="ID_PERE" foreign-key="ID"/> 
	  		<one-to-many class="Document"/>
		</bag>

Et Voici le mapping de l'élément contenu dans la liste :

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
 
<class name="Document" table="DOCUMENTATION">
		<id name="id" column="ID">
			<generator class="sequence">
      			<param name="sequence">SEQ_DOCUMENTATION</param>
    		</generator>
		</id>
 
		<property name="chemin" column="CHEMIN"/>
		<property name="commentaire" column="COM"/>
		<property name="extension" column="EXT"/>
		<property name="nom" column="FICHIER"/>
 
		<many-to-one name="pere" class="PERE" column="ID_PERE" lazy="false" cascade="all"/>
 
 
	</class>

ps : l'association est bidirectionnelle, c'est peut-être ça qui bloque ?

[DevServlet]

A ma connaissance ne fait pas de suppression en cascade, tu devrais effectuer une boucle sur ta collection et supprimer les elements un à un, et enfin supprimer l'element père, c'est d'ailleurs dit quelque part dans la doc d'hibernate.

[Fichman]

Tout d'abord merci pour ta réponse.

Dans l'absolu, une suppression de l'objet père va entrainer une suppression des fils, j'ai déjà pu faire le test, ça marche.

Par contre moi mon problème c'est dans le cas d'un update du père pour lequel un (ou plusieurs) élément d'une de ses listes a été supprimé par l'utilisateur.

[DevServlet]

J'ai déjà eu à faire à ce cas d'utilisation, pour gérer ca je fais une sauvegarde de l'id du fils selectionné à chaque fois que c fait, à la fin je fais une boucle sur la collection en deletant tous les id que j'avais sauvés. c'est manuel mais ca marche.

[Fichman]

Ouais c'est une solution. En fait je fais déjà un truc comme ça : je prends la collection qui m'est retourné par l'utilisateur et je supprime en base tous les éléments qui ne sont pas dedans.

Le problème c'est que pour le cas qui m'occupe, j'ai environ une dizaine de listes à gérer qu'elles soient rattachées directement à l'objet père ou bien à l'un de ces objets fils

[JeitEmgie]

essayez

Code :

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

1
2
 
 	@Cascade(org.hibernate.annotations.CascadeType.DELETE_ORPHAN)

sur vos @OneToMany

ou regardez l'équivalent en XML (si çà existe…) si vous utilisez des fichiers de configuration et non JPA

[Fichman]

Y a une notation équivalente mais sur le tag <bag> (voir mon premier post).

Malheureusement, ça ne marche pas. ALors je ne sais pas si c'est hibernate qui ne gère pas ça ou bien si c'est ma config qui ne va pas

[Fichman]

Bon je crois que je vais me palucher ça à la main faute de mieux.

Mais c'est quand même étrange que ça ne soit pas géré

[Rei Ichido]

Hibernate gère la chose si tu conserves effectivement ta collection d'origine de l'objet, et donc que tu ne la recrées pas avec la Collection que t'impose l'utilisateur.

Typiquement donc, en sortie de l'IHM, au lieu de créer une Collection et de la coller dans ton objet, il faut prendre fait un get de la Collection qui vient d'hibernate et y effectuer les ajouts / suppressions.

[Fichman]

Merci pour ta participation Rei (c'est pas le Collège fou fou fou ?)

Sinon je ne comprend pas trop ta solution. Je récupère une grappe d'objets que j'envoie à l'utilisateur. Là il fait ses modifications (ajout, suppression) et je récupère ensuite cette grappe modifiée du côté serveur. Et là on fait quoi : on re-récupère les listes via hibernate et on les 'synchronise" avec les listes retournées, ou bien on ne touche pas aux listes dans l'ihm, on ne transmet que le "différentiel" et on fait la synchronisation à la main du côté serveur ?

[tchize_]

Y a une notation équivalente mais sur le tag <bag> (voir mon premier post).

Malheureusement, ça ne marche pas. ALors je ne sais pas si c'est hibernate qui ne gère pas ça ou bien si c'est ma config qui ne va pas

Sur ta notation, tu a mis "inverse=true", ce qui signifie que le bag n'est pas le gestionnaire de l'association. En conséquence, supprimer des éléments du bag n'a aucun effet pour hibernante. C'est uniquement en mettant le pere à null que tu déclenche les opérations. Le mieux dans ton cas est de changer la direcitonnalité qui gère l'association, puisque tu veux que ce soit le bag qui la gère. Met donc le inverse sur le mapping de document et non pas sur le mapping du bag

[Fichman]

J'ai enlevé le inverse=true du bag document, le problème c'est que quand j'enregistre ma liste avec mon objet père, après l'insertion d'un nouveau document, il me fait un update en mettant à null l'id du père dans mon objet Document.... et évidemment ça plante

Y a quelque chose qui a du m'échapper.

[Fichman]

bon j'ai reglé le problème de l'update à null, mais toujours est-il que la suppression d'un élément dans la collection n'est pas prise en compte

[tchize_]

on peux voir ton code?

[Fichman]

Tout d'abord, merci de t'occuper d'une âme perdue

Alors je remets le mapping original :

L'objet père (extrait)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
<class name="Parent" table="PARENT">
 
 
		<bag name="documents" table="DOCUMENTATION" lazy="false" inverse="true" cascade="all">
			<key column="ID_PARENT" foreign-key="ID"/> 
	  		<one-to-many class="Document"/>
		</bag>
 
 
	</class>

L'objet document

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
 
<class name="Document" table="DOCUMENTATION">
		<id name="id" column="ID">
			<generator class="sequence">
      			<param name="sequence">SEQ_DOCUMENTATION</param>
    		</generator>
		</id>
 
		<property name="chemin" column="CHEMIN"/>
		<property name="commentaire" column="COM"/>
		<property name="extension" column="EXT"/>
		<property name="nom" column="FICHIER"/>
 
		<many-to-one name="parent" class="Parent" column="ID_PARENT" lazy="false" cascade="all"/>
 
 
	</class>

Et pour sauvegarder je fais ceci :

Code :

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

1
2
3
4
5
6
7
8
9
 
try {
			HibernateTemplate hbt = this.getHibernateTemplate();
			hbt.saveOrUpdate(parent);
		} catch (DataAccessException e) {
                        logger.error("Erreur lors de la sauvegarde du parent");
			logger.error(e);
			throw new HibernateException(e);
		}

C'est peut-être le saveOrUpdate qui bloque ?

[tchize_]

le inverse est toujours la. LE cascade="all' n'inclue pas le delete-orphan, c'est probablement pour ca que t'as eu un null qui t'as posé problème dans ta db.

[Fichman]

Oui pardon.
Je mets donc ceci :

Code :

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

1
2
3
4
5
 
<bag name="documents" table="DOCUMENTATION" lazy="false" inverse="false" cascade="all-delete-orphan">
			<key column="ID_PARENT" foreign-key="ID"/> 
	  		<one-to-many class="Document"/>
		</bag>

Et à l'enregistrement du père j'ai cette erreur, alors que je ne supprime pas d'élément dans la liste :

Code :

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

1
2
 
impossible de mettre à jour ("FICHMAN"."DOCUMENTATION"."ID_PARENT") avec NULL

Vu l'erreur, je modifie le mapping comme ceci, on rajoutant un not-null à true dans l'id_parent du document :

Code :

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

1
2
3
4
5
 
<bag name="documents" table="DOCUMENTATION" lazy="false" inverse="false" cascade="all-delete-orphan">
			<key column="ID_CRI" foreign-key="ID" not-null="true"/> 
	  		<one-to-many class="Document"/>
		</bag>

Et là Hibernate me dit :

Code :

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

1
2
3
 
 
 org.hibernate.MappingException: Repeated column in mapping for entity: Document column: ID_PARENT (should be mapped with insert="false" update="false")

Je m'exécute donc en modifiant le mapping de document :

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
 
<class name="Document" table="DOCUMENTATION">
		<id name="id" column="ID">
			<generator class="sequence">
      			<param name="sequence">SEQ_DOCUMENTATION</param>
    		</generator>
		</id>
 
		<property name="chemin" column="CHEMIN"/>
		<property name="commentaire" column="COM"/>
		<property name="extension" column="EXT"/>
		<property name="nom" column="FICHIER"/>
 
		<many-to-one name="parent" class="PARENT" column="ID_PARENT" lazy="false" cascade="all" insert="false" update="false"/>
 
 
	</class>

Résultat, ça s'enregistre bien mais lorsque je supprime un document de la collection, et bien il ne se passe toujours rien

[tchize_]

Tes mappings devraient plutot ressembler à ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<class name="Document" table="DOCUMENTATION">
		<id name="id" column="ID">
			<generator class="sequence">
      			<param name="sequence">SEQ_DOCUMENTATION</param>
    		</generator>
		</id>
 
		<property name="chemin" column="CHEMIN"/>
		<property name="commentaire" column="COM"/>
		<property name="extension" column="EXT"/>
		<property name="nom" column="FICHIER"/>
 
		<many-to-one name="parent" class="PARENT" column="ID_PARENT" lazy="false" cascade="all" inverse="true"/>

Code :

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

1
2
3
4
<bag name="documents" table="DOCUMENTATION" lazy="false" inverse="false" cascade="all-delete-orphan">
			<key column="ID_CRI" foreign-key="ID" not-null="true"/> 
	  		<one-to-many class="Document"/>
		</bag>

[Fichman]

L'attribut inverse n'existe pas pour la balise many-to-one

J'ai même enlevé le not-null = true sur la key du bag et enlever la contrainte correspondante en base.

A la suppression d'un élément de la collection j'ai bien l'ID_PARENT qui passe à null mais la ligne n'est toujours pas supprimée malgré le delete-orphan.

Je suis dubitatif

[Rei Ichido]

Merci pour ta participation Rei (c'est pas le Collège fou fou fou ?)

Sinon je ne comprend pas trop ta solution. Je récupère une grappe d'objets que j'envoie à l'utilisateur. Là il fait ses modifications (ajout, suppression) et je récupère ensuite cette grappe modifiée du côté serveur. Et là on fait quoi : on re-récupère les listes via hibernate et on les 'synchronise" avec les listes retournées, ou bien on ne touche pas aux listes dans l'ihm, on ne transmet que le "différentiel" et on fait la synchronisation à la main du côté serveur ?

Tu peux faire des modifications dans l'IHM, mais il faut que ces modifications soient faites en gardant la collection de base, qui est normalement une collection héritant de PersistentCollection qui stocke les modifications faites à la collection. Ton code côté IHM doit donc éviter de recréer une collection. Maintenant il se peut aussi que l'erreur vienne des attributs du mapping, là j'avoue que je n'ai que survolé
(Et oui, c'est bien le nom du chef de la bande des joyeux loufoques).

Edit grammatical ^^