Correction REPO user

This commit is contained in:
xdrm-brackets 2015-11-11 22:02:13 +01:00
parent db6fa9ff3e
commit f33351abb1
2 changed files with 15 additions and 13 deletions

View File

@ -125,7 +125,7 @@ class DataBase{
* *
*/ */
public static function studentSemestre($etudiant, $semestre_pair, $annee){ public static function studentSemestre($etudiant, $semestre_pair, $annee){
if( $semestre = semestreRepo::studentCurrent($etudiant, $semestre_pair, $annee) ) // si on a un résultat if( $semestre = semestreRepo::forStudent($etudiant, $semestre_pair, $annee) ) // si on a un résultat
return $semestre; return $semestre;
else else
return 'error'; return 'error';
@ -630,7 +630,9 @@ class DataBase{
/**************************************************/ /**************************************************/
public function getModulesByUEByEnseignant($enseignant, $semestre_pair, $annee){ public function getModulesByUEByEnseignant($enseignant, $semestre_pair, $annee){
// on vérifie que l'enseignant enseigne bien cette année // on vérifie que l'enseignant enseigne bien cette année
if( !($enseignantUID=userRepo::UID($enseignant, null, $annee, $semestre_pair)) ) return 'unknown_teacher'; echo 'a';
if( !($enseignantUID=userRepo::UID($enseignant, null, $semestre_pair, $annee)) ) return 'unknown_teacher';
echo 'b';
$UEList = $this->getUEsEnseignant($enseignant, $semestre_pair, $annee); // on récupère la liste des UEs $UEList = $this->getUEsEnseignant($enseignant, $semestre_pair, $annee); // on récupère la liste des UEs
@ -668,12 +670,11 @@ class DataBase{
/*** retourne les UEs d'un enseignant ***/ /*** retourne les UEs d'un enseignant ***/
/****************************************/ /****************************************/
public function getUEsEnseignant($enseignant, $semestre_pair, $annee){ public function getUEsEnseignant($enseignant, $semestre_pair, $annee){
// on vérifie que l'enseignant enseigne bien cette année // on vérifie que l'enseignant enseigne bien cette année
if( !($enseignantUID=userRepo::UID($enseignant, null, $annee, $semestre_pair)) ) return 'unknown_teacher'; if( !($enseignantUID=userRepo::UID($enseignant, null, $semestre_pair, $annee)) ) return 'unknown_teacher';
// on retourne la liste des UEs en supprimant les doublons à indices numériques // on retourne la liste des UEs en supprimant les doublons à indices numériques
return ueRepo::forTeacher($enseignantUID, $annee); return ueRepo::forTeacher($enseignantUID, $semestre_pair, $annee);
} }

View File

@ -16,15 +16,16 @@ class userRepo extends DBAccess{
* *
* @semestre<int> ***OPTIONNEL*** l'UID du semestre auquel on veut vérifier que l'étudiant est inscrit * @semestre<int> ***OPTIONNEL*** l'UID du semestre auquel on veut vérifier que l'étudiant est inscrit
* *
* @annee<int> ***OPTIONNEL*** l'année auquelle on veut vérifier si l'enseignant enseigne
* @semestre_pair<Boolean> ***OPTIONNEL*** détermine si le semestre est pair ou non * @semestre_pair<Boolean> ***OPTIONNEL*** détermine si le semestre est pair ou non
* @annee<int> ***OPTIONNEL*** l'année auquelle on veut vérifier si l'enseignant enseigne
* *
* @return UID<String> si l'utilisateur est dans la BDD, retourne son UID * @return UID<String> si l'utilisateur est dans la BDD, retourne son UID
* @return FALSE<Boolean> FAUX si l'utilisateur n'est pas présent dans la BDD * @return FALSE<Boolean> FAUX si l'utilisateur n'est pas présent dans la BDD
* *
*/ */
public static function UID($identifiant, $semestre=null, $annee=null, $semestre_pair=null){ public static function UID($identifiant, $semestre=null, $semestre_pair=null, $annee=null){
if( $semestre == null && $annee == null && $semestre_pair == null ){ // on cherche un utilisateur avec cet identifiant
if( $semestre == null && $semestre_pair == null && $annee == null ){ // on cherche un utilisateur avec cet identifiant
/* [1] Cas on cherche juste si l'utilisateur existe /* [1] Cas on cherche juste si l'utilisateur existe
=============================================================*/ =============================================================*/
@ -42,9 +43,8 @@ class userRepo extends DBAccess{
"AND app.id_semestre = :semestre"); "AND app.id_semestre = :semestre");
$getUtilisateurUID->execute(array( ':identifiant' => $identifiant, ':semestre' => $semestre )); $getUtilisateurUID->execute(array( ':identifiant' => $identifiant, ':semestre' => $semestre ));
}elseif( $annee != null && $semestre_pair != null ){ }elseif( is_bool($semestre_pair) && $annee != null ){
$semestre_pair = ($semestre_pair) ? '0' : '1';
/* [2] Cas on cherche si un enseignant enseigne l'année donnée /* [2] Cas on cherche si un enseignant enseigne l'année donnée
==============================================================*/ ==============================================================*/
@ -54,10 +54,11 @@ class userRepo extends DBAccess{
"AND ens.id_mcc_module = mcc_m.id_mcc_module ". "AND ens.id_mcc_module = mcc_m.id_mcc_module ".
"AND mcc_m.id_mcc_ue = mcc_ue.id_mcc_ue ". "AND mcc_m.id_mcc_ue = mcc_ue.id_mcc_ue ".
"AND mcc_ue.id_semestre = s.id_semestre ". "AND mcc_ue.id_semestre = s.id_semestre ".
"AND s.annee = :annee ".
"AND u.identifiant = :identifiant ".
"AND s.rang % 2 = :semestre_pair ". "AND s.rang % 2 = :semestre_pair ".
"AND u.identifiant = :identifiant"); "AND s.annee = :annee");
$getUtilisateurUID->execute(array( ':identifiant' => $identifiant, ':semestre_pair' => $semestre_pair, ':annee' => $annee )); $getUtilisateurUID->execute(array( ':identifiant' => $identifiant, ':semestre_pair' => ($semestre_pair) ? '0' : '1', ':annee' => $annee ));
}else // si les paramètres sont pas bons }else // si les paramètres sont pas bons
return false; return false;