Discussion :

[viny1976]

Bonjour

Je dois creer un petite application de réservation de véhicules.
Dans un formulaire un utilisateur doit renseigner les champs suivants :

• le nombre de personnes transportées( formalisé sous forme des noms des personnes séparés par une virgule)
• la date de réservation au format DD/MM/YYYY
• heure de début au format HH:MM (exple 08:00 ce sont des heurs justes pas de minutes)
• le motif du deplacement (textarea)
• la destination

J'ai une fonction qui calcule les réservations existantes avec comme paramètre la date auquel la personne souhaite réserver, heure:minute début, heure:minute de fin et un paramètre qui dit si la réservation est confirmée.
soit $oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoiture($dte, $time_jusqua, $time_a);
une regle de gestion imposé est que voiture qui a le parcours prioritaire annulerait une reservation avec un cet élément inférieur.
soit $parcoursPrioritaireForml = $this->_parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));

voici le main.php du module reservation

Code php :

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
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
<?php
class module_reservations extends abstract_module{
 
	public function before(){
		$this->oLayout=new _layout('bootstrap');
 
		$this->oLayout->addModule('menu','menu::index');
	}
 
 
	public function _index(){
	    //on considere que la page par defaut est la page de listage
	    $this->_list();
	}
 
 
	public function _list(){
 
		$tReservations=model_reservations::getInstance()->findAll();
 
		$oView=new _view('reservations::list');
		$oView->tReservations=$tReservations;
 
				$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
				$oView->tJoinmodel_centres=model_centres::getInstance()->getSelect();
				$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
				$oView->tJoinmodel_users=model_users::getInstance()->getSelectPersTrans();
				$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
 
 
		$oModulePagination=new module_pagination;
		$oModulePagination->setModuleAction('reservations::list');
		$oModulePagination->setParamPage('page');
		$oModulePagination->setLimit(10);
		$oModulePagination->setPage( _root::getParam('page') );
		$oModulePagination->setTab( $tReservations );
 
		$oView->tReservations=$oModulePagination->getPageElement();
 
		$this->oLayout->add('main',$oView);
 
 
		$oViewPagination=$oModulePagination->build();
 
		$this->oLayout->add('main',$oViewPagination);
 
	}
 
