Discussion :

[concas]

Bonjour à tous je viens par ce post solliciter votre aimable appui face à une difficulté que je rencontre depuis quelques temps dans mon projet.
En effet j'utilise Primefaces Schedule dans mon projet pour réaliser un agenda.Pour ce faire j'ai choisi de stocker tous les événements dans une table MySQL nommée agenda.
voici mon entité

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
 
@Entity
@ManagedBean
public class Agenda {
	@Id
	@GeneratedValue( strategy = GenerationType.IDENTITY )
	private Long pkIdag;
	@ManyToOne
	@JoinColumn(name="pkIdet")
	private EspaceTravail espaceTravail;
	@NotNull(message="Veuillez entrer le nom de l'évènement!")
	@Size(min=3,message="Votre évènement doit comporter au minimum 3 caractères!")
	private String nom;
	@NotNull(message="Veuillez entrer le lieu de l'évènement!")
	@Size(min=3,message="Le nom du lieu de l'évènement doit avoir au minimum 3 caractères!")
	private String lieu;
	@NotNull(message="Veuillez entrer la description de l'évènement!")
	@Size(max=100,message="La description l'évènement doit avoir au maximum 100 caractères!")
	private String description;
	private Timestamp dateDbt;
	private Timestamp dateFin;
	private String statut;
 
	public Long getPkIdag() {
		return pkIdag;
	}
	public void setPkIdag(Long pkIdag) {
		this.pkIdag = pkIdag;
	}
	public EspaceTravail getEspaceTravail() {
		return espaceTravail;
	}
	public void setEspaceTravail(EspaceTravail espaceTravail) {
		this.espaceTravail = espaceTravail;
	}
	public String getNom() {
		return nom;
	}
	public void setNom(String nom) {
		this.nom = nom;
	}
	public String getLieu() {
		return lieu;
	}
	public void setLieu(String lieu) {
		this.lieu = lieu;
	}
	public String getDescription() {
		return description;
	}
	public void setDescription(String description) {
		this.description = description;
	}
	public Timestamp getDateDbt() {
		return dateDbt;
	}
	public void setDateDbt(Timestamp dateDbt) {
		this.dateDbt = dateDbt;
	}
	public Timestamp getDateFin() {
		return dateFin;
	}
	public void setDateFin(Timestamp dateFin) {
		this.dateFin = dateFin;
	}
	public String getStatut() {
		return statut;
	}
	public void setStatut(String statut) {
		this.statut = statut;
	}

Voici mon backing bean

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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
 
public class AgendaBean implements Serializable{
	private static final long serialVersionUID = 1L;
	private Agenda agenda;
	private EspaceTravail espaceTravail;
	private String nom;
	private Date dateDbt;
	private Date dateFin;
    private ScheduleModel eventModel;
    private ScheduleEvent event;
    List<Agenda>agendas=null;
	@EJB
	private EspaceTravailDao espaceTravailDao;
	@EJB
	private AgendaDao agendaDao;
	public AgendaBean(){
		agenda=new Agenda();
		event= new DefaultScheduleEvent(nom, dateDbt, dateFin, agenda) ;
	}
	@PostConstruct
    public void init() {
		eventModel=new DefaultScheduleModel();
		agendas=agendaDao.trouverAgendaTotal();
		for(Agenda agendItem:agendas){
			Agenda agendaCourant=new Agenda();
			agendaCourant=agendItem;
			String title=agendaCourant.getNom();
			Date dtDbt=new java.util.Date(agendaCourant.getDateDbt().getTime());
			Date dtFin=new java.util.Date(agendaCourant.getDateFin().getTime());
			event=(ScheduleEvent) new DefaultScheduleEvent(title, dtDbt, dtFin,agendaCourant);
			eventModel.addEvent(event);
		}
	}
 
	 public List<Agenda> getAgendas() {
		return agendas;
	}
	public void setAgendas(List<Agenda> agendas) {
		this.agendas = agendas;
	}
	public Agenda getAgenda() {
		return agenda;
	}
	public void setAgenda(Agenda agenda) {
		this.agenda = agenda;
	}
 
	public String getNom() {
		return nom;
	}
	public void setNom(String nom) {
		this.nom = nom;
	}
 
