Piou-piou / ribs-framework

core/modules/GestionModule.class.php

Braces +6 added lines, -12 removed lines

@@ -140,8 +140,7 @@ 
 				}
 
 				return $installer;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -160,8 +159,7 @@ 
 				foreach ($query as $obj) {
 					if ($obj->activer == 1) {
 						return true;
-					}
-					else {
+					} else {
 						return false;
 					}
 				}
@@ -183,8 +181,7 @@ 
 				foreach ($query as $obj) {
 					if ($obj->mettre_jour == 1) {
 						return false;
-					}
-					else {
+					} else {
 						return true;
 					}
 				}
@@ -208,8 +205,7 @@ 
 						//si pas de version a checker, cela veut dire qu'on vient d'installer le module
 						//du coup on met le next_check aa la semaine pro
 						$set_next = true;
-					}
-					else if (($obj->next_check_version <= $today) && ($obj->mettre_jour != 1)) {
+					} else if (($obj->next_check_version <= $today) && ($obj->mettre_jour != 1)) {
 						//avant tout on regarde si on doit delete une vieille version
 						if ($obj->delete_old_version == 1) {
 							$import = new ImportModule();
@@ -246,8 +242,7 @@ 
 								$dbc->prepare("UPDATE module SET mettre_jour=:update, online_version=:online_version WHERE ID_module=:id_module", $value);
 
 								$set_next = true;
-							}
-							else {
+							} else {
 								$set_next = true;
 							}
 						}
@@ -284,8 +279,7 @@ 
 				$this->setListeModuleMettreJour($nom_module, $version, $online_version);
 
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}

core/auth/Membre.class.php

Braces +8 added lines, -16 removed lines

@@ -87,12 +87,10 @@ 
 			if ((strlen($new_pseudo) < 5) || (strlen($new_pseudo) > 15)) {
 				$err = "Votre pseudo doit être entre 5 et 15 caractères";
 				$this->erreur = $err;
-			}
-			else if ($dbc->rechercherEgalite("identite", "pseudo", $new_pseudo) == false) {
+			} else if ($dbc->rechercherEgalite("identite", "pseudo", $new_pseudo) == false) {
 				$err = "Ce pseudo est déjà utilisé, veuillez en choisir un autre";
 				$this->erreur = $err;
-			}
-			else {
+			} else {
 				$dbc->query("UPDATE identite set pseudo=$new_pseudo WHERE ID_identite=".$_SESSION["idlogin".CLEF_SITE]);
 				$this->pseudo = $new_pseudo;
 			}
@@ -114,16 +112,13 @@ 
 			if (md5($old_mdp) != $mdp) {
 				$err = "Votre mot de passe est incorrect";
 				$this->erreur = $err;
-			}
-			else if ($new_mdp != $verif_new_mdp) {
+			} else if ($new_mdp != $verif_new_mdp) {
 				$err = "Vos mots de passe sont différents";
 				$this->erreur = $err;
-			}
-			else if ((strlen($new_mdp) < 5) || ($testmdp < 40)) {
+			} else if ((strlen($new_mdp) < 5) || ($testmdp < 40)) {
 				$err = "Votre mot de passe est trop simple";
 				$this->erreur = $err;
-			}
-			else {
+			} else {
 				$mdpok = Encrypt::setEncryptMdp($new_mdp, $this->id_identite);
 				//le nouveau mdp est bon on update
 				$dbc->query("UPDATE identite SET mdp='$mdpok' WHERE ID_identite=".$this->id_identite);
@@ -155,16 +150,13 @@ 
 				if ($lettre >= 'a' && $lettre <= 'z') {
 					$point = $point + 1;
 					$point_min = 1;
-				}
-				else if ($lettre >= 'A' && $lettre <= 'Z') {
+				} else if ($lettre >= 'A' && $lettre <= 'Z') {
 					$point = $point + 2;
 					$point_maj = 2;
-				}
-				else if ($lettre >= '0' && $lettre <= '9') {
+				} else if ($lettre >= '0' && $lettre <= '9') {
 					$point = $point + 3;
 					$point_chiffre = 3;
-				}
-				else {
+				} else {
 					$point = $point + 5;
 					$point_caracteres = 5;
 				}

core/auth/Inscription.class.php

Braces +11 added lines, -22 removed lines

@@ -20,8 +20,7 @@ 
 
 				if (isset($datas[$i][2])) {
 					$this->$function($datas[$i][1], $datas[$i][2]);
-				}
-				else {
+				} else {
 					$this->$function($datas[$i][1]);
 				}
 			}
