Discussion :

[blured]

Bonjour,

Je tente le test suivant :

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
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.

[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.

[blured]

em.persist() retourne void donc la solution que vous proposez n'est pas exploitable

[OButterlin]

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

[blured]

On ExportProfil I've got :

Code :

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

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

on DocumentType I've got :

Code :

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

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

...

[OButterlin]

Remplace

Code :

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

em.merge(ep3);

par

Code :

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

em.persist(ep3);

[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
>>

[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.

[blured]

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

Quelqu'un pourrait confirmer ?

[OButterlin]

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 :

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
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();

[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.

[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.