	public Date getDateDbt() {
		return dateDbt;
	}
	public void setDateDbt(Date dateDbt) {
		this.dateDbt = dateDbt;
	}
	public Date getDateFin() {
		return dateFin;
	}
	public void setDateFin(Date dateFin) {
		this.dateFin = dateFin;
	}
	public String outcome(){
		return "listeagenda?faces-redirect=true&includeViewParams=true&nom="+nom;
	}
	public ScheduleModel getEventModel() {
        return eventModel;
    }
    public ScheduleEvent getEvent() {
        return event;
    }
 
    public void setEvent(ScheduleEvent event) {
        this.event = event;
    }
 
    public void addEvent(ActionEvent actionEvent){
        if(event.getId() == null)
        {
	        	agenda.setDateDbt(new java.sql.Timestamp(event.getStartDate().getTime()));
	        	agenda.setDateFin(new java.sql.Timestamp(event.getEndDate().getTime()));
	        	agenda.setNom(event.getTitle());
	        	agenda.setStatut("ouvert");
	        	espaceTravail=espaceTravailDao.trouverEspaceTravailNom(nom);
	        	agenda.setEspaceTravail(espaceTravail);
	        	Agenda nouveauAgenda=new Agenda();
	        	nouveauAgenda=agenda;
	        	agendaDao.creer(nouveauAgenda);
         }
        else{
        	    eventModel.updateEvent(event);
	        	agenda.setDateDbt(new java.sql.Timestamp(event.getStartDate().getTime()));
	        	agenda.setDateFin(new java.sql.Timestamp(event.getEndDate().getTime()));
	        	agenda.setNom(event.getTitle());
	        	agenda.setStatut("ouvert");
	        	espaceTravail=espaceTravailDao.trouverEspaceTravailNom(nom);
	        	agenda.setEspaceTravail(espaceTravail);
	        	agendaDao.modifier(agenda);
        }
        init();
    }
 
    public void onEventSelect(SelectEvent selectEvent) {
        event = (ScheduleEvent) selectEvent.getObject();
        Agenda agendaReceuilli=(Agenda)event.getData();
        agenda.setDescription(agendaReceuilli.getDescription());
        agenda.setLieu(agendaReceuilli.getLieu());
        agenda.setPkIdag(agendaReceuilli.getPkIdag());
    }
 