	public function _new(){
 
		$tMessage=$this->processSave();
 
		$oReservations=new row_reservations;
		$tGroupUsers=model_groupsusers::getInstance()->findByUserId(_root::getAuth()->getAccount()->id);
 
 
		$tCentres_id = model_centres::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
		$tServices_id = model_services::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
 
		$oView=new _view('reservations::new');
		$oView->oReservations=$oReservations;
 
 
				$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
				$oView->tCentres_id = $tCentres_id;
				$oView->tServices_id = $tServices_id;
				$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
				$oView->tJoinmodel_users=model_users::getInstance()->getSelectPersTrans();
				$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
				$oView->iGroupUsers=$tGroupUsers;
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	public function _edit(){
		$tMessage=$this->processSave();
 
		$oReservations=model_reservations::getInstance()->findById( _root::getParam('id') );
		$tCentres_id=model_centres::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
		$tServices_id=model_services::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
 
		$oView=new _view('reservations::edit');
		$oView->oReservations=$oReservations;
		$oView->tId=model_reservations::getInstance()->getIdTab();
 
				$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
				$oView->tJoinmodel_centres=model_centres::getInstance()->getSelect();
				$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
				$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
				$oView->tCentres_id = $tCentres_id;
				$oView->tServices_id = $tServices_id;
				$oView->tJoinmodel_annulation=['0'=>'NON','1'=>'OUI'];
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	public function _show(){
		$oReservations=model_reservations::getInstance()->findById( _root::getParam('id') );
 
		$tServices_id=model_services::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
		$tCentres_id=model_centres::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
 
		$oView=new _view('reservations::show');
		$oView->oReservations=$oReservations;
 
				$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
				$oView->tJoinmodel_centres=model_centres::getInstance()->getSelect();
				$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
				$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
				$oView->tServices_id = $tServices_id;
				$oView->tCentreName= $tCentres_id->name;
				$oView->tJoinmodel_annulation=['0'=>'NON','1'=>'OUI'];
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	public function _delete(){
		$tMessage=$this->processDelete();
 
		$oReservations=model_reservations::getInstance()->findById( _root::getParam('id') );
		$tCentres_id=model_centres::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
		$tServices_id=model_services::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
 
		$oView=new _view('reservations::delete');
		$oView->oReservations=$oReservations;
 
				$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
				$oView->tJoinmodel_centres=model_centres::getInstance()->getSelect();
				$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
				$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
				$oView->tCentres_id = $tCentres_id;
				$oView->tServices_id = $tServices_id;
				$oView->tJoinmodel_annulation=['0'=>'NON','1'=>'OUI'];
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
	public function _parcoursPrioritaire($centre,$bareme,$pers){
 
		$vDepart = (model_centres::getInstance()->findById($centre));
		$villeDepart = $vDepart ->name ;
		$vArrivee = (model_bareme::getInstance()->findById($bareme));
		$villeArrivee = $vArrivee ->ville_arrivee;
 
		$km = model_bareme::getInstance()->findBaremeKm($villeDepart,$villeArrivee);
		$pers_trans=explode(',',$pers);
 
		$parcours_prioritaire = $km->km * 2 * count($pers_trans);
		return $parcours_prioritaire;
	}
 
	/*public function _ReservationsExistantes($date,$voiture,$param){
		
		if ($date !== null) {
			$dte =$date->toString('Y-m-d');
			$oReservationsExistantes = "";
			if ($param == "Confirmee") {
				
				//savoir si il existe deja une réservation avec les parametres voitures, date_du
				$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoiture($voiture, $dte);
				
			}else{
				//savoir si il existe deja une réservation avec les parametres voitures, date_du
				$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnulee($voiture, $dte);
			}
		}else{
			$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnuleeByUser($voiture);
		}
		
		
		return $oReservationsExistantes;

	}*/
 
	public function _ReservationsExistantes($date,$time_jusqua, $time_a, $param){
 
			if ($date !== null) {
				$dte =$date->toString('Y-m-d');
				$oReservationsExistantes = "";
				if ($param == "Confirmee") {
 
					//savoir si il existe deja une réservation avec les parametres voitures, date_du
					$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoiture($dte, $time_jusqua, $time_a);
					$parcoursPrioritaireForml = $this->_parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
 
 
 
				}else{
					//savoir si il existe deja une réservation avec les parametres voitures, date_du
					$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnulee($dte, $time_jusqua, $time_a);
				}
			}else{
				$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnuleeByUser();
			}
 
 
			return $oReservationsExistantes;
 
		}
 
	private function processSave(){
		if(!_root::getRequest()->isPost() ){ //si ce n'est pas une requete POST on ne soumet pas
			return null;
		}
 
		$oPluginXsrf=new plugin_xsrf();
		if(!$oPluginXsrf->checkToken( _root::getParam('token') ) ){ //on verifie que le token est valide
			return array('token'=>$oPluginXsrf->getMessage() );
		}
 
 
		$oReservationsExistantes = $this->_ReservationsExistantes(
																	new plugin_datetime(_root::getParam('date_du'),'d/m/Y'),
																	_root::getParam('time_jusqua'),
																	_root::getParam('time_a'),
														"Confirmee");
 
		$iId=_root::getParam('id',null);
 
 
		if($iId==null && $oReservationsExistantes==null){
			//en mode new
			$oReservations=new row_reservations;
			$tColumn=array('services_id','users_id','centres_id','baremes_id','time_a','time_jusqua','personnes_transportees','motif_deplacement');
 
			foreach($tColumn as $sColumn){
				$oReservations->$sColumn=_root::getParam($sColumn,null) ;
			}
 
			$parcoursPrioritaire = $this->_parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
			$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
			$oBonneDate= $oDate3->toString('Y-m-d');
			$oReservations->date_du = $oBonneDate;
			$oReservations->parcours_prioritaire = $parcoursPrioritaire;
			$oReservations->situation = "Confirmee";
		}else if ($iId==null && $oReservationsExistantes !==null){
			$parcoursPrioritaireForml = $this->_parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
 
 
 
			foreach ($oReservationsExistantes as $key => $value) {
				if ($parcoursPrioritaireForml > $value->parcours_prioritaire){
 
					$value->situation ="Annulee";
					$date2 = new DateTime(date('Y-m-d H:i:s'),new DateTimeZone('Indian/Reunion'));
					$value->date_updated = $date2->format('Y-m-d H:i:s');
					$value->save();
 
					$oReservations=new row_reservations;
					$oReservations->services_id = _root::getParam('services_id',null);
					$oReservations->users_id = _root::getParam('users_id',null);
					$oReservations->voitures_id= _root::getParam('voitures_id',null);
					$oReservations->centres_id= _root::getParam('centres_id',null);
					$oReservations->baremes_id= _root::getParam('baremes_id',null);
					$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
					$oBonneDate= $oDate3->toString('Y-m-d');
					$oReservations->date_du = $oBonneDate;
					$oReservations->time_a= _root::getParam('time_a',null);
					$oReservations->time_jusqua= _root::getParam('time_jusqua',null);
					$oReservations->personnes_transportees= _root::getParam('personnes_transportees',null);
					$oReservations->motif_deplacement= _root::getParam('motif_deplacement',null);
					$oReservations->situation= "Confirmee";
					$oReservations->parcours_prioritaire = $parcoursPrioritaireForml;
 
 
				}else {
					$oReservations=new row_reservations;
					$oReservations->services_id = _root::getParam('services_id',null);
					$oReservations->users_id = _root::getParam('users_id',null);
					$oReservations->voitures_id= _root::getParam('voitures_id',null);
					$oReservations->centres_id= _root::getParam('centres_id',null);
					$oReservations->baremes_id= _root::getParam('baremes_id',null);
					$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
					$oBonneDate= $oDate3->toString('Y-m-d');
					$oReservations->date_du = $oBonneDate;
					$oReservations->time_a= _root::getParam('time_a',null);
					$oReservations->time_jusqua= _root::getParam('time_jusqua',null);
					$oReservations->personnes_transportees= _root::getParam('personnes_transportees',null);
					$oReservations->motif_deplacement= _root::getParam('motif_deplacement',null);
					$oReservations->situation= "Reservation impossible";
					$oReservations->parcours_prioritaire = $parcoursPrioritaireForml;
 
				}
 
			}
			/*
			echo "<pre>";
			var_dump("parcoursPrioritaireForml  value->parcours_prioritaire ".$parcoursPrioritaireForml);
			var_dump($resTri);
			echo "</pre>";
			*/
 
 
 
		}else if ($iId !==null && $oReservationsExistantes !==null){
 
			$oReservationEditer = model_reservations::getInstance()->findById($iId);
 
			$parcoursPrioritaireForml = $this->_parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
 
			if (_root::getParam('annulation',null) == "1") {
 
				$reservationsExistantes = $this->_ReservationsExistantes(null,_root::getParam('voitures_id',null),"Annulee");
 
				$res=[];
				function plusGrand($a,$b){
					if ($a > $b) {
						return $a;
					}
					return $b;
				}
				foreach ($reservationsExistantes as $key => $reservation) {
					$res[$reservation->id][$reservation->id]=$reservation->parcours_prioritaire;
					$reserv= array_reduce($res,"plusGrand");
 
				}
 
				$oReservations = model_reservations::getInstance()->findById(key($reserv));
				$oReservations->situation = "Confirmee";
				$date2 = new DateTime(date('Y-m-d H:i:s'),new DateTimeZone('Indian/Reunion'));
				$oReservations->date_updated = $date2->format('Y-m-d H:i:s');
				$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
				$oBonneDate= $oDate3->toString('Y-m-d');
				$oReservations->date_du = $oBonneDate;
				$oReservations->save();
 
 
				$oReservationEditer->situation = "Annulee par l'utilisateur";
				$oReservationEditer->date_updated = $date2->format('Y-m-d H:i:s');
				$oReservationEditer->date_annulation = $date2->format('Y-m-d H:i:s');
				$oReservationEditer->annulation = 1;
				$oReservationEditer->save();
				_root::redirect('reservations::list');
 
 
 
			}else{
				foreach ($oReservationsExistantes as $key => $value) {
					if ($parcoursPrioritaireForml > $value->parcours_prioritaire){
 
						$value->situation ="Annulee";
						$date2 = new DateTime(date('Y-m-d H:i:s'),new DateTimeZone('Indian/Reunion'));
						$value->date_updated = $date2->format('Y-m-d H:i:s');
						$value->save();
 
						$oReservations=new row_reservations;
						$oReservations->services_id = _root::getParam('services_id',null);
						$oReservations->users_id = _root::getParam('users_id',null);
						$oReservations->voitures_id= _root::getParam('voitures_id',null);
						$oReservations->centres_id= _root::getParam('centres_id',null);
						$oReservations->baremes_id= _root::getParam('baremes_id',null);
						$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
						$oBonneDate= $oDate3->toString('Y-m-d');
						$oReservations->date_du = $oBonneDate;
						$oReservations->time_a= _root::getParam('time_a',null);
						$oReservations->time_jusqua= _root::getParam('time_jusqua',null);
						$oReservations->personnes_transportees= _root::getParam('personnes_transportees',null);
						$oReservations->motif_deplacement= _root::getParam('motif_deplacement',null);
						$oReservations->situation= "Confirmee";
						$oReservations->parcours_prioritaire = $parcoursPrioritaireForml;
						$oReservations->save();
						_root::redirect('reservations::list');
 
					}else if ($parcoursPrioritaireForml == $value->parcours_prioritaire) {
						$oReservations=new row_reservations;
						$oReservations->services_id = _root::getParam('services_id',null);
						$oReservations->users_id = _root::getParam('users_id',null);
						$oReservations->voitures_id= _root::getParam('voitures_id',null);
						$oReservations->centres_id= _root::getParam('centres_id',null);
						$oReservations->baremes_id= _root::getParam('baremes_id',null);
						$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
						$oBonneDate= $oDate3->toString('Y-m-d');
						$oReservations->date_du = $oBonneDate;
						$oReservations->time_a= _root::getParam('time_a',null);
						$oReservations->time_jusqua= _root::getParam('time_jusqua',null);
						$oReservations->personnes_transportees= _root::getParam('personnes_transportees',null);
						$oReservations->motif_deplacement= _root::getParam('motif_deplacement',null);
						$oReservations->situation= "Reservation impossible";
						$oReservations->parcours_prioritaire = $parcoursPrioritaireForml;
						$oReservations->save();
						_root::redirect('reservations::list');
					}
 
				}
			}
 
 
		}else{
			//en mode edit
			$oReservations=model_reservations::getInstance()->findById( _root::getParam('id',null) );
			$tColumn=array('services_id','users_id','voitures_id','centres_id','baremes_id','time_a','time_jusqua','personnes_transportees','motif_deplacement');
			foreach($tColumn as $sColumn){
				$oReservations->$sColumn=_root::getParam($sColumn,null) ;
			}
			$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
			$oBonneDate= $oDate3->toString('Y-m-d');
			$oReservations->date_du = $oBonneDate;
		}
 
 
 
 
 
		if($oReservations->save() ){
 
			//une fois enregistre on redirige (vers la page liste)
			_root::redirect('reservations::list');
		}else{
			return $oReservations->getListError();
		}
 
	}
 
 
 
	public function processDelete(){
		if(!_root::getRequest()->isPost() ){ //si ce n'est pas une requete POST on ne soumet pas
			return null;
		}
 
		$oPluginXsrf=new plugin_xsrf();
		if(!$oPluginXsrf->checkToken( _root::getParam('token') ) ){ //on verifie que le token est valide
			return array('token'=>$oPluginXsrf->getMessage() );
		}
 
		$oReservations=model_reservations::getInstance()->findById( _root::getParam('id',null) );
 
		$oReservations->delete();
		//une fois enregistre on redirige (vers la page liste)
		_root::redirect('reservations::list');
 
	}
 
 
	public function after(){
		$this->oLayout->show();
	}
 
 
}

voici le model_reservations.php

Code php :

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
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
<?php
class model_reservations extends abstract_model{
 
	protected $sClassRow='row_reservations';
 
	protected $sTable='reservations';
	protected $sConfig='pdoMysqlExple';
 
	protected $tId=array('id');
 
	public static function getInstance(){
		return self::_getInstance(__CLASS__);
	}
 
	public function findById($uId){
		return $this->findOne('SELECT * FROM '.$this->sTable.' WHERE id=?',$uId );
	}
	public function findAll(){
		return $this->findMany('SELECT * FROM '.$this->sTable .' ORDER BY date_created DESC');
	}
 
	/*public function findReservationsByVoiture($voiture,$date_du,$time_jusqua,$time_a){
		return $this->findAll('
							SELECT 
								voitures_id, date_du, time_a, time_jusqua, situation 
							FROM 
								'.$this->sTable.' 
							WHERE 
								voitures_id=? AND 
								date_du=? AND
								situation = Confirmée AND
								time_a <=? AND 
								time_jusqua >=?',
					$voiture,$date_du,$time_jusqua,$time_a);
	}*/
 
	/*public function findReservationsByVoiture($voiture, $date_du){
		return $this->findMany('SELECT * FROM '.$this->sTable.' 
								WHERE  situation like "Confirmee" AND 
								voitures_id=? AND 
								date_du=?  ',$voiture,$date_du);
	}*/
	public function findReservationsByVoiture($date_du, $time_jusqua, $time_a){
		return $this->findMany('SELECT * FROM '.$this->sTable.' 
								WHERE  situation like "Confirmee" AND 
								date_du=?  AND
								hour(time_a) <=? AND 
								hour(time_jusqua) >=?',
								$date_du,$time_jusqua,$time_a);
	}
 
	/*public function findReservationsByVoitureAnnulee($voiture,$date_du){
		return $this->findMany('SELECT * FROM '.$this->sTable.' 
								WHERE  situation like "Annulee" AND 
								voitures_id=? AND 
								date_du=?  ',$voiture,$date_du);
	}*/
	public function findReservationsByVoitureAnnulee($date_du, $time_jusqua, $time_a){
		return $this->findMany('SELECT * FROM '.$this->sTable.' 
								WHERE  situation like "Annulee" AND 
								date_du=?AND
								time_a <=? AND 
								time_jusqua >=?',
								$date_du,$time_jusqua,$time_a);
	}
 
	/*public function findReservationsByVoitureAnnuleeByUser($voiture){
		return $this->findMany('SELECT * FROM '.$this->sTable.' 
								WHERE  situation like "Annulee" AND 
								voitures_id=? ',$voiture);
	}*/
	public function findReservationsByVoitureAnnuleeByUser(){
		return $this->findMany('SELECT * FROM '.$this->sTable.' 
								WHERE  situation like "Annulee par l\'utilisateur"');
	}
}
 
class row_reservations extends abstract_row{
 
	protected $sClassModel='model_reservations';
 
	/*exemple jointure
	public function findAuteur(){
		return model_auteur::getInstance()->findById($this->auteur_id);
	}
	*/
	/*exemple test validation*/
	private function getCheck(){
		$oPluginValid=new plugin_valid($this->getTab());
 
 
		/* renseigner vos check ici
		$oPluginValid->isEqual('champ','valeurB','Le champ n\est pas &eacute;gal &agrave; '.$valeurB);
		$oPluginValid->isNotEqual('champ','valeurB','Le champ est &eacute;gal &agrave; '.$valeurB);
		$oPluginValid->isUpperThan('champ','valeurB','Le champ n\est pas sup&eacute; &agrave; '.$valeurB);
		$oPluginValid->isUpperOrEqualThan('champ','valeurB','Le champ n\est pas sup&eacute; ou &eacute;gal &agrave; '.$valeurB);
		$oPluginValid->isLowerThan('champ','valeurB','Le champ n\est pas inf&eacute;rieur &agrave; '.$valeurB);
		$oPluginValid->isLowerOrEqualThan('champ','valeurB','Le champ n\est pas inf&eacute;rieur ou &eacute;gal &agrave; '.$valeurB);
		$oPluginValid->isEmpty('champ','Le champ n\'est pas vide');
		$oPluginValid->isNotEmpty('champ','Le champ ne doit pas &ecirc;tre vide');
		$oPluginValid->isEmailValid('champ','L\email est invalide');
		$oPluginValid->matchExpression('champ','/[0-9]/','Le champ n\'est pas au bon format');
		$oPluginValid->notMatchExpression('champ','/[a-zA-Z]/','Le champ ne doit pas &ecirc;tre a ce format');
		*/
 
		return $oPluginValid;
	}
 
	public function isValid(){
		return $this->getCheck()->isValid();
	}
	public function getListError(){
		return $this->getCheck()->getListError();
	}
 
	//on surcharge le constructeur
	public function __construct($tRow = ''){
 
		if ($tRow != '' ) {
			//on modifie le champ date avant d'initialiser l'objet pour l'affichage
			$oDate = new plugin_date($tRow['date_du']);
			$tRow['date_du'] = $oDate->toString('d/m/Y');
			$oDate = new plugin_datetime($tRow['date_created'],'Y-m-d h:i:s');
			$tRow['date_created'] = $oDate->toString('d/m/Y H:i:s');
 
			if  ($tRow['date_updated']){
				$oDate = new plugin_datetime($tRow['date_updated'],'Y-m-d h:i:s');
				$tRow['date_updated'] = $oDate->toString('d/m/Y H:i:s');
			}
 
		}
 
		//on appelle le constructeur normal de la row
		parent::__construct($tRow);
	 }
 
	public function save(){
		if(!$this->isValid()){
			return false;
		}
 
		/*FORMATAGE DES DATES*/
		//on formate à la volée
       /*$oDate = new plugin_date($this->date_du,'d/m/Y');
       $this->date_du = $oDate->toString('Y-m-d');
	   $oDate = new plugin_datetime($this->date_created,'Y-m-d h:i:s');
       $this->date_created = $oDate->toString('d/m/Y h:i:s');*/
 
		parent::save();
		return true;
	}
 
}

J'aurais aimé savoir comment vous auriez procéder procéder??

Merci pour vos conseils

[imikado]

Bonsoir,
Excusez moi, mais quel est la question ? vous voulez que je vous aide à ecrire quoi ? le formulaire ? la view ?

[viny1976]

En fait la question comment auriez vous formaliser la situation au niveau du modèle et au niveau du contrôleur.
Désolez si je n’ai pas été assez clair

[imikado]

Bonsoir,

Plusieurs remarques deja qui peuvent porter à confusion: ne nommez pas, dans votre model vos methodes internes avec un underscore, la convention c'est de nommez les "actions" (ou pages) avec cet underscore, hors les actions/pages ne sont pas des méthodes qui acceptent des parametres

Voila déjà le module avec les méthodes corrigées

Code php :

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
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
<?php
class module_reservations extends abstract_module{
 
	public function before(){
		$this->oLayout=new _layout('bootstrap');
 
		$this->oLayout->addModule('menu','menu::index');
	}
 
 
	public function _index(){
	    //on considere que la page par defaut est la page de listage
	    $this->_list();
	}
 
 
	public function _list(){
 
		$tReservations=model_reservations::getInstance()->findAll();
 
		$oView=new _view('reservations::list');
		$oView->tReservations=$tReservations;
 
		$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
		$oView->tJoinmodel_centres=model_centres::getInstance()->getSelect();
		$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
		$oView->tJoinmodel_users=model_users::getInstance()->getSelectPersTrans();
		$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
 
 
		$oModulePagination=new module_pagination;
		$oModulePagination->setModuleAction('reservations::list');
		$oModulePagination->setParamPage('page');
		$oModulePagination->setLimit(10);
		$oModulePagination->setPage( _root::getParam('page') );
		$oModulePagination->setTab( $tReservations );
 
		$oView->tReservations=$oModulePagination->getPageElement();
 
		$this->oLayout->add('main',$oView);
 
 
		$oViewPagination=$oModulePagination->build();
 
		$this->oLayout->add('main',$oViewPagination);
 
	}
 
	public function _new(){
 
		$tMessage=$this->processSave();
 
		$oReservations=new row_reservations;
		$tGroupUsers=model_groupsusers::getInstance()->findByUserId(_root::getAuth()->getAccount()->id);
 
 
		$tCentres_id = model_centres::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
		$tServices_id = model_services::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
 
		$oView=new _view('reservations::new');
		$oView->oReservations=$oReservations;
 
 
		$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
		$oView->tCentres_id = $tCentres_id;
		$oView->tServices_id = $tServices_id;
		$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
		$oView->tJoinmodel_users=model_users::getInstance()->getSelectPersTrans();
		$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
		$oView->iGroupUsers=$tGroupUsers;
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	public function _edit(){
		$tMessage=$this->processSave();
 
		$oReservations=model_reservations::getInstance()->findById( _root::getParam('id') );
		$tCentres_id=model_centres::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
		$tServices_id=model_services::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
 
		$oView=new _view('reservations::edit');
		$oView->oReservations=$oReservations;
		$oView->tId=model_reservations::getInstance()->getIdTab();
 
		$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
		$oView->tJoinmodel_centres=model_centres::getInstance()->getSelect();
		$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
		$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
		$oView->tCentres_id = $tCentres_id;
		$oView->tServices_id = $tServices_id;
		$oView->tJoinmodel_annulation=['0'=>'NON','1'=>'OUI'];
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	public function _show(){
		$oReservations=model_reservations::getInstance()->findById( _root::getParam('id') );
 
		$tServices_id=model_services::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
		$tCentres_id=model_centres::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
 
		$oView=new _view('reservations::show');
		$oView->oReservations=$oReservations;
 
		$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
		$oView->tJoinmodel_centres=model_centres::getInstance()->getSelect();
		$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
		$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
		$oView->tServices_id = $tServices_id;
		$oView->tCentreName= $tCentres_id->name;
		$oView->tJoinmodel_annulation=['0'=>'NON','1'=>'OUI'];
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	public function _delete(){
		$tMessage=$this->processDelete();
 
		$oReservations=model_reservations::getInstance()->findById( _root::getParam('id') );
		$tCentres_id=model_centres::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
		$tServices_id=model_services::getInstance()->findById(_root::getAuth()->getAccount()->centres_id);
 
		$oView=new _view('reservations::delete');
		$oView->oReservations=$oReservations;
 
		$oView->tJoinmodel_services=model_services::getInstance()->getSelect();
		$oView->tJoinmodel_centres=model_centres::getInstance()->getSelect();
		$oView->tJoinmodel_bareme=model_bareme::getInstance()->getSelect();
		$oView->tJoinmodel_voitures=model_voitures::getInstance()->getSelect();
		$oView->tCentres_id = $tCentres_id;
		$oView->tServices_id = $tServices_id;
		$oView->tJoinmodel_annulation=['0'=>'NON','1'=>'OUI'];
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
	public function parcoursPrioritaire($centre,$bareme,$pers){
 
		$vDepart = (model_centres::getInstance()->findById($centre));
		$villeDepart = $vDepart ->name ;
		$vArrivee = (model_bareme::getInstance()->findById($bareme));
		$villeArrivee = $vArrivee ->ville_arrivee;
 
		$km = model_bareme::getInstance()->findBaremeKm($villeDepart,$villeArrivee);
		$pers_trans=explode(',',$pers);
 
		$parcours_prioritaire = $km->km * 2 * count($pers_trans);
		return $parcours_prioritaire;
	}
 
 
	public function ReservationsExistantes($date,$time_jusqua, $time_a, $param){
 
		if ($date !== null) {
			$dte =$date->toString('Y-m-d');
			$oReservationsExistantes = "";
			if ($param == "Confirmee") {
 
				//savoir si il existe deja une réservation avec les parametres voitures, date_du
				$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoiture($dte, $time_jusqua, $time_a);
				$parcoursPrioritaireForml = $this->parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
 
 
 
			}else{
				//savoir si il existe deja une réservation avec les parametres voitures, date_du
				$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnulee($dte, $time_jusqua, $time_a);
			}
		}else{
			$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnuleeByUser();
		}
 
 
		return $oReservationsExistantes;
 
	}
 
	private function processSave(){
		if(!_root::getRequest()->isPost() ){ //si ce n'est pas une requete POST on ne soumet pas
			return null;
		}
 
		$oPluginXsrf=new plugin_xsrf();
		if(!$oPluginXsrf->checkToken( _root::getParam('token') ) ){ //on verifie que le token est valide
			return array('token'=>$oPluginXsrf->getMessage() );
		}
 
 
		$oReservationsExistantes = $this->ReservationsExistantes(
																	new plugin_datetime(_root::getParam('date_du'),'d/m/Y'),
																	_root::getParam('time_jusqua'),
																	_root::getParam('time_a'),
														"Confirmee");
 
		$iId=_root::getParam('id',null);
 
 
		if($iId==null && $oReservationsExistantes==null){
			//en mode new
			$oReservations=new row_reservations;
			$tColumn=array('services_id','users_id','centres_id','baremes_id','time_a','time_jusqua','personnes_transportees','motif_deplacement');
 
			foreach($tColumn as $sColumn){
				$oReservations->$sColumn=_root::getParam($sColumn,null) ;
			}
 
			$parcoursPrioritaire = $this->parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
			$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
			$oBonneDate= $oDate3->toString('Y-m-d');
			$oReservations->date_du = $oBonneDate;
			$oReservations->parcours_prioritaire = $parcoursPrioritaire;
			$oReservations->situation = "Confirmee";
		}else if ($iId==null && $oReservationsExistantes !==null){
			$parcoursPrioritaireForml = $this->parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
 
 
 
			foreach ($oReservationsExistantes as $key => $value) {
				if ($parcoursPrioritaireForml > $value->parcours_prioritaire){
 
					$value->situation ="Annulee";
					$date2 = new DateTime(date('Y-m-d H:i:s'),new DateTimeZone('Indian/Reunion'));
					$value->date_updated = $date2->format('Y-m-d H:i:s');
					$value->save();
 
					$oReservations=new row_reservations;
					$oReservations->services_id = _root::getParam('services_id',null);
					$oReservations->users_id = _root::getParam('users_id',null);
					$oReservations->voitures_id= _root::getParam('voitures_id',null);
					$oReservations->centres_id= _root::getParam('centres_id',null);
					$oReservations->baremes_id= _root::getParam('baremes_id',null);
					$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
					$oBonneDate= $oDate3->toString('Y-m-d');
					$oReservations->date_du = $oBonneDate;
					$oReservations->time_a= _root::getParam('time_a',null);
					$oReservations->time_jusqua= _root::getParam('time_jusqua',null);
					$oReservations->personnes_transportees= _root::getParam('personnes_transportees',null);
					$oReservations->motif_deplacement= _root::getParam('motif_deplacement',null);
					$oReservations->situation= "Confirmee";
					$oReservations->parcours_prioritaire = $parcoursPrioritaireForml;
 
 
				}else {
					$oReservations=new row_reservations;
					$oReservations->services_id = _root::getParam('services_id',null);
					$oReservations->users_id = _root::getParam('users_id',null);
					$oReservations->voitures_id= _root::getParam('voitures_id',null);
					$oReservations->centres_id= _root::getParam('centres_id',null);
					$oReservations->baremes_id= _root::getParam('baremes_id',null);
					$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
					$oBonneDate= $oDate3->toString('Y-m-d');
					$oReservations->date_du = $oBonneDate;
					$oReservations->time_a= _root::getParam('time_a',null);
					$oReservations->time_jusqua= _root::getParam('time_jusqua',null);
					$oReservations->personnes_transportees= _root::getParam('personnes_transportees',null);
					$oReservations->motif_deplacement= _root::getParam('motif_deplacement',null);
					$oReservations->situation= "Reservation impossible";
					$oReservations->parcours_prioritaire = $parcoursPrioritaireForml;
 
				}
 
			}
			/*
			echo "<pre>";
			var_dump("parcoursPrioritaireForml  value->parcours_prioritaire ".$parcoursPrioritaireForml);
			var_dump($resTri);
			echo "</pre>";
			*/
 
 
 
		}else if ($iId !==null && $oReservationsExistantes !==null){
 
			$oReservationEditer = model_reservations::getInstance()->findById($iId);
 
			$parcoursPrioritaireForml = $this->parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
 
			if (_root::getParam('annulation',null) == "1") {
 
				$reservationsExistantes = $this->ReservationsExistantes(null,_root::getParam('voitures_id',null),"Annulee");
 
				$res=[];
				function plusGrand($a,$b){
					if ($a > $b) {
						return $a;
					}
					return $b;
				}
				foreach ($reservationsExistantes as $key => $reservation) {
					$res[$reservation->id][$reservation->id]=$reservation->parcours_prioritaire;
					$reserv= array_reduce($res,"plusGrand");
 
				}
 
				$oReservations = model_reservations::getInstance()->findById(key($reserv));
				$oReservations->situation = "Confirmee";
				$date2 = new DateTime(date('Y-m-d H:i:s'),new DateTimeZone('Indian/Reunion'));
				$oReservations->date_updated = $date2->format('Y-m-d H:i:s');
				$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
				$oBonneDate= $oDate3->toString('Y-m-d');
				$oReservations->date_du = $oBonneDate;
				$oReservations->save();
 
 
				$oReservationEditer->situation = "Annulee par l'utilisateur";
				$oReservationEditer->date_updated = $date2->format('Y-m-d H:i:s');
				$oReservationEditer->date_annulation = $date2->format('Y-m-d H:i:s');
				$oReservationEditer->annulation = 1;
				$oReservationEditer->save();
				_root::redirect('reservations::list');
 
 
 
			}else{
				foreach ($oReservationsExistantes as $key => $value) {
					if ($parcoursPrioritaireForml > $value->parcours_prioritaire){
 
						$value->situation ="Annulee";
						$date2 = new DateTime(date('Y-m-d H:i:s'),new DateTimeZone('Indian/Reunion'));
						$value->date_updated = $date2->format('Y-m-d H:i:s');
						$value->save();
 
						$oReservations=new row_reservations;
						$oReservations->services_id = _root::getParam('services_id',null);
						$oReservations->users_id = _root::getParam('users_id',null);
						$oReservations->voitures_id= _root::getParam('voitures_id',null);
						$oReservations->centres_id= _root::getParam('centres_id',null);
						$oReservations->baremes_id= _root::getParam('baremes_id',null);
						$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
						$oBonneDate= $oDate3->toString('Y-m-d');
						$oReservations->date_du = $oBonneDate;
						$oReservations->time_a= _root::getParam('time_a',null);
						$oReservations->time_jusqua= _root::getParam('time_jusqua',null);
						$oReservations->personnes_transportees= _root::getParam('personnes_transportees',null);
						$oReservations->motif_deplacement= _root::getParam('motif_deplacement',null);
						$oReservations->situation= "Confirmee";
						$oReservations->parcours_prioritaire = $parcoursPrioritaireForml;
						$oReservations->save();
						_root::redirect('reservations::list');
 
					}else if ($parcoursPrioritaireForml == $value->parcours_prioritaire) {
						$oReservations=new row_reservations;
						$oReservations->services_id = _root::getParam('services_id',null);
						$oReservations->users_id = _root::getParam('users_id',null);
						$oReservations->voitures_id= _root::getParam('voitures_id',null);
						$oReservations->centres_id= _root::getParam('centres_id',null);
						$oReservations->baremes_id= _root::getParam('baremes_id',null);
						$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
						$oBonneDate= $oDate3->toString('Y-m-d');
						$oReservations->date_du = $oBonneDate;
						$oReservations->time_a= _root::getParam('time_a',null);
						$oReservations->time_jusqua= _root::getParam('time_jusqua',null);
						$oReservations->personnes_transportees= _root::getParam('personnes_transportees',null);
						$oReservations->motif_deplacement= _root::getParam('motif_deplacement',null);
						$oReservations->situation= "Reservation impossible";
						$oReservations->parcours_prioritaire = $parcoursPrioritaireForml;
						$oReservations->save();
						_root::redirect('reservations::list');
					}
 
				}
			}
 
 
		}else{
			//en mode edit
			$oReservations=model_reservations::getInstance()->findById( _root::getParam('id',null) );
			$tColumn=array('services_id','users_id','voitures_id','centres_id','baremes_id','time_a','time_jusqua','personnes_transportees','motif_deplacement');
			foreach($tColumn as $sColumn){
				$oReservations->$sColumn=_root::getParam($sColumn,null) ;
			}
			$oDate3 = new plugin_date(_root::getParam('date_du'),'d/m/Y');
			$oBonneDate= $oDate3->toString('Y-m-d');
			$oReservations->date_du = $oBonneDate;
		}
 
 
 
 
 
		if($oReservations->save() ){
 
			//une fois enregistre on redirige (vers la page liste)
			_root::redirect('reservations::list');
		}else{
			return $oReservations->getListError();
		}
 
	}
 
 
 
	public function processDelete(){
		if(!_root::getRequest()->isPost() ){ //si ce n'est pas une requete POST on ne soumet pas
			return null;
		}
 
		$oPluginXsrf=new plugin_xsrf();
		if(!$oPluginXsrf->checkToken( _root::getParam('token') ) ){ //on verifie que le token est valide
			return array('token'=>$oPluginXsrf->getMessage() );
		}
 
		$oReservations=model_reservations::getInstance()->findById( _root::getParam('id',null) );
 
		$oReservations->delete();
		//une fois enregistre on redirige (vers la page liste)
		_root::redirect('reservations::list');
 
	}
 
 
	public function after(){
		$this->oLayout->show();
	}
 
 
}

En regardant le code je vois pas mal de complexité, je vous invite a sortir cette logique métier dans une classe à part afin de pouvoir la soumettre à des test unitaires pour eviter les surprises

qu'en dites vous ?

[viny1976]

Bonjour,

Le nommage est une erreur de ma part.
Merci pour la remarque.

je vais faire ce que vous avez préconisé.
Mais dites moi vous avez deja des exemples de classes personnalisées sur votre site ?
Pour l'ide GEANY peux tu me dire les extensions utiles pour le développement en php?

Et les autres utilisateurs du mkframework vos remarques sont les bienvenues.

[imikado]

Vous avez un répertoire "myClasses" dans votre projet ?

dedans il doit y avoir un fichier d'exmple "metier.php" ?

c'est un exemple de classe metier

dans l'idée vous creer une classe avec des methodes qui ne travaillent qu'avec les arguments passées, sinon vous ne pourrez pas faire de test unitaires dessus

vous avez deja utilisé phpunit ?

[viny1976]

Non je n'ai jamais utilisé phpunit.

Donc je ne crée que des méthodes dans cette classe que j'appelle ensuite dans mon contrôleur??

[imikado]

Phpunit est un framework php qui permet de tester votre code

Pour cela

1. vous creer votre classe metier qui aura pour chaque methodes des paramètres et un retour
2. vous creer un repertoire "tests" à la racine de votre projet et une classe php "lenomDeVotreClasseAtesterTest" de test dans l qui instanciera votre classe metier et permettra de tester chacune de ses méthodes
3. vous telecharger phpunit https://phpunit.de/index.html
4. dans le builder vous pouvez lancer les test unitaire e nindiquant le chemin vers phpunit

[viny1976]

Voila ce que j'ai fait pour ma classe metier

Code php :

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
<?php
 
class reservationExistante{
 
	public function parcoursPrioritaire($centre,$bareme,$pers){
 
		$vDepart = (model_centres::getInstance()->findById($centre));
		$villeDepart = $vDepart ->name ;
		$vArrivee = (model_bareme::getInstance()->findById($bareme));
		$villeArrivee = $vArrivee ->ville_arrivee;
 
		$km = model_bareme::getInstance()->findBaremeKm($villeDepart,$villeArrivee);
		$pers_trans=explode(',',$pers);
 
		$parcours_prioritaire = $km->km * 2 * count($pers_trans);
		return $parcours_prioritaire;
	}
 
	/*public function reservationsExistantes($date,$voiture,$param){
		
		if ($date !== null) {
			$dte =$date->toString('Y-m-d');
			$oReservationsExistantes = "";
			if ($param == "Confirmee") {
				
				//savoir si il existe deja une réservation avec les parametres voitures, date_du
				$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoiture($voiture, $dte);
				
			}else{
				//savoir si il existe deja une réservation avec les parametres voitures, date_du
				$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnulee($voiture, $dte);
			}
		}else{
			$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnuleeByUser($voiture);
		}
		
		
		return $oReservationsExistantes;

	}*/
 
	public function reservationsExistantes($date,$time_jusqua, $time_a, $param){
 
			if ($date !== null) {
				$dte =$date->toString('Y-m-d');
				$oReservationsExistantes = "";
				if ($param == "Confirmee") {
 
					//savoir si il existe deja une réservation avec les parametres voitures, date_du
					$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoiture($dte, $time_jusqua, $time_a);
					$parcoursPrioritaireForml = $this->parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
 
 
 
				}else{
					//savoir si il existe deja une réservation avec les parametres voitures, date_du
					$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnulee($dte, $time_jusqua, $time_a);
				}
			}else{
				$oReservationsExistantes = model_reservations::getInstance()->findReservationsByVoitureAnnuleeByUser();
			}
 
 
			return $oReservationsExistantes;
 
		}
}

qd je fais $test = new reservationExistantes(); puis $test->reservationsExistantes(param...)

ma classe metier doit s'appeler my_nomdelaclasse??

[imikado]

Oui votre fichier doit s'appeler reservationExistante.php et votre classe my_reservationExistante

Ensuite pour la classe elle meme il faut permettre qu'elle soit testable, voila les modification a y apporter:

Code php :

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
 
<?php
 
class my_reservationExistante{
 
	protected $oModelCentres;
	protected $oModelBareme;
	protected $oModel_reservations;
 
	public function __construct($oModelCentres,$oModelBareme,$oModel_reservations){
		$this->oModelCentres=$oModelCentres;
		$this->oModelBareme=$oModelBareme;
		$this->oModel_reservations=$oModel_reservations;
	}
 
	public function parcoursPrioritaire($centre,$bareme,$pers){
 
		$vDepart = ($this->oModelCentres->findById($centre));
		$villeDepart = $vDepart ->name ;
		$vArrivee = ($this->oModelBareme->findById($bareme));
		$villeArrivee = $vArrivee ->ville_arrivee;
 
		$km = $this->oModelBareme->findBaremeKm($villeDepart,$villeArrivee);
		$pers_trans=explode(',',$pers);
 
		$parcours_prioritaire = $km->km * 2 * count($pers_trans);
		return $parcours_prioritaire;
	}
 
	/*public function reservationsExistantes($date,$voiture,$param){

		if ($date !== null) {
			$dte =$date->toString('Y-m-d');
			$oReservationsExistantes = "";
			if ($param == "Confirmee") {

				//savoir si il existe deja une réservation avec les parametres voitures, date_du
				$oReservationsExistantes = $this->oModel_reservations->findReservationsByVoiture($voiture, $dte);

			}else{
				//savoir si il existe deja une réservation avec les parametres voitures, date_du
				$oReservationsExistantes = $this->oModel_reservations->findReservationsByVoitureAnnulee($voiture, $dte);
			}
		}else{
			$oReservationsExistantes = $this->oModel_reservations->findReservationsByVoitureAnnuleeByUser($voiture);
		}


		return $oReservationsExistantes;

	}*/
 
	public function reservationsExistantes($date,$time_jusqua, $time_a, $param){
 
			if ($date !== null) {
				$dte =$date->toString('Y-m-d');
				$oReservationsExistantes = "";
				if ($param == "Confirmee") {
 
					//savoir si il existe deja une réservation avec les parametres voitures, date_du
					$oReservationsExistantes = $this->oModel_reservations->findReservationsByVoiture($dte, $time_jusqua, $time_a);
					$parcoursPrioritaireForml = $this->parcoursPrioritaire(_root::getParam('centres_id',null),_root::getParam('baremes_id',null),_root::getParam('personnes_transportees',null));
 
 
 
				}else{
					//savoir si il existe deja une réservation avec les parametres voitures, date_du
					$oReservationsExistantes = $this->oModel_reservations->findReservationsByVoitureAnnulee($dte, $time_jusqua, $time_a);
				}
			}else{
				$oReservationsExistantes = $this->oModel_reservations->findReservationsByVoitureAnnuleeByUser();
			}
 
 
			return $oReservationsExistantes;
 
		}
}

[imikado]

je vais vous poster un exemple de fichier de tes unitaire phpunit

[imikado]

Voila un exemple de test unitaire avec votre classe:

tests/reservationExistanteTest.php

Code php :

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
 
<?php
 
 
$tClassToInclude=array(
	'model/model_centres.php',
	'model/model_bareme.php',
	'model/model_reservations.php',
	'myClasses/reservationExistante.php',
);
 
foreach($tClassToInclude as $sClassFilename){
	include(__DIR__.'/../'.$sClassFilename);
 
}
 
/**
 * @runTestsInSeparateProcesses
 * @preserveGlobalState disabled
 */
class reservationExistanceTest extends PHPUnit_Framework_TestCase
{
 
	 public function run(PHPUnit_Framework_TestResult $result = NULL)
    {
        $this->setPreserveGlobalState(false);
        return parent::run($result);
    }
 
		public function test_parcoursPrioritaireShouldFinisOk(){
 
			//a la constuction de la classe, celle-ci prend les 3 modeles __construct($oModelCentres,$oModelBareme,$oModel_reservations)
 
			//nous les reproduisons avec des Mocks: ces classes permettent de simuler un comportement
 
			//Arrange
 
			$oMookModelCentres=$this->createMock('model_centres',new model_centres());
			$oModelBareme=$this->createMock('model_bareme',new model_bareme());
			$oModelReservations=$this->createMock('model_reservations',new model_reservations());
 
			//on simule le retour de nos requetes:
			$oVilleDepart=new stdclass();
			$oVilleDepart->name='paris';
			$oMookModelCentres->method('findById')->willReturn($oVilleDepart);
 
			$oVilleArrivee=new stdclass();
			$oVilleArrivee->ville_arrivee='marseille';
			$oModelBareme->method('findById')->willReturn($oVilleArrivee);
 
			$oKm=new stdclass();
			$oKm->km=774;
			$oModelBareme->method('findBaremeKm')->willReturn($oKm);
 
			$oReservation=new my_reservationExistante($oMookModelCentres,$oModelBareme,$oModelReservations);
 
			$centre=12;
			$bareme=24;
			$pers='toto,titi,tata';
 
			//Act
			$resultCalculated=$oReservation->parcoursPrioritaire($centre,$bareme,$pers);
 
			//Assert
 
			//on attend de avec ces parametres le resultat suivant nb kim x 2 x nb personnes
			$resultExpected=774*2*3;
 
			$this->assertEquals($resultExpected,$resultCalculated);
 
			//si vous voulez tester le test unitaire faites un assert faux: 343 par exemple
			//$this->assertEquals(343,$resultCalculated);
 
		}
}

[viny1976]

MErci pour votre aiguillage et votre temps.

Je n'ai pas pu m'y pencher aujourd'hui, fête des mères oblige.

Merci pour votre soutien et je pense que vous avoir dans une équipe de développement ce doit être un plaisir de travailler avec vous.

Je vous tiens au courant de la suite.

[imikado]

Merci, c'est un plaisir

Merci d'utiliser ce framework, il manque de notoriété

je prepare un article pour developpez pour presenter le nouveau mode de distribution et en profiter pour presenter le framework à ceux qui ne connaitrait pas

[viny1976]

Bonjour,

J'ai fait ce que vous aviez préconisé.
Mais lorsque je lance phpunit depuis le builder, j'obtiens ce message :

PHP Fatal error: Class 'abstract_model' not found in /var/www/html/projet5/mkframework/data/genere/oreli3/model/model_centres.php on line 2

Voilà pourtant la casse model_centres :

Code php :

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
<?php
class model_centres extends abstract_model{
 
	protected $sClassRow='row_centres';
 
	protected $sTable='centres';
	protected $sConfig='pdoMysqlExple';
 
	protected $tId=array('id');
 
	public static function getInstance(){
		return self::_getInstance(__CLASS__);
	}
 
	public function findById($uId){
		return $this->findOne('SELECT * FROM '.$this->sTable.' WHERE id=?',$uId );
	}
	public function findAll(){
		return $this->findMany('SELECT * FROM '.$this->sTable);
	}
 
 
	public function getSelect(){
		$tab=$this->findAll();
		$tSelect=array();
		if($tab){
		foreach($tab as $oRow){
			$tSelect[ $oRow->name ]=$oRow->name;
		}
		}
		return $tSelect;
	}
 
 
 
}
 
class row_centres extends abstract_row{
 
	protected $sClassModel='model_centres';
 
	/*exemple jointure 
	public function findAuteur(){
		return model_auteur::getInstance()->findById($this->auteur_id);
	}
	*/
	/*exemple test validation*/
	private function getCheck(){
		$oPluginValid=new plugin_valid($this->getTab());
 
 
		/* renseigner vos check ici
		$oPluginValid->isEqual('champ','valeurB','Le champ n\est pas &eacute;gal &agrave; '.$valeurB);
		$oPluginValid->isNotEqual('champ','valeurB','Le champ est &eacute;gal &agrave; '.$valeurB);
		$oPluginValid->isUpperThan('champ','valeurB','Le champ n\est pas sup&eacute; &agrave; '.$valeurB);
		$oPluginValid->isUpperOrEqualThan('champ','valeurB','Le champ n\est pas sup&eacute; ou &eacute;gal &agrave; '.$valeurB);
		$oPluginValid->isLowerThan('champ','valeurB','Le champ n\est pas inf&eacute;rieur &agrave; '.$valeurB);
		$oPluginValid->isLowerOrEqualThan('champ','valeurB','Le champ n\est pas inf&eacute;rieur ou &eacute;gal &agrave; '.$valeurB);
		$oPluginValid->isEmpty('champ','Le champ n\'est pas vide');
		$oPluginValid->isNotEmpty('champ','Le champ ne doit pas &ecirc;tre vide');
		$oPluginValid->isEmailValid('champ','L\email est invalide');
		$oPluginValid->matchExpression('champ','/[0-9]/','Le champ n\'est pas au bon format');
		$oPluginValid->notMatchExpression('champ','/[a-zA-Z]/','Le champ ne doit pas &ecirc;tre a ce format');
		*/
 
		return $oPluginValid;
	}
 
	public function isValid(){
		return $this->getCheck()->isValid();
	}
	public function getListError(){
		return $this->getCheck()->getListError();
	}
	public function save(){
		if(!$this->isValid()){
			return false;
		}
		parent::save();
		return true;
	}
 
}

[imikado]

Oui en effet il faut l'inclure galement dans votre classe de test, j'avais fait le test avec une classe vide

faites un include de

Code :

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

1
2
 
include '/cheminVersLeFramework/lib/framework/abstract/abstract_model.php

pour vous ça devraiit etre:

Code php :

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

1
2
 
include ('/var/www/html/projet5/mkframework/lib/framework/abstract/abstract_model.php');

[viny1976]

Mon environnement :
php 7.0.27
raspbian
PHPUnit 6.5.8

Pour ceux qui sont dans la panade comme moi au début voilà le fichier modifié pour que cela fonctionne :

Code php :

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
<?php
include ('/var/www/html/projet5/mkframework/lib/framework/abstract/abstract_model.php');
include ('/var/www/html/projet5/mkframework/lib/framework/abstract/abstract_row.php');
 
 
$tClassToInclude=array(
	'model/model_centres.php',
	'model/model_bareme.php',
	'model/model_reservations.php',
	'myClasses/reservationExistante.php',
);
 
foreach($tClassToInclude as $sClassFilename){
	include(__DIR__.'/../'.$sClassFilename);
}
 
 
 
 
/**
 * @runTestsInSeparateProcesses
 * @preserveGlobalState disabled
 */
class reservationExistanteTest extends \PHPUnit\Framework\TestCase
{
 
	 public function run(\PHPUnit\Framework\TestResult $result = NULL)
    {
        $this->setPreserveGlobalState(false);
        return parent::run($result);
    }
 
		public function test_parcoursPrioritaireShouldFinisOk(){
 
			//a la constuction de la classe, celle-ci prend les 3 modeles __construct($oModelCentres,$oModelBareme,$oModel_reservations)
 
			//nous les reproduisons avec des Mocks: ces classes permettent de simuler un comportement
 
			//Arrange
 
			$oMookModelCentres=$this->createMock('model_centres',new model_centres());
			$oModelBareme=$this->createMock('model_bareme',new model_bareme());
			$oModelReservations=$this->createMock('model_reservations',new model_reservations());
 
			//on simule le retour de nos requetes:
			$oVilleDepart=new stdclass();
			$oVilleDepart->name='paris';
			$oMookModelCentres->method('findById')->willReturn($oVilleDepart);
 
			$oVilleArrivee=new stdclass();
			$oVilleArrivee->ville_arrivee='marseille';
			$oModelBareme->method('findById')->willReturn($oVilleArrivee);
 
			$oKm=new stdclass();
			$oKm->km=774;
			$oModelBareme->method('findBaremeKm')->willReturn($oKm);
 
			$oReservation=new my_reservationExistante($oMookModelCentres,$oModelBareme,$oModelReservations);
 
			$centre=12;
			$bareme=24;
			$pers='toto,titi,tata';
 
			//Act
			$resultCalculated=$oReservation->parcoursPrioritaire($centre,$bareme,$pers);
 
			//Assert
 
			//on attend de avec ces parametres le resultat suivant nb kim x 2 x nb personnes
			$resultExpected=774*2*3;
 
			$this->assertEquals($resultExpected,$resultCalculated);
 
			//si vous voulez tester le test unitaire faites un assert faux: 343 par exemple
			//$this->assertEquals(343,$resultCalculated);
 
		}
}

Quelle est la suite?

[imikado]

Vous arrivez à lancer le test unitaire ?

si oui il faut parametrer vos autres methodes "complexe" pour verifier si l'algorythme est ok

ensuite il fautr dans votre fichier main.php remplacer le code actuel par l'appel à cette classe comme vous le faites dans le test unitaire

[viny1976]

Oui j'ai lancé le test unitaire et j'ai eu un OK.

Je vais m'atteler à créer ces méthodes.

Vous êtes quelqu'un de trés pédagogue, vous donnez des cours ?

Un moyen pour faire connaître votre framework encore plus serait de proposer des podcasts sur des projets plus en lien avec des projets réels....

[imikado]

Merci

Non j'écris des tutoriels sur developpez et pour le magazine programmez

je fais egalement des slides sur slidesshare

Il faudrait en effet que je refasse des videos screencast (mais c'est plus lourd à mettre en place: besoin d'une piece sans bruit, preparer sa presentation pour eviter les "trou" ..)
il faut que j'en refasse une pour presenter le nouveau mode de distribution d'ailleurs