@@ -54,8 +53,7 @@ 
 		protected function getTestRequired($value) {
 			if (($value == "") || ($value == null) || (strlen($value) == 0)) {
 				return false;
-			}
-			else {
+			} else {
 				return true;
 			}
 		}
@@ -68,8 +66,7 @@ 
 		protected function getTestLongueur($value, $longeur) {
 			if (strlen($value) > $longeur) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -81,8 +78,7 @@ 
 		protected function getTestInt($value) {
 			if (is_numeric($value)) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -100,17 +96,14 @@ 
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>$erreur</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
 					$this->$varaible = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
 					$this->erreur .= "<li>$erreur_long</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -158,8 +151,7 @@ 
 		protected function setVerifRetapeMdp($value) {
 			if ($this->mdp == $value) {
 				return true;
-			}
-			else {
+			} else {
 				$this->erreur .= "<li>Les mots de passent ne correspondent pas</li>";
 				$this->mdp = null;
 				return false;
@@ -187,17 +179,14 @@ 
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>Le champs E-mail ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) != false)) {
 					$this->mail = $value;
 					return true;
-				}
-				else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
+				} else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
 					$this->erreur .= "<li>Le champs E-mail doit être une adresse E-mail valide</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}

core/functions/DateHeure.class.php

Doc Comments -1 removed lines

@@ -77,7 +77,6 @@
 		/**
 		 * affiche la date du jour au format jeudi 12 janvier
 		 * @param integer $date si NULL on prend la date du jour sinon on prend la date qui est mise
-		 * @param int $abreger si NOT NULL, on abrege la date, on enleve la semaine et on coupe le mois à 3 lettres
 		 * @return string
 		 */
 		public static function date_fr_texte($date = 0) {

Braces +8 added lines, -16 removed lines

@@ -28,8 +28,7 @@ 
 				$minutesec = $minute * 60;
 				$heureseconde = $heuresec + $minutesec;
 				return $heureseconde;
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("La/les valeurs entrée n'est/ne sont pas de type int");
 				FlashMessage::getFlash();
 				die();
@@ -48,8 +47,7 @@ 
 
 				$temps = $heure."h".$minute."m".$seconde;
 				return $temps;
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("La valeur entrée n'est pas de type int");
 				FlashMessage::getFlash();
 				die();
@@ -66,8 +64,7 @@ 
 				$chaine = str_replace(":", "h", $temps);
 
 				return $chaine;
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("La valeur entrée n'est pas de type h:m");
 				FlashMessage::getFlash();
 				die();
@@ -86,8 +83,7 @@ 
 
 			if ($date == 0) {
 				return $jours[date("w")]." ".date("j").(date("j") == 1 ? "er" : " ").$mois[date("n") - 1]." ".date("Y");
-			}
-			else if ((strpos($date, "-") > 0) || (strpos($date, "/") > 0)) {
+			} else if ((strpos($date, "-") > 0) || (strpos($date, "/") > 0)) {
 				$pos = strpos($date, "-");
 
 				//si $pos > 0 cela veut dire qu'on est en YYYY-mm-jj
@@ -96,8 +92,7 @@ 
 					$jour_d = $explode[2];
 					$mois_d = $explode[1];
 					$annee_d = $explode[0];
-				}
-				else {
+				} else {
 					$explode = explode("/", $date);
 					$jour_d = $explode[0];
 					$mois_d = $explode[1];
@@ -108,8 +103,7 @@ 
 
 				return $jour_semaine." ".$jour_d." ".$mois[$mois_d - 1]." ".$annee_d;
 
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("Format de date passé en paramètre ne correspond pas à YYYY-mm-jj ou jj/mm/YYYY");
 				FlashMessage::getFlash();
 				die();
@@ -135,8 +129,7 @@ 
 				self::$annee = $annee;
 
 				return $jour."/".$mois."/".$annee;
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("format de date passé en paramètre ne correspond pas à YYYY-mm-jj");
 				FlashMessage::getFlash();
 				die();
@@ -158,8 +151,7 @@ 
 				$annee = $explode[2];
 
 				return $annee."-".$mois."-".$jour;
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("Format de date passé en paramètre ne correspond pas à jj/mm/aaaa");
 				FlashMessage::getFlash();
 				die();

core/Database.class.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php

 	namespace core;

 

-	use core\HTML\flashmessage\FlashMessage;

+	use core\HTML\flashmessage\FlashMessage;
 	use PDO;

 

 	class Database {

Braces +8 added lines, -11 removed lines

@@ -53,8 +53,7 @@ 
 			if ($query) {

 				$obj = $query->fetchAll(PDO::FETCH_OBJ);

 				return $obj;

-			}

-			else {

+			} else {

 				FlashMessage::setFlash("Une erreur est survenue en executant cette requette : ".$req);

 			}

 		}

@@ -73,8 +72,7 @@ 
 						$err = true;

 					}

 				}

