Discussion :

[nanourene]

Bonjour,

J'ai un souci lors de la persistance d'une entité avec JPA et Hibernate.
Mon entité Client contient une jointure @OneToMany vers une liste de Contact. Au moment de persister la nouvelle entité Client qui contient des nouveaux Contact, j'obtiens cette erreur :

org.postgresql.util.PSQLException: ERREUR: une valeur NULL viole la contrainte NOT NULL de la colonne « client_id »

Mes tables :

Code :

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

1
2
3
4
5
6
7
8
CREATE TABLE CLIENT (
	id SERIAL,
	ref VARCHAR(8) UNIQUE NOT NULL,
	lastname VARCHAR(50) NOT NULL,
	firstname VARCHAR(50),
        ...
	PRIMARY KEY (id)
);

Code :

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

1
2
3
4
5
6
7
8
9
CREATE TABLE CONTACT (
	id SERIAL,
	client_id INTEGER NOT NULL,
	lastname VARCHAR(50) NOT NULL,
	firstname VARCHAR(50),
        ...
	PRIMARY KEY (id),
	FOREIGN KEY (client_id) REFERENCES CLIENT (id)
);

Mes entités :

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
@Entity
@Table(name = "client", uniqueConstraints = @UniqueConstraint(columnNames = "ref") )
public class Client {
	private Integer id;
	private String ref;
	private String lastname;
	private String firstname;
        private Set<Contact> contacts = new HashSet<Contact>(0);
        ...
 
        @Id
	@SequenceGenerator(name = "client_id_seq", sequenceName = "client_id_seq", allocationSize = 1)
	@GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "client_id_seq")
	@Column(name = "id", unique = true, nullable = false, updatable = false)
	public Integer getId() {
		return this.id;
	}
 
	public void setId(Integer id) {
		this.id = id;
	}
 
        ...
 
	@OneToMany(fetch = FetchType.LAZY, mappedBy = "client", cascade = {CascadeType.ALL})
	public Set<Contact> getContacts() {
		return this.contacts;
	}
 
	public void setContacts(Set<Contact> contacts) {
		this.contacts = contacts;
	}
}

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
@Entity
@Table(name = "contact")
public class Contact {
 
	private Integer id;
	private Client client;
	private String lastname;
	private String firstname;
        ...
 
        @Id
	@SequenceGenerator(name = "contact_id_seq", sequenceName = "contact_id_seq", allocationSize = 1)
	@GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "contact_id_seq")
	@Column(name = "id", unique = true, nullable = false, updatable = false)
	public Integer getId() {
		return this.id;
	}
 
	public void setId(Integer id) {
		this.id = id;
	}
 
	@ManyToOne(fetch = FetchType.LAZY, cascade = {CascadeType.ALL})
	@JoinColumn(name = "client_id", nullable = false, updatable = false, insertable = true)
	public Client getClient() {
		return this.client;
	}
 
	public void setClient(Client client) {
		this.client = client;
	}
 
        ...
}

Sauriez-vous comment faire pour que le nouvel id de Client soit définit dans les entités Contact ?

Merci pour votre aide.

[jeffray03]

salut,
peux-tu nous montrer le code de persistence de l´objet en question?

Eric

[nanourene]

Je fais un persist classique :

Code :

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

1
2
3
4
5
6
7
8
9
10
public void persist(final E transientInstance) {
	LOGGER.debug("persisting " + this.persistentClassName + " instance");
	try {
		this.entityManager.persist(transientInstance);
		LOGGER.debug("persist " + this.persistentClassName + " successful");
	} catch (RuntimeException re) {
		LOGGER.error("persist " + this.persistentClassName + " failed", re);
		throw re;
	}
}

[jeffray03]

salut
,
comment est ce que tu crees cet objet:

Code :

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

transientInstance

Eric

[nanourene]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Client client = new Client();
client.setRef("myRef");
client.setLastname("myLastname");
 
Contact contact = new Contact();
contact.setLastname("myLastname");
 
Set<Contact> contactList = new HashSet<Contact>();
contactList.add(contact);
client.setContacts(contactList);
 
this.dao.persist(client);

Si je ne définis pas de contact, mon entité est correctement insérée en bdd.

[JeitEmgie]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Client client = new Client();
client.setRef("myRef");
client.setLastname("myLastname");
 
Contact contact = new Contact();
contact.setLastname("myLastname");
 
Set<Contact> contactList = new HashSet<Contact>();
contactList.add(contact);
client.setContacts(contactList);
 
this.dao.persist(client);

Si je ne définis pas de contact, mon entité est correctement insérée en bdd.

Code :

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

1
2
 
contact.setClient(client);

ou mieux

Code :

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

1
2
 
client.addContact(contact);

qui veillera à ce que tout soit fait correctement.

[nanourene]

Ok merci, ça fonctionne bien en définissant le client sur le contact. Mais je trouve quand même cela étonnant de devoir définir le client sur le contact, alors que le client porte déjà le contact. Pourquoi avoir besoin de faire le lien dans les 2 sens ? Le framework n'est pas suffisamment "intelligent" ? Ou y a-t-il un moyen de le définir par les annotations ? D'autant plus qu'il est possible de définir du coup un client, d'ajouter les contacts et de définir d'autres clients sur ces contacts, ce qui est totalement incohérent... Quoi qu'il en soit, merci beaucoup !

[tchize_]

Tu dois le définir des deux côtés car tu as créé une association bidirectionnelle. Dans une relation bidirectionnelle, jpa considère un seul sens comme étant celui définissant les données a stocker. Ici le sens contact client. L'autre est juste ignoré à l'écriture. Jpa est là pour mapper ton graphe d'objets entre memoire et db. Il n'est pas là pour assurer la cohérence. Ça c'est ton boulot. D'ailleurs quand il y a inchorence comme dans ton cas il a fait son taf et donné priorité à une des valeurs. Dommage pas celle que tu voulais.

[nanourene]

Ok, merci beaucoup !