Discussion :

[2boubacar10]

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

Dans mon model_pointage.php j'ai ça

Code :

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

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 :

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

'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 :

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

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 :

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

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);

[imikado]

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

dans le fichier conf/site.ini.php

Code :

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

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 php :

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

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 php :

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

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 php :

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

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);
 
 
 
	}

[2boubacar10]

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

[2boubacar10]

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 php :

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

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!