diff --git a/App/Helpers/Formatter.php b/App/Helpers/Formatter.php index f1c001714..63bacb954 100644 --- a/App/Helpers/Formatter.php +++ b/App/Helpers/Formatter.php @@ -104,7 +104,7 @@ public static function timestamp2Duree($timestamp) $heures = abs($timestamp/3600); $duree = sprintf('%02d:%02d', $heures, $secondes); - return (0 < $timestamp) + return (0 <= $timestamp) ? $duree : '-' . $duree ; diff --git a/App/ProtoControllers/Employe/AHeure.php b/App/ProtoControllers/Employe/AHeure.php index f5a04c218..8ed886bff 100644 --- a/App/ProtoControllers/Employe/AHeure.php +++ b/App/ProtoControllers/Employe/AHeure.php @@ -311,7 +311,9 @@ protected function transformChampsRecherche(array $post) $champs['timestampDebut'] = \utilisateur\Fonctions::getTimestampPremierJourAnnee($value); $champs['timestampFin'] = \utilisateur\Fonctions::getTimestampDernierJourAnnee($value); } else { - $champs[$key] = (int) $value; + if ($value !== "all") { // si la valeur est différent de "all" le paramètres est passé au champ pour la futur requête sql + $champs[$key] = (int) $value; + } } } diff --git a/App/ProtoControllers/Employe/Conge.php b/App/ProtoControllers/Employe/Conge.php index c3c81cb5e..1fbd328a7 100644 --- a/App/ProtoControllers/Employe/Conge.php +++ b/App/ProtoControllers/Employe/Conge.php @@ -21,14 +21,14 @@ class Conge public function getListe() { $return = ''; - $errorsLst=[]; - if($_SESSION['config']['where_to_find_user_email']=="ldap"){ - include_once CONFIG_PATH .'config_ldap.php'; + $errorsLst = []; + if ($_SESSION['config']['where_to_find_user_email'] == "ldap") { + include_once CONFIG_PATH . 'config_ldap.php'; } - if(!empty($_POST) && !$this->isSearch($_POST)) { + if (!empty($_POST) && !$this->isSearch($_POST)) { if (0 < (int) \utilisateur\Fonctions::postDemandeCongesHeure($_POST, $errorsLst)) { - $return .= '
'._('suppr_succes').'
'; + $return .= '
' . _('suppr_succes') . '
'; } } // on initialise le tableau global des jours fériés s'il ne l'est pas déjà : @@ -38,20 +38,15 @@ public function getListe() if (!empty($_POST) && $this->isSearch($_POST)) { $champsRecherche = $_POST['search']; - $champsSql = $this->transformChampsRecherche($_POST); + $champsSql = $this->transformChampsRecherche($_POST); } else { - $champsRecherche = [ - 'type' => 'cp', - ]; - $champsSql = []; + $champsRecherche = []; + $champsSql = []; } $params = $champsSql + [ 'p_login' => $_SESSION['userlogin'], - 'type' => 'cp', - 'p_etat' => 'demande', - ]; // champs par défaut écrasés par postés - - $return.= $this->getFormulaireRecherche($champsRecherche); + ]; // champs par défaut écrasés par posté + $return .= $this->getFormulaireRecherche($champsRecherche); $table = new \App\Libraries\Structure\Table(); $table->addClasses([ @@ -61,14 +56,14 @@ public function getListe() 'table-condensed', 'table-striped', ]); - $childTable = '' . _('divers_debut_maj_1') . ''. _('divers_fin_maj_1') .''. _('divers_type_maj_1') .''. _('divers_nb_jours_pris_maj_1') .'Statut'; + $childTable = '' . _('divers_debut_maj_1') . '' . _('divers_fin_maj_1') . '' . _('divers_type_maj_1') . '' . _('divers_nb_jours_pris_maj_1') . 'Statut'; $childTable .= ''; $childTable .= ''; $listId = $this->getListeId($params); $session = session_id(); if (empty($listId)) { $colonnes = 8; - $childTable .= '
' . _('aucun_resultat') .'
'; + $childTable .= '
' . _('aucun_resultat') . '
'; } else { $i = true; $listeConges = $this->getListeSQL($listId); @@ -83,7 +78,7 @@ public function getListe() list($date, $heure) = explode(' ', $conges["p_date_demande"]); $dateDemande = '(' . \App\Helpers\Formatter::dateIso2Fr($date) . ' ' . $heure . ') '; } - if(null != $conges["p_date_traitement"]) { + if (null != $conges["p_date_traitement"]) { list($date, $heure) = explode(' ', $conges["p_date_traitement"]); $dateReponse = '(' . \App\Helpers\Formatter::dateIso2Fr($date) . ' ' . $heure . ') '; } @@ -101,38 +96,39 @@ public function getListe() $messageReponse = ''; } - $demi_j_deb = ($conges["p_demi_jour_deb"]=="am") ? 'matin' : 'après-midi'; + $demi_j_deb = ($conges["p_demi_jour_deb"] == "am") ? 'matin' : 'après-midi'; + $demi_j_fin = ($conges["p_demi_jour_fin"] == "am") ? 'matin' : 'après-midi'; - $demi_j_fin = ($conges["p_demi_jour_fin"] =="am") ? 'matin' : 'après-midi'; + $childTable .= ''; - $childTable .= ''; $childTable .= '' . \App\Helpers\Formatter::dateIso2Fr($conges["p_date_deb"]) . ' ' . schars($demi_j_deb) . ''; - $childTable .= '' . \App\Helpers\Formatter::dateIso2Fr($conges["p_date_fin"]) . ' ' . schars($demi_j_fin) . '' ; - $childTable .= ''.schars($conges["ta_libelle"]).'' ; - $childTable .= ''.affiche_decimal($conges["p_nb_jours"]).'' ; + $childTable .= '' . \App\Helpers\Formatter::dateIso2Fr($conges["p_date_fin"]) . ' ' . schars($demi_j_fin) . ''; + $childTable .= '' . schars($conges["ta_libelle"]) . ''; + $childTable .= '' . affiche_decimal($conges["p_nb_jours"]) . ''; $childTable .= '' . \App\Models\Conge::statusText($conges["p_etat"]) . ''; $childTable .= ''; if (!empty($messageDemande) || !empty($messageReponse)) { $childTable .= ''; } - $childTable .= '' ; + $childTable .= ''; $childTable .= ''; $user_modif_demande = ''; $user_suppr_demande = ''; // si on peut modifier une demande on defini le lien à afficher - if($conges["p_etat"] == \App\Models\Conge::STATUT_DEMANDE) { - if(!$interdictionModification){ - $user_modif_demande = '' ; + if ($conges["p_etat"] == \App\Models\Conge::STATUT_DEMANDE) { + if (!$interdictionModification) { + $user_modif_demande = ''; } $user_suppr_demande = ''; } - - if(!$interdictionModification) { + + if (!$interdictionModification) { $childTable .= $user_modif_demande . '      '; + } - $childTable .= ($user_suppr_demande) . ''."\n" ; + $childTable .= ($user_suppr_demande) . '' . "\n"; $childTable .= ''; $i = !$i; } @@ -171,6 +167,7 @@ protected function getFormulaireRecherche(array $champs) $form = ''; $form = ''; + $form .= '
 Reset
'; return $form; } @@ -212,9 +210,12 @@ protected function transformChampsRecherche(array $post) foreach ($search as $key => $value) { if ('annee' === $key) { $champs['dateDebut'] = ((int) $value) . '-01-01'; - $champs['dateFin'] = ((int) $value) . '-12-31'; + $champs['dateFin'] = ((int) $value) . '-12-31'; } else { - $champs[$key] = $value; + if ($value !== "all") { + // si la valeur est différent de tout le paramètres est passé au champ pour la futur requête sql + $champs[$key] = $value; + } } } @@ -225,13 +226,13 @@ protected function transformChampsRecherche(array $post) * SQL */ - /** - * Retourne une liste d'id de congés - * - * @param array $params Paramètres de recherche - * - * @return array - */ + /** + * Retourne une liste d'id de congés + * + * @param array $params Paramètres de recherche + * + * @return array + */ protected function getListeId(array $params) { $sql = \includes\SQL::singleton(); @@ -294,14 +295,15 @@ protected function getListeSQL(array $listId) * Retourne les demandes d'un employé * */ - public static function getIdDemandesUtilisateur($user) { + public static function getIdDemandesUtilisateur($user) + { $ids = []; $sql = \includes\SQL::singleton(); $req = 'SELECT p_num AS id FROM conges_periode WHERE p_login = \'' . $sql->quote($user) . '\' - AND p_etat = \''. \App\Models\Conge::STATUT_DEMANDE.'\''; + AND p_etat = \'' . \App\Models\Conge::STATUT_DEMANDE . '\''; $res = $sql->query($req); while ($data = $res->fetch_array()) { $ids[] = (int) $data['id']; @@ -310,7 +312,6 @@ public static function getIdDemandesUtilisateur($user) { return $ids; } - /** * Vérifie l'existence de congé basée sur les critères fournis * @@ -351,7 +352,7 @@ public function exists(array $params) public function isChevauchement($user, $dateDebut, $typeCreneauDebut, $dateFin, $typeCreneauFin) { return $this->isChevauchementHeuresRepos($user, $dateDebut, $typeCreneauDebut, $dateFin, $typeCreneauFin) - || $this->isChevauchementHeuresAdditionnelles($user, $dateDebut, $typeCreneauDebut, $dateFin, $typeCreneauFin); + || $this->isChevauchementHeuresAdditionnelles($user, $dateDebut, $typeCreneauDebut, $dateFin, $typeCreneauFin); } /** @@ -402,14 +403,12 @@ private function isChevauchementHeures($user, $dateDebut, $typeCreneauDebut, $da { $sql = \includes\SQL::singleton(); $filtresDates[] = '(dateDebutHeure > "' . $dateDebut . '" AND dateDebutHeure < "' . $dateFin . '")'; - if (Creneau::TYPE_PERIODE_MATIN_APRES_MIDI - === $typeCreneauDebut) { + if (Creneau::TYPE_PERIODE_MATIN_APRES_MIDI === $typeCreneauDebut) { $filtresDates[] = '(dateDebutHeure = "' . $dateDebut . '")'; } else { $filtresDates[] = '(dateDebutHeure = "' . $dateDebut . '" AND type_periode IN (' . $typeCreneauDebut . ',' . Creneau::TYPE_PERIODE_MATIN_APRES_MIDI . '))'; } - if (Creneau::TYPE_PERIODE_MATIN_APRES_MIDI - === $typeCreneauFin) { + if (Creneau::TYPE_PERIODE_MATIN_APRES_MIDI === $typeCreneauFin) { $filtresDates[] = '(dateDebutHeure = "' . $dateFin . '")'; } else { $filtresDates[] = '(dateDebutHeure = "' . $dateFin . '" AND type_periode IN (' . $typeCreneauFin . ',' . Creneau::TYPE_PERIODE_MATIN_APRES_MIDI . '))'; @@ -424,7 +423,7 @@ private function isChevauchementHeures($user, $dateDebut, $typeCreneauDebut, $da SELECT * FROM (SELECT *, DATE_FORMAT(FROM_UNIXTIME(debut), "%Y-%m-%d") AS dateDebutHeure - FROM ' . $typeHeure . ') tmp + FROM ' . $typeHeure . ') tmp WHERE statut IN ("' . implode('","', $etats) . '") AND login = "' . $sql->quote($user) . '" AND (' . implode(' OR ', $filtresDates) . ') diff --git a/App/ProtoControllers/Employe/Heure/Additionnelle.php b/App/ProtoControllers/Employe/Heure/Additionnelle.php index 2d149fa9d..59d47dcdc 100644 --- a/App/ProtoControllers/Employe/Heure/Additionnelle.php +++ b/App/ProtoControllers/Employe/Heure/Additionnelle.php @@ -273,7 +273,7 @@ public function getListe() } $champsRecherche = (!empty($_POST) && $this->isSearch($_POST)) ? $this->transformChampsRecherche($_POST) - : ['statut' => AHeure::STATUT_DEMANDE]; + : []; $params = $champsRecherche + [ 'login' => $_SESSION['userlogin'], ]; @@ -329,6 +329,7 @@ public function getListe() protected function getFormulaireRecherche(array $champs) { $form = '