Problème de mise a jour des données avec mkframeworkPackage-1.2.0

Version imprimable

je travail sur une application avec mkframeworkPackage-1.2.0, j'ai importer abstract_rownoincrement et abstract_modelnoincrement. toutefois, je rencontre un problème dans la mise à jour des données. Voila l'erreur quand je clique "Edit" et que je valide

Code:

04/08/2019, 10h32
imikado

Bonjour,
d'apres la requete

Code:

UPDATE type_panne SET WHERE Type_Panne=?

l'objet ne doit pas avoir de champ modifié d'ou le soucis lors de l'update en base de donnée

Il faut au minimum modifier un champ de l'objet pour que l'update fonctionne bien
05/08/2019, 10h38
Edja

ça suppose si je n'abuse que toutes les tables doivent avoir un Id automatique
05/08/2019, 11h31
imikado

Non, je voulais dire que l'erreur est produite car aucun champ de l'objet n'a été mis à jour

pour optimiser les requetes, les update ne sont fait que sur les champs modifiés dans l'objet

ainsi, meme sur une table de 15 champs, si on modifie uniquement le champ titre, seul celui-ci sera updaté

Puis-je voir le code du fichier main:

Projects\ParcInfoMAE\module\BackOffice\main.php autour de la ligne 1088

je suppose la méthode processSave
05/08/2019, 12h40
Edja

aussi, je pense que pour résoudre se problème, le module de génération de la vue Edit doit ajouter un champs caché qui récupère l'Id. puis tous les traitement seront fait sur ce champs comme ça on évitera cette erreur et l'obligation d'avoir des id automatiques....

voici le contenu de mon fichier main

Code:

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
//-----------------------------------------------------------------Gestion des Types de Panne----------------------------------------------------------------------//
    public function _TypePanneList(){
 
		$tType_panne=model_type_panne::getInstance()->findAll();
 
		$oView=new _view('BackOffice::TypePanneList');
		$oView->tType_panne=$tType_panne;
 
 
		$this->oLayout->add('main',$oView);
 
	}
 
 
	public function _TypePanneNew(){
		$tMessage=$this->TypePanneProcessSave();
 
		$oType_panne=new row_type_panne;
 
		$oView=new _view('BackOffice::TypePanneNew');
		$oView->oType_panne=$oType_panne;
 
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	public function _TypePanneEdit(){
		$tMessage=$this->TypePanneProcessSave();
 
		$oType_panne=model_type_panne::getInstance()->findById( _root::getParam('id') );
 
		$oView=new _view('BackOffice::TypePanneEdit');
		$oView->oType_panne=$oType_panne;
		$oView->tId=model_type_panne::getInstance()->getIdTab();
 
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	public function _TypePanneDelete(){
		$tMessage=$this->TypePanneProcessDelete();
 
		$oType_panne=model_type_panne::getInstance()->findById( _root::getParam('id') );
 
		$oView=new _view('BackOffice::TypePanneDelete');
		$oView->oType_panne=$oType_panne;
 
 
		$oPluginXsrf=new plugin_xsrf();
		$oView->token=$oPluginXsrf->getToken();
		$oView->tMessage=$tMessage;
 
		$this->oLayout->add('main',$oView);
	}
 
 
 
	private function TypePanneProcessSave(){
		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() );
		}
 
		$iId=_root::getParam('id',null);
		if($iId==null){
			$oType_panne=new row_type_panne;	
		}else{
			$oType_panne=model_type_panne::getInstance()->findById( _root::getParam('id',null) );
		}
 
		$tColumn=array('Type_Panne');
		foreach($tColumn as $sColumn){
			$oType_panne->$sColumn=_root::getParam($sColumn,null) ;
		}
 
 
		if($oType_panne->save()){
			//une fois enregistre on redirige (vers la page liste)
			_root::redirect('BackOffice::TypePanneList');
		}else{
			return $oType_panne->getListError();
		}
 
	}
 
 
	public function TypePanneProcessDelete(){
		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() );
		}
 
		$oType_panne=model_type_panne::getInstance()->findById( _root::getParam('id',null) );
 
		$oType_panne->delete();
		//une fois enregistre on redirige (vers la page liste)
		_root::redirect('BackOffice::TypePanneList');
 
	}

05/08/2019, 14h19
imikado
En regardant le code on voit le soucis ici:
Code:

1 2 3 4 $tColumn=array('Type_Panne'); foreach($tColumn as $sColumn){ $oType_panne->$sColumn=_root::getParam($sColumn,null) ; }
il y a un seul champ mis à jour ici c'est le champ Type_Panne