    public void onDateSelect(SelectEvent selectEvent) {
    	agenda=new Agenda();
        event = (ScheduleEvent) new DefaultScheduleEvent("", (Date) selectEvent.getObject(), (Date) selectEvent.getObject(),agenda);
    }
 
}

Voici ma vue

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
 
<h:form>
			    <p:growl id="messages" showDetail="true" />
			    <h:panelGrid columnClasses="value">
			        <h3 style="margin-top: 0" >#{agendaBean.nom}</h3>
					<p:schedule id="schedule" value="#{agendaBean.eventModel}" widgetVar="myschedule" timeZone="GMT" locale="fr" process="@form" axisFormat="HH:mm" >
			            <p:ajax event="dateSelect" listener="#{agendaBean.onDateSelect}" update="eventDetails" oncomplete="PF('eventDialog').show();" />
			            <p:ajax event="eventSelect" listener="#{agendaBean.onEventSelect}" update="eventDetails" oncomplete="PF('eventDialog').show();" />
			        </p:schedule>
			    </h:panelGrid>
			    <p:dialog widgetVar="eventDialog" header="Evènement" showEffect="clip" hideEffect="clip" height="400" width="400" id="eventDetails" resizable="false" >
			    		<h:outputLabel for="idCleAg">REF <span class="requis">*</span></h:outputLabel>
			    		<h:inputText id="idCleAg" value="#{agendaBean.agenda.pkIdag}"/>
			    		<br/>
			    		<h:outputLabel for="nom">Nom <span class="requis">*</span></h:outputLabel>
						<p:inputText id="nom" value="#{agendaBean.event.title}" required="true" maxlength="60" style="background:#fff;width:370px;margin-left:-4px;"/>
						<h:message id="nomMessage" for="nom" errorClass="erreur" />
						<br/>
						<h:outputLabel for="lieu">Lieu<span class="requis">*</span></h:outputLabel>
						<p:inputText id="lieu" value="#{agendaBean.agenda.lieu}" required="true" maxlength="60" style="background:#fff;width:370px;margin-left:-4px;"/>
						<h:message id="lieuMessage" for="lieu" errorClass="erreur" />
						<br/>
						<h:outputLabel for="description">Lieu<span class="requis">*</span></h:outputLabel>
						<p:inputTextarea id="description" value="#{agendaBean.agenda.description}" rows="10" cols="50" queryDelay="750" minQueryLength="4" required="true"/>
						<h:message id="descriptionMessage" for="lieu" errorClass="erreur" />
						<br/>
			            <p:outputLabel for="du" value="Du:" />
			            <p:calendar locale="fr" id="du" value="#{agendaBean.event.startDate}" pattern="yyyy-MM-dd HH:mm:ss"/>
			 			<br/>
			            <p:outputLabel for="au" value="Au:" />
			            <p:calendar id="au" value="#{agendaBean.event.endDate}" pattern="yyyy-MM-dd HH:mm:ss"/>
			 			<br/>
			            <p:commandButton id="addButton" value="enregistrer" actionListener="#{agendaBean.addEvent}" update="@form" ajax="true" > 
			            	<f:setPropertyActionListener target="#{agendaBean.nom}" value="#{agendaBean.nom}" />
			            </p:commandButton>
			    </p:dialog> 
	</h:form>

Voilà en fait tout marche à merveille mais le problème se trouve au niveau de la mise à jour. Quand c'est pour la première fois la mise à jour fonctionne mais les autres essais échouent et MySQL me dit qu'elle ne retrouve pas la clé primaire or pourtant elle est bien là.
Toute suggestion serait la bienvenue, je vous remercie!

[tchize_]

D'abotd, tu t'obstine constament à créer des variable avec des new pour ensuite immédiatement écraser la valeur avec autre chose. C'est inutile.

Exemple ici dans ton init, ton new Agenda() ne sert à rien

Code :

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

1
2
3
4
5
6
7
8
9
		for(Agenda agendItem:agendas){
			Agenda agendaCourant=new Agenda();
			agendaCourant=agendItem;
			String title=agendaCourant.getNom();
			Date dtDbt=new java.util.Date(agendaCourant.getDateDbt().getTime());
			Date dtFin=new java.util.Date(agendaCourant.getDateFin().getTime());
			event=(ScheduleEvent) new DefaultScheduleEvent(title, dtDbt, dtFin,agendaCourant);
			eventModel.addEvent(event);
		}

et ça s'écrit aussi simplement

Code :

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

1
2
3
4
5
6
7
8
9
		for(Agenda agendaItem:agendas){
			Agenda agendaCourant=new Agenda();
			agendaCourant=agendItem;
			String title=agendaItem.getNom();
			Date dtDbt=agendaItem.getDateDbt());
			Date dtFin=agendaItem.getDateFin());
			event=(ScheduleEvent) new DefaultScheduleEvent(title, dtDbt, dtFin,agendaItem);
			eventModel.addEvent(event);
		}

Ensuite, ton code de mise à jour modifie constamment le même objet "agenda" qui est une champ de ta classe. Est-ce que ton dao fait bien une copie lors des appels de mise à jour / create?
Ton update de son coté, transfère toutes les données de l'event dans un objet agenda, sauf l'id. Donc quand tu fait un update, le DAO ignore quel id il doit mettre à jour.

D'une manière général ton code n'est pas performant. Recharger la liste des agendas à chaque changement n'est peut-être pas une bonne idée.
Ensuite, arrête de copier des choses dans ce champ "agenda" de ta classe, il ne fait qu'embrouiller les choses. Tu a associé un objet Agenda à chaque event, utilise les! Contente toi de faire des choses du genre

Code :

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

1
2
3
4
    public void onEventSelect(SelectEvent selectEvent) {
        event = (ScheduleEvent) selectEvent.getObject();
        agenda=(Agenda)event.getData();
    }

Pareil, tes champs lieu/description/... sont inutiles dans ton Bean, pointe directement vers les propriétés de Agenda dans ton jsf. D'ailleurs pour le bout de code que tu donne, c'est déjà ce que tu fais. Ca te permettra de simplifier fortement ton bean jsf.

[concas]

Merci tchize_ pour tes différentes recommandations je vais tout implémenter et revenir.

[concas]

Merci tchize_ j'ai essayé et tout fonctionne à merveille. Infiniment merci et bonne journée.