Discussion :

[chido]

Bonjour à tous,

J'ai comme entity : Book, OrderLine et Order.
Une orderLine pointe sur un book et un order.

Je n'arrive pas à mettre à jour mon orderLine, voici ma fonction avec des commentaires qui vont bien :

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
// Ajouter au panier
	public Order addInOrder(Book book, int orderId) {
 
		//Vérification si une ligne existe déjà avec ce livre
		TypedQuery<OrderLine> queryName = em
				.createQuery(
						"Select cl From OrderLine cl Where order.id = :orderId AND book.isbn = :isbn",
						OrderLine.class);
		queryName.setParameter("orderId", orderId);
		queryName.setParameter("isbn", book.getIsbn());
 
		//List car getSimpleResult plante si il est vide...
		List<OrderLine> orderLine = queryName.getResultList();
 
		//Si il n'y a pas d'orderLine avec ce livre dans notre order
		if (orderLine.size() == 0) {
 
			//On rechercher notre order en base
			TypedQuery<Order> queryName2 = em.createQuery(
					"Select cl From Order cl Where cl.id = :orderId", Order.class);
			queryName2.setParameter("orderId", orderId);
 
			Order Order = queryName2.getSingleResult();
 
			//on créé une nouvelle ligne
			OrderLine orderline = new OrderLine();
			orderline.setBook(book);
			orderline.setQuantity(1);
 
			// On fait en sorte que notre order ai cette ligne ? Automatique ?
// Si je met un persist(orderline) j'ai l'erreur Caused by: javax.validation.UnexpectedTypeException: No validator could be found for type: java.lang.Integer
 
		} else {
 
			//Si on a trouvé une ligne avec ce livre, on rajoute 1 à la quantité de livre
			int quantity = orderLine.get(0).getQuantity();
			quantity++;
			orderLine.get(0).setQuantity(quantity);
 
			// Ne fait apparement rien sauf enlever la police rouge de ma quantité quand je pointe mon objet avec ma sourie
			// comme si il l'avait mis à jour....
			em.persist(orderLine.get(0));
		}
 
		// Récupération de notre order mis à jour... ou pas...
		TypedQuery<Order> queryName2 = em.createQuery(
				"Select cl From Order cl Where cl.id = :orderId", Order.class);
		queryName2.setParameter("orderId", orderId);
 
		//Récupération de l'ancien order, il n'est pas à jour, la ligne non plus.
		Order order2 = queryName2.getSingleResult();
 
                //Obligé de retourner notre order pour mettre notre Order de notre bean à jour ?
		return order2;
 
	}

Pourtant ailleurs dans mon code, ça fonctionne :

Code :

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

1
2
3
4
5
6
// Changer le mot de passe
	public User updateMdp(User user, String mdp) {
		User userM = em.find(User.class, user.getLogin());
		userM.setMotDePasse(mdp);
		return userM;
	}

Je suis en JTA donc pas de commit.

[-Dici-]

J'ai pas lu tout ton code mais a priori la deuxième méthode ne fonctionne pas non plus car tu ne fais que changer la valeur d'un attribut sur un objet Java, ça ne veut aucunement dire que l'entrée associée dans la BDD est mise à jour (tu pourras d'ailleurs l'observer de tes propres yeux en faisant un select de tes password avant et après l'appel à cette méthode).

Pour ce faire, il faut utiliser em.merge(objetAMettreAJour). Attention à ce que cet objet ne pointe vers aucune Entity qui n'ait pas encore été persistée, si c'est le cas il faut d'abord les persister puis éventuellement merger, mais dans ton cas ça ira car tu récupères directement l'objet dans la base et ne lui change qu'un attribut simple.

Cette remarque est aussi valable dans la première méthode : il faut faire appel à merge une fois que l'objet Java a été mis à jour.

[chido]

Si justement ma méthode me modification du mot de passe (la 2eme) fonctionne très bien et modifie ma valeur en base .