hors quel est le champ d'identification de cette table ?

IL faut 2 champs au minimum: un champ d'identification et un champ de contenu, sinon en effet ça ne peut pas marcher
Dans votre cas, il faudrait (comme vous l'indiquiez) sans doute ajouter un champ d'id incrémental
05/08/2019, 15h16
Edja

Merci,. mais est-ce-qu'on ne peut pas envisager ceci:

lors de la génération de la vue Edit, qu'il y ai un champ hidden qui recupère l'ID comme dans mon cas. et qte la clause where soit basée sur se champs hidden tan disque la valeur à modifier reste "Type_panne"

cordialement

Bonjour,
Je pense qu'il y a un quiproquo

vous parlez d'ajouter un champ hidden contenant l'id, mais on a l'equivalent ici avec l'id dans la barre d'url
le problème est ailleursl:

habituellement vous avez au minium 2 champs pour une table: un identifiant et N champs de contenu
on peut ainsi dans un formulaire faire un édit avec l'id de l'enregistrement et modifier un ou plusiuers champs provoquant une requete SQL du type:

Code:

UPDATE MA_TABLE SET MON_CHAMP=ma_valeur WHERE MON_ID=L_IDENTIFIANT

hors ici, on se retrouve avec apparement un seul champ sur la table d'ou le soucis: il faut au minium un id et un champ de valeur pour que la génération de CRUD fonctionne ici

oui je pourrais faire une modification mais elle est impactante pour tous les autres utilisateurs, c'est ça le soucis

sinon dans votre cas il faudrait modifier le fichier main.php pour ne pas utiliser la fonction save mais ecrire une methode particulière

ou sinon il faudrait modifier la classe modèle du framework: abstract_modelnoincrement

pour remplacer
Code:

1 2 3 4 5 6 7 public function update($oRow){ $tColumnToUpdate=$oRow->getToUpdate(); unset($tColumnToUpdate[$this->tId[0] ]); $this->getSgbd()->update($this->sTable,$tColumnToUpdate,$oRow->getWhere()); /*LOG*/_root::getLog()->info('sql update:'.$this->getSgbd()->getRequete()); }
par
Code:

1 2 3 4 5 6 7 public function update($oRow){ $tColumnToUpdate=$oRow->getToUpdate(); //unset($tColumnToUpdate[$this->tId[0] ]); $this->getSgbd()->update($this->sTable,$tColumnToUpdate,$oRow->getWhere()); /*LOG*/_root::getLog()->info('sql update:'.$this->getSgbd()->getRequete()); }
et ainsi cela devrait fonctionner

22/08/2019, 20h12
Edja

toujours problème

Bonjour imicado, j'ai appliqué la solution, mais elle n'a pas marché. La résolution de ce problème est très capital. En fait je travail sur un projet de mon ministère. J'ai des tables issues d'une hierachisation-spécialisation, et je n'arrive pas à mettre toujour à jour ces tables. Toujours une erreur

//----------------------------------------------------------------erreur------------------------------------------------------------------------------------

PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens
#0 [internal function]: exception_error_handler(2, 'PDOStatement::e...', 'C:\\wamp64\\www\\m...', 251, Array)
#1 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_sgbd_pdo.php(251): PDOStatement->execute(Array)
#2 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_sgbd_pdo.php(204): abstract_sgbd_pdo->query('UPDATE ordinate...', Array)
#3 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_modelnoincrement.php(177): abstract_sgbd_pdo->update('ordinateur', Array, Array)
#4 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_rownoincrement.php(65): abstract_modelnoincrement->update(Object(row_ordinateur))
#5 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_rownoincrement.php(54): abstract_rownoincrement->update()
#6 C:\wamp64\www\mkframeworkPackage-1.2.0\Projects\ParcInfoMAE - V2\model\model_ordinateur.php(79): abstract_rownoincrement->save()
#7 C:\wamp64\www\mkframeworkPackage-1.2.0\Projects\ParcInfoMAE - V2\module\BackOffice\main.php(3079): row_ordinateur->save()
#8 C:\wamp64\www\mkframeworkPackage-1.2.0\Projects\ParcInfoMAE - V2\module\BackOffice\main.php(3016): module_BackOffice->OrdinateurProcessSave()
#9 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\class_root.php(264): module_BackOffice->_OrdinateurEdit()
#10 C:\wamp64\www\mkframeworkPackage-1.2.0\Projects\ParcInfoMAE - V2\public\index.php(48): _root->run()
#11 {main}

Detail:
#0 exception_error_handler( , 'PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens' , 'C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_sgbd_pdo.php' , , Array ( [sReq] => UPDATE ordinateur SET `TypeOrdinateur`=?,`MicroIntegre`=?,`EnceinteIntegre`=?,`Materiel_SN`=? WHERE Materiel_SN=? [tParam] => Array ( [0] => ORDINATEUR DE BUREAU [1] => NON [2] => OUI [3] => CZC62485B0 ) [tATTRERRMODE] => Array ( [SILENT] => 0 [WARNING] => 1 [EXCEPTION] => 2 ) [tATTRCASE] => Array ( [LOWER] => 2 [NATURAL] => 0 [UPPER] => 1 ) [sth] => PDOStatement Object ( [queryString] => UPDATE ordinateur SET `TypeOrdinateur`=?,`MicroIntegre`=?,`EnceinteIntegre`=?,`Materiel_SN`=? WHERE Materiel_SN=? ) ) )
#1 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_sgbd_pdo.php (251)
PDOStatement -> execute( Array ( [0] => ORDINATEUR DE BUREAU [1] => NON [2] => OUI [3] => CZC62485B0 ) )
#2 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_sgbd_pdo.php (204)
abstract_sgbd_pdo -> query( 'UPDATE ordinateur SET `TypeOrdinateur`=?,`MicroIntegre`=?,`EnceinteIntegre`=?,`Materiel_SN`=? WHERE Materiel_SN=?' , Array ( [0] => ORDINATEUR DE BUREAU [1] => NON [2] => OUI [3] => CZC62485B0 ) )
#3 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_modelnoincrement.php (177)
abstract_sgbd_pdo -> update( 'ordinateur' , Array ( [TypeOrdinateur] => ORDINATEUR DE BUREAU [MicroIntegre] => NON [EnceinteIntegre] => OUI [Materiel_SN] => CZC62485B0 ) , Array ( [Materiel_SN] => CZC62485B0 ) )
#4 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_rownoincrement.php (65)
abstract_modelnoincrement -> update( )
#5 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\abstract\abstract_rownoincrement.php (54)
abstract_rownoincrement -> update( )
#6 C:\wamp64\www\mkframeworkPackage-1.2.0\Projects\ParcInfoMAE - V2\model\model_ordinateur.php (79)
abstract_rownoincrement -> save( )
#7 C:\wamp64\www\mkframeworkPackage-1.2.0\Projects\ParcInfoMAE - V2\module\BackOffice\main.php (3079)
row_ordinateur -> save( )
#8 C:\wamp64\www\mkframeworkPackage-1.2.0\Projects\ParcInfoMAE - V2\module\BackOffice\main.php (3016)
module_BackOffice -> OrdinateurProcessSave( )
#9 C:\wamp64\www\mkframeworkPackage-1.2.0\Lib\mkframeworkLib\class_root.php (264)
module_BackOffice -> _OrdinateurEdit( )
#10 C:\wamp64\www\mkframeworkPackage-1.2.0\Projects\ParcInfoMAE - V2\public\index.php (48)
_root -> run( )
#11 {main}

Bonjour,
j'ai trouvé une solution: mais il faut pour l'instant modifier un fichier du framework,

la classe abstract/abstract_sgbd_pdo.php

remplacer
Code:

1 2 3 4 5 6 7 8 9 10 public function update($sTable,$tProperty,$tWhere){ $sReq='UPDATE '.$sTable.' SET '.$this->getUpdateFromTab($tProperty).' WHERE '.$this->getWhereFromTab($tWhere); $tPropertyAndWhere=array_merge($tProperty,$tWhere); $tParam=array_values($tPropertyAndWhere); $this->query($sReq,$tParam); }
par
Code:

1 2 3 4 5 6 7 8 9 10 public function update($sTable,$tProperty,$tWhere){ $sReq='UPDATE '.$sTable.' SET '.$this->getUpdateFromTab($tProperty).' WHERE '.$this->getWhereFromTab($tWhere); $tPropertyAndWhere=array_merge(array_values($tProperty),array_values($tWhere)); $tParam=array_values($tPropertyAndWhere); $this->query($sReq,$tParam); }
Mais je repose ma question: pourquoi ne pas ajouter un champ id incrémtal ?
car la je sais pas à quoi sert cette table, mais on ne peut pas avoir de jointeure avec cette table. Par exemple, si on voudrais changer l'intitulé du type de panne, pour changer l'affichage de toutes les interventions liés à celle-ci on ne peut pas :(

En tout cas j'espère que ce patch reglera votre soucis