-			}

-			else {

+			} else {

 				if (!$query->execute($value)) {

 					$err = true;

 				}

@@ -96,8 +94,7 @@ 
 		public function rechercherEgalite($table, $champ, $value, $id_table = null, $id = null) {

 			if ($id == null) {

 				$query = $this->getPdo()->query("SELECT COUNT($champ) as nb FROM $table WHERE $champ LIKE '$value'");

-			}

-			else {

+			} else {

 				$query = $this->getPdo()->query("SELECT COUNT($champ) as nb FROM $table WHERE $champ LIKE '$value' AND $id_table != $id");

 			}

 

@@ -106,9 +103,10 @@ 
 					$nb = $obj["nb"];

 				}

 

-				if ((isset($nb)) && ($nb != 0)) return true;

-			}

-			else {

+				if ((isset($nb)) && ($nb != 0)) {
+					return true;
+				}
+			} else {

 				return false;

 			}

 		}

@@ -125,8 +123,7 @@ 
 

 			if ($query->rowCount() > 0) {

 				return true;

-			}

-			else {

+			} else {

 				return false;

 			}

 		}

core/modules/RouterModule.class.php

Braces +2 added lines, -4 removed lines

@@ -59,12 +59,10 @@
 				if (in_array($explode[$i], $this->getAllModules())) {
 					$this->module = $explode[$i];
 					$debut_url = $explode[$i];
-				}
-				else if ($i == 1) {
+				} else if ($i == 1) {
 					$centre_url = $explode[$i];
 					$this->page = $explode[$i];
-				}
-				else {
+				} else {
 					$this->parametre = $explode[$i];
 				}
 			}

core/Querybuilder.class.php

Spacing +8 added lines, -8 removed lines

@@ -248,7 +248,7 @@ 
 		 * pour intialiser la ou les clauses where d'une requete

 		 */

 		public function where($champ, $cond, $champ_test, $closure = null) {

-			if ( $closure === null) {

+			if ($closure === null) {

 				$this->conditions[] = $champ.$cond.$champ_test;

 			}

 			else {

@@ -264,10 +264,10 @@ 
 		 * fonction qui permet de récupérer un select fait sur une table

 		 */

 		public function get() {

-			$requete = $this->req_beginning . implode(",", $this->champs) . " FROM " . implode(",", $this->table);

+			$requete = $this->req_beginning.implode(",", $this->champs)." FROM ".implode(",", $this->table);

 

 			if (!empty($this->conditions)) {

-				$requete .= " WHERE ". implode(" ", $this->conditions);

+				$requete .= " WHERE ".implode(" ", $this->conditions);

 			}

 

 			$this->unsetQueryBuilder();

@@ -282,15 +282,15 @@ 
 

 			$datas = [];

 			$count = count($this->champs);

-			for ($i=0 ; $i<$count ; $i++) {

+			for ($i = 0; $i < $count; $i++) {

 				$datas[] = $this->champs[$i]."=:".$this->champs[$i];

 			}

 

 			//si on a des conditions alors on sera dans un insert

-			$requete = $this->req_beginning . implode(",", $this->table) . " SET " . implode(", ", $datas);

+			$requete = $this->req_beginning.implode(",", $this->table)." SET ".implode(", ", $datas);

 

 			if (!empty($this->conditions)) {

-				$requete .= " WHERE " . implode(" ", $this->conditions);

+				$requete .= " WHERE ".implode(" ", $this->conditions);

 			}

 

 			$this->prepare($requete, $values);

@@ -301,10 +301,10 @@ 
 		 * fonction utilisée pour finir un delete

 		 */

 		public function del() {

-			$requete = $this->req_beginning . implode(",", $this->table);

+			$requete = $this->req_beginning.implode(",", $this->table);

 

 			if (!empty($this->conditions)) {

-				$requete .= " WHERE " . implode(" ", $this->conditions);

+				$requete .= " WHERE ".implode(" ", $this->conditions);

 			}

 

 			$this->query($requete);

Braces +1 added lines, -2 removed lines

@@ -110,8 +110,7 @@
 		public function where($champ, $cond, $champ_test, $closure = null) {

 			if ( $closure === null) {

 				$this->conditions[] = $champ.$cond.$champ_test;

-			}

-			else {

+			} else {

 				$this->conditions[] = $champ.$cond.$champ_test." ".$closure;

 			}