Problèmes requête base de données

Version imprimable

Bonjour, depuis peu je rencontre un problème très bizarre sur un des modules.

Dans mon model_pointage.php j'ai ça
Code:

1 2 3 4 5 6 7 public function findHeureTotal($user,$date1,$date2){ return $this->findOne('SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur="'.$user.'" AND date_pointage BETWEEN "'.$date1.'" AND "'.$date2.'"'); //return 'SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur="'.$user.'" AND date_pointage BETWEEN "'.$date1.'" AND "'.$date2.'"'; }
Lorsque j'utilise le return que j'ai commenté ci-dessus, je me rend compte que la requête est bonne car j'ai ça (j'ai fais exprès de ne pas utiliser les '?' juste pour tester)

Code:

'SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM pointage WHERE id_pointeur="10" AND date_pointage BETWEEN "2017-10-01" AND "2017-10-31"'

Lorsque je copie ce code et que je lance la requête dans phpmyadmin ça fonctionne correctement et cela me retourne bien une valeur 03:15:00

Mais lorsque j'utilise la methode findOne, j'ai null comme résultat
Code:

1 2 3 4 5 6 7 8 object(row_pointage)[24] protected 'sClassModel' => string 'model_pointage' (length=14) private '_bChooseUpdate' (abstract_row) => boolean true protected '_tProperty' => array (size=1) 'htt' => null protected '_tPropertyToUpdate' => null
Cela marchait à merveille auparavant, mais je ne sais pas pourquoi maintenant cela ne marche plus :(

La requête normale, c'était ça au départ

Code:

return $this->findOne('SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur=? AND date_pointage BETWEEN ? AND ?',$user,$date1,$date2);

Bonjour,
Vous pouvez verifier la requete executée en activant dans la log :

dans le fichier conf/site.ini.php
Code:

1 2 3 4 5 6 7 8 9 [log] class=plugin_log error=0 warning=0 application=0 information=1 file.enabled=1 apache.enabled=1
Mettez information=1

Ensuite, en rechargeant votre page, vous aurez un fichier dans data/log et vous y trouverez la requete reellement executée

Afin de verifier qu'elle correspond bien à celle attendue

Ensuite une chose qui peut etre faite c'est de remplacer findOne par findOneSimple afin de recuperer l'objet direct de pdo

Remplacez donc
Code:

1 2 3 4 5 6 7 public function findHeureTotal($user,$date1,$date2){ return $this->findOne('SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur="'.$user.'" AND date_pointage BETWEEN "'.$date1.'" AND "'.$date2.'"'); //return 'SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur="'.$user.'" AND date_pointage BETWEEN "'.$date1.'" AND "'.$date2.'"'; }
par
Code:

1 2 3 4 5 6 7 public function findHeureTotal($user,$date1,$date2){ return $this->findOneSimple('SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur="'.$user.'" AND date_pointage BETWEEN "'.$date1.'" AND "'.$date2.'"'); //return 'SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur="'.$user.'" AND date_pointage BETWEEN "'.$date1.'" AND "'.$date2.'"'; }
Enfin 3: il est deconseille de concatener les variables au sein de la requete SQL, c'est tres dangereux (SQLinjection), preferez l'utilisation de prepare statement

ce qui donnerait ici:
Code:

1 2 3 4 5 6 7 public function findHeureTotal($user,$date1,$date2){ return $this->findOneSimple('SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur=? AND date_pointage BETWEEN ? AND ? ',$user,$date1,$date2); }

26/10/2017, 15h46
2boubacar10

Résolu

Bonjour,
Merci des réponses super rapide. En faite, la requête était bonne. Le bon résultat était bien NULL. Je vais essayer de le gérer pour la suite au cas ou on a une réponse NULL ;)

Citation:
Envoyé par imikado
Bonjour,

Enfin 3: il est deconseille de concatener les variables au sein de la requete SQL, c'est tres dangereux (SQLinjection), preferez l'utilisation de prepare statement

ce qui donnerait ici:
Code:

1 2 3 4 5 6 7 public function findHeureTotal($user,$date1,$date2){ return $this->findOneSimple('SELECT SEC_TO_TIME( SUM( TIME_TO_SEC( subtime(time(heure_s),time(heure_e)) ) ) ) as htt FROM '.$this->sTable.' WHERE id_pointeur=? AND date_pointage BETWEEN ? AND ? ',$user,$date1,$date2); }
Et, juste pour info, c'est ce que j'utilise d'habitude. C'était juste pour trouver ou est ce que se situait le problème que j'ai essayé les autres méthodes. Merci encore pour votre superbe travail!