Remove sur un set non pris en compte par l'entity manager

Version imprimable

1 pièce(s) jointe(s)

Bonjour,

Je tente le test suivant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
 
EntityManagerFactory emf = Persistence.createEntityManagerFactory("jpa");
EntityManager em = emf.createEntityManager();
//début transaction
EntityTransaction tx = em.getTransaction();
tx.begin();
 
ExportProfil ep3 = new ExportProfil();
ep3.setCreationDate(new Date());
ep3.setExportName("EP2");
ep3.setLanguage("FR");
 
ExportProcessType ept3 = new ExportProcessType();
ept3.setLanguage("fr");
ept3.setProcessTypeDN("ProcessTypeDN-EP2");
ept3.setProcessType("ProcessType-EP2");
ep3.addProcessType(ept3);
 
ExportDocumentType edt3 = new ExportDocumentType();
edt3.setLanguage("fr");
edt3.setDocumentType("DocumentType-EP2");
edt3.setDocumentTypeDn("DocumentTypeDN-EP2");
ep3.addDocumentType(edt3);
 
ExportDocumentField edf3 = new ExportDocumentField();
edf3.setLanguage("fr");
edf3.setDocumentFieldDN("DocFieldDN1ForDocumentType-EP2");
edf3.setDocumentField("DocField1ForDocumentType-EP2");
edt3.addDocumentField(edf3);
edf3 = new ExportDocumentField();
edf3.setLanguage("fr");
edf3.setDocumentFieldDN("DocFieldDN2ForDocumentType-EP2");
edf3.setDocumentField("DocField2ForDocumentType-EP2");
edt3.addDocumentField(edf3);
 
em.persist(ep3);
//fin transaction
tx.commit();
//fin EntityManager
em.close();
//fin EntityMangerFactory
 
 
//emf = Persistence.createEntityManagerFactory("jpa");
em = emf.createEntityManager();
//début transaction
tx = em.getTransaction();
tx.begin();
 
Set<ExportDocumentType> exportDocumentTypes3 = ep3.getDocumentTypes();
 
for (ExportDocumentType documentType : exportDocumentTypes3) {    
    int i = 1;
    if (i==1) {
        LOG.info("DOCUMENT TYPE WHICH SHOULD BE REMOVED : " + documentType);
        ep3.remove(documentType);    
        i++;
    }
    /*
    for (ExportDocumentField documentField : documentType.getDocumentFields()) {
        documentField.setDocumentFieldDN(documentField.getDocumentFieldDN() + "pouet");
    }
    */
}
 
em.merge(ep3);
tx.commit();
//fin EntityManager
em.close();
//fin EntityMangerFactory
emf.close();

L'instance de documentType que je remove du set derrière le ExportProfil est bien supprimée au niveau JVM. Par contre le em.merge(ep3) devrait le supprimer de la base ce qui n'est pas le cas.

Ca doit être tout con mais je trouve pas...

J'ai attaché les sources des objets du modèle au cas ou.

A+,
Blured.

29/01/2010, 16h55
salber

A mon avis l’objet dont la référence est ep3 n’est pas managé (pas d’identifiant) au moment où vous faites un merge la dessus (à vérifier).
Pour vérifier cela, je vous propose de mettre ep3 = em.persist(ep3); à la ligne où vous faites le persist.
01/02/2010, 13h18
blured

em.persist() retourne void donc la solution que vous proposez n'est pas exploitable :(
01/02/2010, 14h16
OButterlin

Regarde du côté des options de cascade sur ton Set, je pense que tu as oublié de les préciser...

On ExportProfil I've got :

Code:

1
2
3
4
5
 
	@OneToMany(mappedBy="profil", cascade={CascadeType.ALL})
	public Set<ExportDocumentType> getDocumentTypes() {
		return documentTypes;
	}

on DocumentType I've got :

Code:

1
2
3
4
5
6
 
    @ManyToOne(cascade=CascadeType.ALL)
    @JoinColumn(name="profil_id")
	public ExportProfil getProfil() {
		return profil;
	}

...

02/02/2010, 09h10
OButterlin

Remplace

Code:

em.merge(ep3);

par

Code:

em.persist(ep3);
02/02/2010, 09h44
blured

Ca ne fonctionne pas car ep3 est déjà persisté puis détaché (la session est explicitement fermée)

Donc si j'essai de repersistert j'obtiens l'exception suivante :
<<
org.hibernate.PersistentObjectException: detached entity passed to persist: com.clog.workey.repository.wizard.data.model.ExportProfil
>>

:(
02/02/2010, 09h58
OButterlin

Je suis un peu rouillé en JPA, je voulais l'équivalent de update() en Hibernate "natif"...
Le problème à mon sens vient du merge tardif.
Comme ton objet n'est pas attaché, il faut commencer par l'attacher, supprimer tes références, et ensuite commiter.
02/02/2010, 10h06
blured

Ouai c'est ce que j'ai constaté. C'est un peu chiant mais bon à priori faut faire avec...

Quelqu'un pourrait confirmer ?

Ton problème vient du fait que tu utilises mal l'api.
Ton premier bloc (insertion) est correct, le deuxième non.

Il ne faut pas perdre de vue que dans ton cas, l'objet ep3 n'est plus attaché, il faut donc commencer par récupérer une instance attachée avant toute modification.
Ce serait donc
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 //emf = Persistence.createEntityManagerFactory("jpa"); em = emf.createEntityManager(); //début transaction em.getTransaction().begin(); ep3 = em.find(ExportProfil.class, id); <-- il faudra le récupérer après insertion à la fin du bloc 1 // ici, tu travailles sur un objet "attaché" Set<ExportDocumentType> exportDocumentTypes3 = ep3.getDocumentTypes(); for (ExportDocumentType documentType : exportDocumentTypes3) { int i = 1; if (i==1) { LOG.info("DOCUMENT TYPE WHICH SHOULD BE REMOVED : " + documentType); ep3.remove(documentType); i++; } } em.getTransaction().commit();

02/02/2010, 10h37
blured

Donc avec JPA/hibernate il est impossible de modifier un graphe d'objet détaché (ajout / suppression d'éléments dans les différents set). Il faut absolument être connecté et faire les modifications à ce moment là pour que les changements soient effectués en base.

Ce qui est bizzarre c'est qu'il est permit de modifier les champs du graphe (pas les collections) en étant déconnecté puis en faisant un merge de l'objet. Peut être une prochaine version d'hibernate prévoirait celà ?

A+,
Blured.
02/02/2010, 10h43
OButterlin

Le problème ne vient pas d'Hibernate qui lui peut le faire mais bien de JPA qui a prévu un mécanisme un peu différent.