Suppression dans collection

Version imprimable

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:

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:

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 ?

17/09/2009, 13h30
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.
17/09/2009, 13h40
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.
17/09/2009, 15h13
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.
17/09/2009, 16h05
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:aie:
17/09/2009, 18h53
JeitEmgie
essayez
Code:

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
17/09/2009, 20h05
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
18/09/2009, 15h52
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é
18/09/2009, 17h34
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.
18/09/2009, 19h05
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 ?
19/09/2009, 00h00
tchize_

Citation:

Envoyé par 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

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 ;)
20/09/2009, 14h44
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.
20/09/2009, 15h22
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
20/09/2009, 15h42
tchize_

on peux voir ton code?

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

Alors je remets le mapping original :

L'objet père (extrait)

Code:

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:

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:

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 ?

20/09/2009, 18h56
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.

Oui pardon.
Je mets donc ceci :

Code:

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:

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:

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:

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:

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 :cry:

Tes mappings devraient plutot ressembler à ceci:

Code:

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:

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>

20/09/2009, 20h32
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 :?
21/09/2009, 13h56
Rei Ichido

Citation:

Envoyé par 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 ?

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 ^^
21/09/2009, 14h12
Fichman

Citation:

qui est normalement une collection héritant de PersistentCollection

euh....non, je ne sais même pas ce que c'est...
Concrètement ça se traduit par quoi ?
21/09/2009, 14h21
Rei Ichido

Dans ton code, par rien :)
Concrètement, tu utiliseras probablement la collection sous la forme d'un Set, ou d'une List. Tu peux toujours regarder l'API pour voir le détail de ce qu'il y a dedans, mais ce n'est pas du tout crucial ; la seule chose dont il faut se souvenir, c'est garder l'objet collection renvoyé par hibernate pour qu'il garde trace des ajouts (ça de toute façon il le gère bien même si tu crées une nouvelle collection) mais surtout des suppressions.

Si tu ne contrôles pas ce qui se passe côté IHM, tu peux toujours faire le différentiel à la main en rechargeant l'objet, voire gérer le cas par une procédure stockée en PL/SQL à laquelle tu enverrais la nouvelle collection.
21/09/2009, 18h21
Fichman

Mais même si je garde du côté serveur la collection telle qu'elle m'a été renvoyée par Hibernate, il faudra quand même que je fasse moi-même la suppression à la main dans cette collection en fonction de ce qui m'est renvoyé par l'utilisateur puis que j'affecte cette collection à mon objet père je suppose... c'est quand même pas terrible :aie: D'autant plus que j'en ai une pelletée de collections dont certaines se trouvent elles-mêmes dans des collections !

Pareil, le fameux delete-orphan... ben il marche pas non plus même si la clé du parent est nulle... Je suis maudit, où alors j'ai rien compris :mrgreen:
22/09/2009, 15h23
henpower

est ce qu'on peut également voir le code de suppression ?
23/09/2009, 16h54
Rei Ichido

Citation:

Envoyé par Fichman

Mais même si je garde du côté serveur la collection telle qu'elle m'a été renvoyée par Hibernate, il faudra quand même que je fasse moi-même la suppression à la main dans cette collection en fonction de ce qui m'est renvoyé par l'utilisateur puis que j'affecte cette collection à mon objet père je suppose... c'est quand même pas terrible :aie: D'autant plus que j'en ai une pelletée de collections dont certaines se trouvent elles-mêmes dans des collections !

Non non, simplement dans ton objet, mettons qu'il y a un set, disons mySet : dans ton objet tu laisses le getter/setter sur mySet, mais tu ne te sers jamais du setter (tu laisses Hibernate le faire).
Ensuite quand on utilisateur veut supprimer un objet, tu implémentes deleteFromMySet, et c'est cette fonction que tu appelles chaque fois qu'il veut faire un delete. Idem, une fonction pour le addToMySet (qui gère le cas du set null, là bien sur faut en créer un, sans souci puisqu'il n'y aura pas eu de suppression).
Maintenant, si dans ton IHM tu récupères forcément un nouveau Set, tu peux faire une fonction qui fait dans l'ordre mySet.clear() puis mySet.addAll(myIHMSet).

Voiloute :)
24/09/2009, 08h58
Fichman

Ah ok.

Le problème est que dans mon cas, le setter sur ma collection est obligatoirement utilisé parce que mon interface graphique est en Flex donc mes objets sont settés à chaque renvoi du côté serveur (d'où certainement une partie de mes problèmes). Peut-être qu'Hibernate considère que ce sont des collections différentes donc il ne supprime rien.

Donc au final je pense que je suis bloqué, et ben tant pis je vais faire ça à la main.
24/09/2009, 10h27
Rei Ichido
Tu peux ruser : tu te construit une collection intermédiaire destinée à l'IHM, et ensuite pour alimenter la collection de ton objet, tu fais ce que j'écrivais plus haut :
Code:

1 2 3 4 5 public void fromIHMToDomainMySet() { this.mySet.clear(); this.mySet.addAll(this.myIHMSet); }
(Note : je te laisse le soin de gérer les cas de nullité :mrgreen: )
24/09/2009, 22h21
Fichman

Je vais effectivement devoir faire une "synchronisation" à la main entre l'ancien objet remonté d'Hibernate et le nouveau récupéré de l'IHM.
J'ai testé une suppression d'élément(s) côté serveur, sans passer par l'IHM, ça entraine bien une suppression en base. ça confirme bien que Hibernate perd tout lien avec l'objet une fois passé à l'IHM (ceci étant dû à la technologie utilisée, Flex, car il me semble pas avoir eu ce problème avec Struts).

Je me suis aussi rendu compte, qu'une requête HQL ne prenait pas en compte tous les aspects du mapping comme la suppression des objets many-to-one(unique à true) ayant un cascade="all" lors de la suppression du "père"...
Je ne sais pas si c'est un bug ou si c'est normal.
26/09/2009, 21h33
Fichman

Bon synchronisation effectuée... c'est très chiant mais ça marche.

Je vais aller me renseigner sur le forum Flex pour voir s'il y en a qui ont une meilleure méthode parce que là c'est moche (je trouve)