Discussion :

[smarties]

Bonjour,

J'ai un formulaire basique :

Code html :

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

1
2
3
4
5
<form:form action="account" method="post" class="form-horizontal" modelAttribute="sessionBean.person">
<!-- some form input elements like -->
<label class="col-sm-3 control-label">E-mail</label><div class="col-md-4"><form:input cssClass="form-control" path="email" maxlength="64" /></div>
<!-- submit button -->
</form:form>

Dont le controlleur est le suivant :

Code java :

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
@Controller
@SessionAttributes("sessionBean")
public class AccountController {
    @Autowired
    private SessionBean sessionBean;
 
    /** Display account page and process updates: password, personnal information, company, delivery and billing addresses
     * @return JSP name
     */
    @RequestMapping("/account")
    public String accountPage(/* request params */) {
        //checks not displayed
        Person p=sessionBean.getPerson();
        s.createQuery("UPDATE Person SET tittle=:t, firstname=:fn, lastname=:ln, phone=:p, fax=:f, email=:e  WHERE id=:id")
            .setParameter("t", p.getTitle())
            .setParameter("fn", p.getFirstname())
            .setParameter("ln", p.getLastname())
            .setParameter("p", p.getPhone())
            .setParameter("f", p.getFax())
            .setParameter("e", p.getEmail())
            .setParameter("id", p.getId())
            .executeUpdate();
    }
}

Les classes SessionBean et personnes sont définies ci-dessous :

Code java :

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
@Component
@Scope(value = "session", proxyMode = ScopedProxyMode.TARGET_CLASS)
public class SessionBean implements Serializable {
    private static final long serialVersionUID = 9139554982970790165L;
    private final String id = UUID.randomUUID().toString();
    private Person person = null;
    private String redirect=null;
    private Order3d order3d=new Order3d();
    //getter and setters
}
 
@Entity
@Table (name="person")
public class Person implements Serializable {
    private static final long serialVersionUID = 3005461811554821039L;
    @Id
    @GeneratedValue(strategy=GenerationType.AUTO)
    @Column (name="id")
    private int id;
    @Column (name="email")
    private String email;
    @Column (name="company")
    private String company;
    @Column (name="siret")
    private long siret;
    @Column (name="payment")
    private String payment;
    @Column (name="title")
    private String title;
    @Column (name="firstname")
    private String firstname;
    @Column (name="lastname")
    private String lastname;
    @Column (name="password", columnDefinition="char", length=64)
    private String password;
    @Column (name="phone")
    private String phone;
    @Column (name="fax")
    private String fax;
}

Dans mon formulaire je récupère sans problème les données à afficher stockées dans Person, mais, quand je veux exécuter mon UPDATE, les données soumises ne sont pas prises en compte.

[ruscov]

Quand tu vas en debug, il t'affiche quelque chose à la ligne

Code :

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

Person p=sessionBean.getPerson();

?

[smarties]

Oui mais j'ai les valeurs avant modification

[fr1man]

Pour tes paramètres enlève les deux points :

Code :

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

1
2
 
.setParameter("t", p.getTitle())

au lieu de

Code :

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

1
2
 
.setParameter(":t", p.getTitle())

[smarties]

Code java :

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

1
2
3
4
5
6
7
8
9
10
			Person p=sessionBean.getPerson();
			s.createQuery("UPDATE Person SET title=:t, firstname=:fn, lastname=:ln, phone=:p, fax=:f, email=:e  WHERE id=:id")
				.setParameter("t", p.getTitle())
				.setParameter("fn", p.getFirstname())
				.setParameter("ln", p.getLastname())
				.setParameter("p", p.getPhone())
				.setParameter("f", p.getFax())
				.setParameter("e", p.getEmail())
				.setParameter("id", p.getId())
				.executeUpdate();

J'ai toujours le même problème, je ne récupère pas les données modifiées.

[fr1man]

Et tu n'as aucun message d'erreur ?

[smarties]

Non, je produits ce résultat

Hibernate: update person set title=?, firstname=?, lastname=?, phone=?, fax=?, email=? where id=?
13:47:29,671 TRACE BasicBinder:84 - binding parameter [1] as [VARCHAR] - M
13:47:29,672 TRACE BasicBinder:84 - binding parameter [2] as [VARCHAR] - Seb
13:47:29,673 TRACE BasicBinder:84 - binding parameter [3] as [VARCHAR] - Seb
13:47:29,673 TRACE BasicBinder:84 - binding parameter [4] as [VARCHAR] - 0612345678
13:47:29,674 TRACE BasicBinder:84 - binding parameter [5] as [VARCHAR] -
13:47:29,675 TRACE BasicBinder:84 - binding parameter [6] as [VARCHAR] - something@yopmail.com
13:47:29,677 TRACE BasicBinder:84 - binding parameter [7] as [INTEGER] - 1

Alors que j'ai changé le numéro de téléphone par 0687654321

[fr1man]

Les données que tu veux mettre à jour sont bien issues de ton objet Personne p ?

[smarties]

Oui, le code concerné du contrôleur est dans mon premier message

[fr1man]

Sauf qu'on ne sait pas ce contient réellement ton objet personne.
Ca donne quoi au debugger comme te l'a suggéré ruscov ?

[ruscov]

Ca vient de faire tilt. C'est normal que tu n'ais pas les nouvelles valeurs, dans ton Controller, je vois que ta méthode reçoit des params. Il faut que tu set le param que tu reçois dans ton sessionBean. C'est pas automatique dans la manière que tu as développé ton Controller et ton Bean.

[smarties]

Merci ruscov, ça marche en faisant ça :

Code java :

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

1
2
3
4
5
6
@RequestMapping("/account")
    public String accountPage(/* request params */
        @ModelAttribute("sessionBean") SessionBean sb) {
    //...
    return "account";
}