Je vais tenter le merge en rentrant chez moi merci

[naf87]

J'ai pas lu tout ton code mais a priori la deuxième méthode ne fonctionne pas non plus car tu ne fais que changer la valeur d'un attribut sur un objet Java,

Tu oublis un détail très important: JPA travail avec un contexte de persistance et si tu modifies une entité qui se trouve dans ce contexte (une entité managée) au moment du commit, les données en bd seront mises à jour. Hors, par défaut, la méthode find ajoute automatiquement l'entité retournée dans ce contexte d'où le résultat.

[naf87]

Code :

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

1
2
//Récupération de l'ancien order, il n'est pas à jour, la ligne non plus.
		Order order2 = queryName2.getSingleResult();

quand est-ce que tu l'as mis à jour dans ton code ?

Je n'arrive pas à mettre à jour mon orderLine

comment sais-tu qu'il n'est pas mis à jour ?

//on créé une nouvelle ligne
OrderLine orderline = new OrderLine();
orderline.setBook(book);
orderline.setQuantity(1);

A quel moment renseignes-tu l'identifiant de l'entité?

Code :

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

// Si je met un persist(orderline) j'ai l'erreur Caused by: javax.validation.UnexpectedTypeException: No validator could be found for type: java.lang.Integer

essaie d'utiliser le type primitif

Code :

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

int

Code :

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

//List car getSimpleResult plante si il est vide...

Il ne plante pas, il lève tout simplement l'exception NoResultException

[chido]

Même avec le merge ça ne fonctionne pas. Voici le process :
J'ai un module EJB avec mes services et un module WEB avec mes bean et facelets.

2 interrogations peuvent m'aider à comprendre :

1 - Quand je crée un nouvel objet(entity) dans mon bean, je présume que celui-ci n'est pas persisté. Donc si on le transporte dans ma couche service, on doit le persister pour qu'il s'enregistre en base. Un simple persist(objet) devrai suffire ?

2 - Si on récupère des objets (entity) via la base (init d'un tableau par exemple), on passe donc par notre couche service qui va récupérer en base puis faire un return de notre (nos) objets vers le bean qui, lui, va donc avoir son objet remplis et il sera affiché dans notre tableau. Cet objet ce trouvant dans notre bean, c'est par exemple :

Code :

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

1
2
3
4
5
 
@postconstruct
private void init(){
    List<Book> books = bookService.getAllBooks();
}

Est-ce que nos objets Book sont encore persistés dans notre bean ? Car si je clique sur une ligne de mon tableau afin de récupérer mon book dans une méthode de mon bean et que cette méthode appel la couche service via par exemple :

Code :

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

1
2
3
4
 
private void UpBook(Book book) {
    bookService.bookUpdate(Book book)
}

on aura dans notre bookService la méthode :

Code :

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

1
2
3
private void bookUpdate(Book book) {
????
}

??? => Que mettre pour enregistrer notre modification de book en base ?
J'ai bien essayé de mettre un em.persist(book); dans cette méthode mais j'ai ce genre de message :

Code :

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

Removing a detached instance com.icl.m3.model.Book#35

[-Dici-]

Tu oublis un détail très important: JPA travail avec un contexte de persistance et si tu modifies une entité qui se trouve dans ce contexte (une entité managée) au moment du commit, les données en bd seront mises à jour. Hors, par défaut, la méthode find ajoute automatiquement l'entité retournée dans ce contexte d'où le résultat.

Ok merci, j'ai pas encore fait beaucoup d'applications utilisant JPA, je sais juste manipuler sans connaître tous les fondements. Juste une question (ça sera la seule pour ne pas polluer le topic) : si on n'appelle pas explicitement merge ou commit dans sa méthode, la modification opérée sur le mot de passe ne prendre pas effet immédiatement, si ? Si c'est le cas, ça pourrait parfois poser problème.