Piou-piou / ribs-framework

core/admin/Admin.class.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 	namespace core\admin;
 
 	use core\App;
-	use core\auth\Connexion;
 	use core\auth\Encrypt;
 	use core\auth\Membre;
 	use core\Configuration;

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
 		 * Pour récupérer la liste de tous les users afin d'activer un compte ou modifier des trucs dessus
 		 * si archiver == null on récupère les utilisateurs actifs sur le site sinon on récupere les utilisateurs archives
 		 */
-		public function getAllUser($archiver=null) {
+		public function getAllUser($archiver = null) {
 			$dbc = \core\App::getDb();
 			$this->setAllUser(null, null, null, null, null, null, null);
 
@@ -229,7 +229,7 @@ 
 
 			if ($oldimg_profil != "") {
 				$oldimg_profil = explode("/", $oldimg_profil);
-				if($oldimg_profil[7] != "defaut.png") {
+				if ($oldimg_profil[7] != "defaut.png") {
 					unlink("../../images/profil/".$oldimg_profil[7]);
 				}
 			}

Braces +5 added lines, -10 removed lines

@@ -50,8 +50,7 @@ 
 
 			if ($archiver == null) {
 				$query = $dbc->query("SELECT * FROM identite WHERE archiver IS NULL AND ID_identite > 1");
-			}
-			else {
+			} else {
 				$query = $dbc->query("SELECT * FROM identite WHERE archiver IS NOT NULL AND ID_identite > 1");
 			}
 
@@ -67,20 +66,17 @@ 
 
 					if ($obj->img_profil == "") {
 						$img_profil[] = "profil/defaut.png";
-					}
-					else {
+					} else {
 						$img_profil[] = $obj->img_profil;
 					}
 
 					if ($config->getValiderInscription() == 1) {
 						if ($obj->valide == 0) {
 							$valide[] = "<a href=".ADMWEBROOT."controller/core/admin/comptes/valider_compte?id_identite=$obj->ID_identite>Valider cet utilisateur</a>";
-						}
-						else {
+						} else {
 							$valide[] = "Utilisateur validé";
 						}
-					}
-					else {
+					} else {
 						$valide = "";
 					}
 				}
@@ -177,8 +173,7 @@ 
 
 				$mail = new Mail($this->mail);
 				$mail->setEnvoyerMail("Réinitialisation de votre E-mail effectuée", "Votre mot de passe a été réinitialisé");
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("le mot de passe de $this->pseudo ne peu pas être réinitialisé car il ne possède pas d'E-mail");
 				$this->erreur = true;
 			}

core/admin/droitsacces/DroitAcces.class.php

Doc Comments +3 added lines, -3 removed lines

@@ -80,8 +80,8 @@ 
 		/**
 		 * pour savoir si en fonction des droits d'accès de l'utilisateur il peu ou non accéder à cete page
 		 * on passe outre les test si on est super admin
-		 * @param $page
-		 * @param null $id_page
+		 * @param string $page
+		 * @param string $page
 		 * @return bool
 		 */
 		public function getDroitAccesPage($page) {
@@ -143,7 +143,7 @@ 
 		 * si on est super admin on passe outre tous les tests
 		 * @param $droit
 		 * @param $id_page
-		 * @return bool
+		 * @return boolean|null
 		 */
 		public function getDroitAccesContenu($droit, $id_page) {
 			$dbc = \core\App::getDb();

Indentation +18 added lines, -18 removed lines

@@ -1,10 +1,10 @@ 
 <?php
-    namespace core\admin\droitsacces;
+	namespace core\admin\droitsacces;
 
 	use core\App;
 
 	class DroitAcces {
-    	private $logged;
+		private $logged;
 
 		//pour la table identite
 		private $id_identite;
@@ -21,10 +21,10 @@ 
 
 
 		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
-        public function __construct() {
+		public function __construct() {
 			$dbc = \core\App::getDb();
 
-            if (isset($_SESSION["idlogin".CLEF_SITE])) {
+			if (isset($_SESSION["idlogin".CLEF_SITE])) {
 				$this->logged = true;
 				$this->id_identite = $_SESSION["idlogin".CLEF_SITE];
 
@@ -44,36 +44,36 @@ 
 			else {
 				$this->logged = false;
 			}
-        }
-        //-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
+		}
+		//-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
     
     
     
-        //-------------------------- GETTER ----------------------------------------------------------------------------//
+		//-------------------------- GETTER ----------------------------------------------------------------------------//
 		public function getLogged(){
-		    return $this->logged;
+			return $this->logged;
 		}
 		public function getSuperAdmin(){
-		    return $this->super_admin;
+			return $this->super_admin;
 		}
 
 		//pour la table liste_droit_acces
 		public function getIdListeDroitAcces(){
-		    return $this->id_liste_droit_acces;
+			return $this->id_liste_droit_acces;
 		}
 
 		//pour les droits sur la gestion des contenus
 		public function getModifSeo(){
-		    return $this->modif_seo;
+			return $this->modif_seo;
 		}
 		public function getModifContenu(){
-		    return $this->modif_contenu;
+			return $this->modif_contenu;
 		}
 		public function getModifNavigation(){
-		    return $this->modif_navigation;
+			return $this->modif_navigation;
 		}
 		public function getSupprimerPage(){
-		    return $this->supprimer_page;
+			return $this->supprimer_page;
 		}
 
 		//autres getter
@@ -240,11 +240,11 @@ 
 				return true;
 			}
 		}
-        //-------------------------- FIN GETTER ----------------------------------------------------------------------------//
+		//-------------------------- FIN GETTER ----------------------------------------------------------------------------//
     
     
     
-        //-------------------------- SETTER ----------------------------------------------------------------------------//
+		//-------------------------- SETTER ----------------------------------------------------------------------------//
 
-        //-------------------------- FIN SETTER ----------------------------------------------------------------------------//
-    }
\ No newline at end of file
+		//-------------------------- FIN SETTER ----------------------------------------------------------------------------//
+	}
\ No newline at end of file

Spacing +7 added lines, -7 removed lines

@@ -50,29 +50,29 @@
     
     
         //-------------------------- GETTER ----------------------------------------------------------------------------//
-		public function getLogged(){
+		public function getLogged() {
 		    return $this->logged;
 		}
-		public function getSuperAdmin(){
+		public function getSuperAdmin() {
 		    return $this->super_admin;
 		}
 
 		//pour la table liste_droit_acces
-		public function getIdListeDroitAcces(){
+		public function getIdListeDroitAcces() {
 		    return $this->id_liste_droit_acces;
 		}
 
 		//pour les droits sur la gestion des contenus
-		public function getModifSeo(){
+		public function getModifSeo() {
 		    return $this->modif_seo;
 		}
-		public function getModifContenu(){
+		public function getModifContenu() {
 		    return $this->modif_contenu;
 		}
-		public function getModifNavigation(){
+		public function getModifNavigation() {
 		    return $this->modif_navigation;
 		}
-		public function getSupprimerPage(){
+		public function getSupprimerPage() {
 		    return $this->supprimer_page;
 		}
 

Braces +31 added lines, -37 removed lines

@@ -31,17 +31,17 @@ 
 				//on test voir si super admin
 				$query = $dbc->query("SELECT super_admin,liste_droit FROM identite WHERE ID_identite=".$this->id_identite);
 				foreach ($query as $obj) {
-					if ($obj->super_admin == 1) $this->super_admin = 1;
+					if ($obj->super_admin == 1) {
+						$this->super_admin = 1;
+					}
 
 					if ($obj->liste_droit == 1) {
 						$this->id_liste_droit_acces = $obj->liste_droit;
-					}
-					else {
+					} else {
 						$this->id_liste_droit_acces = 0;
 					}
 				}
-			}
-			else {
+			} else {
 				$this->logged = false;
 			}
         }
@@ -93,14 +93,15 @@ 
 
 				if (!in_array($page, $all_access)) {
 					$query = $dbc->query("SELECT droit_acces FROM droit_acces WHERE page LIKE '%$page%'");
-					foreach ($query as $obj) $droit_acces = $obj->droit_acces;
+					foreach ($query as $obj) {
+						$droit_acces = $obj->droit_acces;
+					}
 				}
 
 				//si la page n'est pas trouvée dans les droit d'accès c'est qu'elle est obligatoirement accessible dans admin
 				if (!isset($droit_acces)) {
 					$acces = true;
-				}
-				else {
+				} else {
 					$acces = false;
 				}
 
@@ -112,25 +113,23 @@ 
 				");
 
 				if (count($query) > 0) {
-					foreach ($query as $obj) $liste_droit_acces[] = $obj->droit_acces;
+					foreach ($query as $obj) {
+						$liste_droit_acces[] = $obj->droit_acces;
+					}
 
 					if (($this->super_admin == 1) || ($acces == true) || (in_array($droit_acces, $liste_droit_acces)) || (($page == "") || ($page == null))) {
 						return true;
-					}
-					else {
+					} else {
 						return false;
 					}
-				}
-				else {
+				} else {
 					if ($acces == true) {
 						return true;
-					}
-					else {
+					} else {
 						return false;
 					}
 				}
-			}
-			else {
+			} else {
 				return true;
 			}
 		}
@@ -155,7 +154,9 @@ 
 								liste_droit_acces.ID_liste_droit_acces = liaison_liste_droit.ID_liste_droit_acces AND
 								liste_droit_acces.ID_liste_droit_acces = $this->id_liste_droit_acces
 				");
-				foreach ($query as $obj) $liste_droit_acces[] = $obj->droit_acces;
+				foreach ($query as $obj) {
+					$liste_droit_acces[] = $obj->droit_acces;
+				}
 
 				if (in_array($droit, $liste_droit_acces)) {
 					//on check si il a le droit de modifier ou supprimer cette page
@@ -178,27 +179,21 @@ 
 							//si les trois sont différent de 0 on renvoit true soinon false
 							if (($this->modif_seo != 0) || ($this->modif_contenu != 0) || ($this->modif_navigation != 0) || ($this->supprimer_page != 0)) {
 								return true;
-							}
-							else {
+							} else {
 								return false;
 							}
-						}
-						else {
+						} else {
 							return false;
 						}
-					}
-					else if ($id_page == "creation_page") {
+					} else if ($id_page == "creation_page") {
 						$this->getDroitAccesPage("gestion-contenus/index");
-					}
-					else {
+					} else {
 						return false;
 					}
-				}
-				else {
+				} else {
 					return false;
 				}
-			}
-			else {
+			} else {
 				$this->modif_seo = 1;
 				$this->modif_contenu = 1;
 				$this->modif_navigation = 1;
@@ -223,20 +218,19 @@ 
 								liste_droit_acces.ID_liste_droit_acces = $this->id_liste_droit_acces
 				");
 				if (count($query) > 0) {
-					foreach ($query as $obj) $liste_droit_acces[] = $obj->droit_acces;
+					foreach ($query as $obj) {
+						$liste_droit_acces[] = $obj->droit_acces;
+					}
 
 					if (($this->super_admin == 1) || (in_array($droit_acces, $liste_droit_acces))) {
 						return true;
-					}
-					else {
+					} else {
 						return false;
 					}
-				}
-				else {
+				} else {
 					return false;
 				}
-			}
-			else {
+			} else {
 				return true;
 			}
 		}

core/auth/Encrypt.class.php

Doc Comments +5 added lines, -1 removed lines

@@ -13,6 +13,10 @@ 
 		
 		
 		//-------------------------- GETTER ----------------------------------------------------------------------------//
+
+		/**
+		 * @return string
+		 */
 		public static function getParams($id_identite=null) {
 			if ($id_identite != null) {
 				$dbc = App::getDb();
@@ -101,7 +105,7 @@ 
 
 		/**
 		 * sauvegarde les parametres pour retrouver le mot de passe dans la bdd
-		 * @param $params
+		 * @param string $params
 		 * @param $id_identite
 		 */
 		public static function setSaveParams($params, $id_identite) {

Spacing +7 added lines, -7 removed lines

@@ -13,7 +13,7 @@ 
 		
 		
 		//-------------------------- GETTER ----------------------------------------------------------------------------//
-		public static function getParams($id_identite=null) {
+		public static function getParams($id_identite = null) {
 			if ($id_identite != null) {
 				$dbc = App::getDb();
 
@@ -38,7 +38,7 @@ 
 		 * @param $id_identite
 		 * @return string
 		 */
-		public static function setEncryptMdp($mdp, $id_identite=null) {
+		public static function setEncryptMdp($mdp, $id_identite = null) {
 			$encrypt_str = ChaineCaractere::random(154);
 
 			//on cache le mdp
@@ -48,9 +48,9 @@ 
 			$longeur_mdp = strlen($mdp);
 
 			//on va couper le mot de passe en 2 suivant un nombre aleatoire
-			$nb_aleatoire_mdp = rand(3, $longeur_mdp-2);
-			$bout1_mdp = mb_substr($mdp, 0, $longeur_mdp/2, "UTF-8");
-			$bout2_mdp = mb_substr($mdp, $longeur_mdp/2, $longeur_mdp, "UTF-8");
+			$nb_aleatoire_mdp = rand(3, $longeur_mdp - 2);
+			$bout1_mdp = mb_substr($mdp, 0, $longeur_mdp / 2, "UTF-8");
+			$bout2_mdp = mb_substr($mdp, $longeur_mdp / 2, $longeur_mdp, "UTF-8");
 
 			//on stock la taille des bouts pour pouvoir les décrypter
 			$taille_bout1 = strlen($bout1_mdp);
@@ -59,7 +59,7 @@ 
 			//on insere le premier bout aleatoirement dans le hashmdp
 			//on calcul sa longeur total (celle duhash + la logneur du mdp que l'on va rajouter dedans)
 			$longueur_hash = strlen($encrypt_str);
-			$debut_bout1 = rand(0, $longueur_hash/2);
+			$debut_bout1 = rand(0, $longueur_hash / 2);
 
 			//on rajouter le premier bout dans le mot de passe + recalcule de la longeur du hash avec le mdp add
 			$encrypt_str = mb_substr($encrypt_str, 0, $debut_bout1).$bout1_mdp.mb_substr($encrypt_str, $debut_bout1, $longueur_hash);
@@ -67,7 +67,7 @@ 
 			//on insere le second bout aleatoirement dans le hashmdp
 			//on calcul sa longeur total (celle duhash + la logneur premier bout du mdp que l'on va rajouter dedans)
 			$longueur_hash = strlen($encrypt_str);
-			$debut_bout2 = rand($longueur_hash/2, $longueur_hash);
+			$debut_bout2 = rand($longueur_hash / 2, $longueur_hash);
 
 			//on rajoute le deuxieme
 			$mdp_encrypt = mb_substr($encrypt_str, 0, $debut_bout2).$bout2_mdp.mb_substr($encrypt_str, $debut_bout2, $longueur_hash);

Braces +2 added lines, -4 removed lines

@@ -21,8 +21,7 @@ 
 				foreach ($query as $obj) {
 					$params = $obj->mdp_params;
 				}
-			}
-			else {
+			} else {
 				$params = self::$params;
 			}
 
@@ -74,8 +73,7 @@ 
 
 			if ($id_identite != null) {
 				self::setSaveParams("$taille_bout1, $debut_bout2, $nb_aleatoire_mdp, $taille_bout2, $debut_bout1, ".ChaineCaractere::random(20), $id_identite);
-			}
-			else {
+			} else {
 				self::$params = "$taille_bout1, $debut_bout2, $nb_aleatoire_mdp, $taille_bout2, $debut_bout1, ".ChaineCaractere::random(20);
 			}
 

core/auth/Inscription.class.php

Doc Comments +3 added lines

@@ -55,6 +55,9 @@
 			}
 		}
 
+		/**
+		 * @param integer $longeur
+		 */
 		protected function getTestLongueur($value, $longeur) {
 			if (strlen($value) > $longeur) {
 				return true;

Spacing +8 added lines, -8 removed lines

@@ -14,7 +14,7 @@ 
 
 		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
 		public function __construct($datas) {
-			for ($i=0 ; $i<count($datas) ; $i++) {
+			for ($i = 0; $i < count($datas); $i++) {
 				$function = "setVerif".ucfirst($datas[$i][0]);
 
 				if (isset($datas[$i][2])) {
@@ -43,7 +43,7 @@ 
 			return $this->mail;
 		}
 		public function getErreur() {
-			return $this->erreur;;
+			return $this->erreur; ;
 		}
 
 		protected function getTestRequired($value) {
@@ -83,7 +83,7 @@ 
 		 * @param null $required
 		 * @return bool
 		 */
-		protected function setVerifNom($value, $required=null) {
+		protected function setVerifNom($value, $required = null) {
 			//test avec le required, si le champe est vide et que le required est != null on return fa	lse sinon on va tester
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs nom ne peut pas être vide</li>";
@@ -110,7 +110,7 @@ 
 		 * @param null $required
 		 * @return bool
 		 */
-		protected function setVerifPrenom($value, $required=null) {
+		protected function setVerifPrenom($value, $required = null) {
 			//test avec le required, si le champe est vide et que le required est != null on return fa	lse sinon on va tester
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs prénom ne peut pas être vide</li>";
@@ -137,7 +137,7 @@ 
 		 * @param null $required
 		 * @return bool
 		 */
-		protected function setVerifMdp($value, $required=null) {
+		protected function setVerifMdp($value, $required = null) {
 			//test avec le required, si le champe est vide et que le required est != null on return fa	lse sinon on va tester
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs mot de passe ne peut pas être vide</li>";
@@ -164,7 +164,7 @@ 
 		 * @param null $required
 		 * @return bool
 		 */
-		protected function setVerifRetapeMdp($value, $required=null) {
+		protected function setVerifRetapeMdp($value, $required = null) {
 			if ($this->mdp == $value) {
 				return true;
 			}
@@ -181,7 +181,7 @@ 
 		 * @param null $required
 		 * @return bool
 		 */
-		protected function setVerifPseudo($value, $required=null) {
+		protected function setVerifPseudo($value, $required = null) {
 			//test avec le required, si le champe est vide et que le required est != null on return fa	lse sinon on va tester
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs pseudo ne peut pas être vide</li>";
@@ -208,7 +208,7 @@ 
 		 * @param null $required
 		 * @return bool
 		 */
-		protected function setVerifMail($value, $required=null) {
+		protected function setVerifMail($value, $required = null) {
 			//test avec le required, si le champe est vide et que le required est != null on return fa	lse sinon on va tester
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs E-mail ne peut pas être vide</li>";

Braces +20 added lines, -40 removed lines

@@ -19,8 +19,7 @@ 
 
 				if (isset($datas[$i][2])) {
 					$this->$function($datas[$i][1], $datas[$i][2]);
-				}
-				else {
+				} else {
 					$this->$function($datas[$i][1]);
 				}
 			}
@@ -49,8 +48,7 @@ 
 		protected function getTestRequired($value) {
 			if (($value == "") || ($value == null) || (strlen($value) == 0)) {
 				return false;
-			}
-			else {
+			} else {
 				return true;
 			}
 		}
@@ -58,8 +56,7 @@ 
 		protected function getTestLongueur($value, $longeur) {
 			if (strlen($value) > $longeur) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -67,8 +64,7 @@ 
 		protected function getTestInt($value) {
 			if (is_numeric($value)) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -88,17 +84,14 @@ 
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs nom ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) == true)) {
 					$this->nom = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) == false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) == false)) {
 					$this->erreur .= "<li>Le champs nom soit être supérieur à deux caractères</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -115,17 +108,14 @@ 
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs prénom ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) == true)) {
 					$this->prenom = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) == false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) == false)) {
 					$this->erreur .= "<li>Le champs prénom soit être supérieur à deux caractères</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -142,17 +132,14 @@ 
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs mot de passe ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) == true)) {
 					$this->mdp = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) == false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) == false)) {
 					$this->erreur .= "<li>Le champs mot de passe soit être supérieur à deux caractères</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -167,8 +154,7 @@ 
 		protected function setVerifRetapeMdp($value, $required=null) {
 			if ($this->mdp == $value) {
 				return true;
-			}
-			else {
+			} else {
 				$this->erreur .= "<li>Les mots de passent ne correspondent pas</li>";
 				$this->mdp = null;
 				return false;
@@ -186,17 +172,14 @@ 
 			if (($required != null) && ($this->getTestRequired($value) == false)) {
 				$this->erreur .= "<li>Le champs pseudo ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) == true)) {
 					$this->pseudo = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) == false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) == false)) {
 					$this->erreur .= "<li>Le champs pseudo soit être supérieur à deux caractères</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -213,17 +196,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/auth/Membre.class.php

Doc Comments +6 added lines, -1 removed lines

@@ -136,6 +136,11 @@ 
 			}
 		}
 		
+		/**
+		 * @param string $old_mdp
+		 * @param string $new_mdp
+		 * @param string $verif_new_mdp
+		 */
 		public function setMdp($old_mdp, $new_mdp, $verif_new_mdp) {
 			$dbc = \core\App::getDb();
 
@@ -271,7 +276,7 @@ 
 		/**
 		 * Fonction  qui permet de verifier la securite d'un mdp
 		 * @param string $mdp
-		 * @return float
+		 * @return integer
 		 */
 		function testpassword($mdp)	{
 			$longueur = strlen($mdp);

Spacing +13 added lines, -13 removed lines

@@ -19,7 +19,7 @@ 
 		
 		//------------------------------ constructeur-----------------------------------
 		//Récupérer en base de données les infos du membre
-		public function __construct($id_identite=null){
+		public function __construct($id_identite = null) {
 			$dbc = \core\App::getDb();
 
 			$this->debut_lien = IMGROOT."profil/";
@@ -82,7 +82,7 @@ 
 		
 		
 		//------------------------------ setter-----------------------------------
-		public function setSupprimUser($id_identite=null) {
+		public function setSupprimUser($id_identite = null) {
 			$dbc = \core\App::getDb();
 
 			if ($id_identite == null) {
@@ -96,7 +96,7 @@ 
 			$oldimg_profil = $obj->img_profil;
 			if ($oldimg_profil != "") {
 				$oldimg_profil = explode("/", $oldimg_profil);
-				if(end($oldimg_profil) == "defaut.png") {
+				if (end($oldimg_profil) == "defaut.png") {
 					
 				}
 				else {
@@ -211,26 +211,26 @@ 
 				//recuperation info sur img
 				$infos_img = getimagesize($_FILES['image']['tmp_name']);
 				
-				if(substr($image, -3) == (!$extension || !$extension2 || !$extension3 || !$extension4)) {
+				if (substr($image, -3) == (!$extension || !$extension2 || !$extension3 || !$extension4)) {
 					$err = "Votre image ne comporte pas l'extension .jpg";
 					$this->erreur = $err;
 				}
-				else if(($infos_img[0] >= $width_max) && ($infos_img[1] >= $height_max) && ($_FILES['image']['size'] >= $max_size)) {
+				else if (($infos_img[0] >= $width_max) && ($infos_img[1] >= $height_max) && ($_FILES['image']['size'] >= $max_size)) {
 					$err = "Problème dans les dimensions ou taille de l\'image.";
 					$this->erreur = $err;
 				}
 				else {
 					$uniqid = uniqid();
 					$chemin_destination = '../images/profil/';
-					if(substr($image, -3) == ($extension)) {
+					if (substr($image, -3) == ($extension)) {
 						move_uploaded_file($_FILES['image']['tmp_name'], $chemin_destination.$uniqid.".jpg");
 						$imageok = $uniqid.".jpg";
 					}
-					else if(substr($image, -3) == ($extension2)) {
+					else if (substr($image, -3) == ($extension2)) {
 						move_uploaded_file($_FILES['image']['tmp_name'], $chemin_destination.$uniqid.".png");
 						$imageok = $uniqid.".png";
 					}
-					else if(substr($image, -3) == ($extension3)) {
+					else if (substr($image, -3) == ($extension3)) {
 						move_uploaded_file($_FILES['image']['tmp_name'], $chemin_destination.$uniqid.".jpeg");
 						$imageok = $uniqid.".jpeg";
 					}
@@ -256,7 +256,7 @@ 
 			if ($oldimg_profil != "") {
 				$oldimg_profil = explode("/", $oldimg_profil);
 				
-				if($oldimg_profil[7] == "defaut.png") {
+				if ($oldimg_profil[7] == "defaut.png") {
 				}
 				else {
 					unlink("../images/profil/".$oldimg_profil[7]);
@@ -273,22 +273,22 @@ 
 		 * @param string $mdp
 		 * @return float
 		 */
-		function testpassword($mdp)	{
+		function testpassword($mdp) {
 			$longueur = strlen($mdp);
 			$point = 0;
 
-			for ($i=0 ; $i<$longueur ; $i++) 	{
+			for ($i = 0; $i < $longueur; $i++) {
 				$lettre = $mdp[$i];
 
 				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;
 				}

Braces +19 added lines, -38 removed lines

@@ -38,8 +38,7 @@ 
 
 					if ($obj->img_profil == "") {
 						$this->img = $this->debut_lien."defaut.png";
-					}
-					else {
+					} else {
 						$this->img = $obj->img_profil;
 					}
 				}
@@ -98,8 +97,7 @@ 
 				$oldimg_profil = explode("/", $oldimg_profil);
 				if(end($oldimg_profil) == "defaut.png") {
 					
-				}
-				else {
+				} else {
 					unlink("../../images/profil/".end($oldimg_profil));
 					
 				}
@@ -121,16 +119,13 @@ 
 			if (strlen($new_pseudo) < 5) {
 				$err = "Votre pseudo est trop court";
 				$this->erreur = $err;
-			}
-			else if (strlen($new_pseudo) > 15) {
+			} else if (strlen($new_pseudo) > 15) {
 				$err = "Votre pseudo est trop long";
 				$this->erreur = $err;
-			}
-			else if ($new_pseudo == $pseudo_bdd) {
+			} else if ($new_pseudo == $pseudo_bdd) {
 				$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;
 			}
@@ -146,24 +141,20 @@ 
 			if (md5($old_mdp) != $mdp) {
 				$err = "Votre mot de passe est incorrect";
 				$this->erreur = $err;
-			}
-			else {
+			} else {
 				if ($new_mdp != $verif_new_mdp) {
 					$err = "Vos mots de passe sont différents";
 					$this->erreur = $err;
-				}
-				else {
+				} else {
 					$testmdp = $this->testpassword($new_mdp);
 					
 					if (strlen($new_mdp) < 5) {
 						$err = "Votre mot de passe est trop court";
 						$this->erreur = $err;
-					}
-					else if ($testmdp < 40) {
+					} else if ($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);
@@ -199,8 +190,7 @@ 
 				
 				$urlimg = $this->debut_lien."profil/defaut.png";
 				$dbc->query("UPDATE identite SET img_profil='$urlimg' WHERE ID_identite='$this->id_identite'");
-			}
-			else {
+			} else {
 				//test si il y a deja une img
 				$query = $dbc->query("SELECT img_profil FROM identite where ID_identite=$this->id_identite");
 
@@ -214,27 +204,22 @@ 
 				if(substr($image, -3) == (!$extension || !$extension2 || !$extension3 || !$extension4)) {
 					$err = "Votre image ne comporte pas l'extension .jpg";
 					$this->erreur = $err;
-				}
-				else if(($infos_img[0] >= $width_max) && ($infos_img[1] >= $height_max) && ($_FILES['image']['size'] >= $max_size)) {
+				} else if(($infos_img[0] >= $width_max) && ($infos_img[1] >= $height_max) && ($_FILES['image']['size'] >= $max_size)) {
 					$err = "Problème dans les dimensions ou taille de l\'image.";
 					$this->erreur = $err;
-				}
-				else {
+				} else {
 					$uniqid = uniqid();
 					$chemin_destination = '../images/profil/';
 					if(substr($image, -3) == ($extension)) {
 						move_uploaded_file($_FILES['image']['tmp_name'], $chemin_destination.$uniqid.".jpg");
 						$imageok = $uniqid.".jpg";
-					}
-					else if(substr($image, -3) == ($extension2)) {
+					} else if(substr($image, -3) == ($extension2)) {
 						move_uploaded_file($_FILES['image']['tmp_name'], $chemin_destination.$uniqid.".png");
 						$imageok = $uniqid.".png";
-					}
-					else if(substr($image, -3) == ($extension3)) {
+					} else if(substr($image, -3) == ($extension3)) {
 						move_uploaded_file($_FILES['image']['tmp_name'], $chemin_destination.$uniqid.".jpeg");
 						$imageok = $uniqid.".jpeg";
-					}
-					else {
+					} else {
 						move_uploaded_file($_FILES['image']['tmp_name'], $chemin_destination.$uniqid.".gif");
 						$imageok = $uniqid.".gif";
 					}
@@ -257,8 +242,7 @@ 
 				$oldimg_profil = explode("/", $oldimg_profil);
 				
 				if($oldimg_profil[7] == "defaut.png") {
-				}
-				else {
+				} else {
 					unlink("../images/profil/".$oldimg_profil[7]);
 				}
 			}
@@ -283,16 +267,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/Cache.class.php

Doc Comments +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
 		
 		//-------------------------- GETTER ----------------------------------------------------------------------------//
 		/**
-		 * @return bool
+		 * @return boolean|null
 		 * fonction verifie en bdd si on a déjà enregistrer le fichier en cache
 		 * si il ne l'est pas on le met, et si il y est et que reload cache == 0 on prend le fichier qui est en cache
 		 * sinon soit on update la bdd et on refait un cache soit on crée tout
@@ -132,7 +132,7 @@ 
 		}
 
 		/**
-		 * @param $contenu_fichier
+		 * @param string $contenu_fichier
 		 * fonction qui met en cache le contenu du fichier enregistrer dans le ob
 		 */
 		private function setCache($contenu_fichier) {

Spacing +4 added lines, -4 removed lines

@@ -15,7 +15,7 @@ 
 
 
 		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
-		public function __construct($page, $admin=null) {
+		public function __construct($page, $admin = null) {
 			$config = new Configuration();
 
 			//on test si le cache est bien active
@@ -34,10 +34,10 @@ 
 			}
 
 			if ($admin == null) {
-				$this->dossier_cache =  ROOT."cache/app/";
+				$this->dossier_cache = ROOT."cache/app/";
 			}
 			else {
-				$this->dossier_cache =  ROOT."cache/admin/";
+				$this->dossier_cache = ROOT."cache/admin/";
 			}
 
 			$page = ChaineCaractere::setUrl($page);
@@ -136,7 +136,7 @@ 
 		 * fonction qui met en cache le contenu du fichier enregistrer dans le ob
 		 */
 		private function setCache($contenu_fichier) {
-			$dbc= App::getDb();
+			$dbc = App::getDb();
 
 			$fichier_cache = $this->chemin_cache;
 

Braces +5 added lines, -10 removed lines

@@ -28,15 +28,13 @@ 
 					mkdir(ROOT."cache/admin");
 					mkdir(ROOT."cache/app");
 				}
-			}
-			else {
+			} else {
 				$this->cache_active = 0;
 			}
 
 			if ($admin == null) {
 				$this->dossier_cache =  ROOT."cache/app/";
-			}
-			else {
+			} else {
 				$this->dossier_cache =  ROOT."cache/admin/";
 			}
 
@@ -69,8 +67,7 @@ 
 					$reload_cache = $obj->reload_cache;
 					$this->no_cache = $obj->no_cache;
 				}
-			}
-			else {
+			} else {
 				$value = [
 					"nom_fichier" => $this->page,
 					"reload_cache" => 0
@@ -101,16 +98,14 @@ 
 					require_once($this->chemin_cache);
 
 					return true;
-				}
-				else {
+				} else {
 					if ($this->no_cache == null) {
 						ob_start();
 					}
 
 					return false;
 				}
-			}
-			else {
+			} else {
 				return false;
 			}
 		}

core/Configuration.class.php

Doc Comments +4 added lines

@@ -59,6 +59,10 @@
         public function getNomSite(){
             return $this->nom_site;
         }
+
+        /**
+         * @return string|null
+         */
         public function getMailSite(){
             return $this->mail_site;
         }

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
     namespace core;
 
 
-    use core\HTML\flashmessage\FlashMessage;
-
 	class Configuration {
 		//pour la configuration générale du site
         private $nom_site; //-> nom du site

Indentation +44 added lines, -44 removed lines

@@ -1,16 +1,16 @@ 
 <?php
-    namespace core;
+	namespace core;
 
 
-    use core\HTML\flashmessage\FlashMessage;
+	use core\HTML\flashmessage\FlashMessage;
 
 	class Configuration {
 		//pour la configuration générale du site
-        private $nom_site; //-> nom du site
-        private $mail_site; //-> pour le gérant du site contact@nomdedomaine;com
-        private $gerant_site; //->nom du gérant du site
-        private $url_site; //-> url de site
-        private $mail_administrateur; //-> mail de l'administrateur web@clicand.com
+		private $nom_site; //-> nom du site
+		private $mail_site; //-> pour le gérant du site contact@nomdedomaine;com
+		private $gerant_site; //->nom du gérant du site
+		private $url_site; //-> url de site
+		private $mail_administrateur; //-> mail de l'administrateur web@clicand.com
 		private $last_save; //-> derniere sauvegarde de la bdd
 		private $acces_admin; //-> si == 1 on a acces à l'admin
 		private $contenu_dynamique; //->savoir si es contenus sont dynamique (stockés in DB)
@@ -23,24 +23,24 @@ 
 		private $activer_connexion;
 
 
-        //-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
-        public function __construct() {
-            $dbc = \core\App::getDb();
+		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
+		public function __construct() {
+			$dbc = \core\App::getDb();
 
 			//pour la configuration générale du site
-            $query = $dbc->query("SELECT * FROM configuration WHERE ID_configuration=1");
-            foreach ($query as $obj) {
-                $this->nom_site = $obj->nom_site;
-                $this->mail_site = $obj->mail_site;
-                $this->gerant_site = $obj->gerant_site;
-                $this->url_site = $obj->url_site;
-                $this->mail_administrateur = $obj->mail_administrateur;
-                $this->last_save = $obj->last_save;
+			$query = $dbc->query("SELECT * FROM configuration WHERE ID_configuration=1");
+			foreach ($query as $obj) {
+				$this->nom_site = $obj->nom_site;
+				$this->mail_site = $obj->mail_site;
+				$this->gerant_site = $obj->gerant_site;
+				$this->url_site = $obj->url_site;
+				$this->mail_administrateur = $obj->mail_administrateur;
+				$this->last_save = $obj->last_save;
 				$this->acces_admin = $obj->acces_admin;
 				$this->contenu_dynamique = $obj->contenu_dynamique;
 				$this->responsive = $obj->responsive;
 				$this->cache = $obj->cache;
-            }
+			}
 
 			//pour la configuration des comptes
 			$query = $dbc->query("SELECT * FROM configuration_compte WHERE ID_configuration_compte=1");
@@ -49,31 +49,31 @@ 
 				$this->activer_inscription = $obj->activer_inscription;
 				$this->activer_connexion = $obj->activer_connexion;
 			}
-        }
-        //-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
+		}
+		//-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
 
 
 
-        //-------------------------- GETTER ----------------------------------------------------------------------------//
+		//-------------------------- GETTER ----------------------------------------------------------------------------//
 		//pour la configuration générale du site
-        public function getNomSite(){
-            return $this->nom_site;
-        }
-        public function getMailSite(){
-            return $this->mail_site;
-        }
-        public function getGerantSite(){
-            return $this->gerant_site;
-        }
-        public function getUrlSite(){
-            return $this->url_site;
-        }
-        public function getMailAdministrateur(){
-            return $this->mail_administrateur;
-        }
-        public function getLastSave(){
-            return $this->last_save;
-        }
+		public function getNomSite(){
+			return $this->nom_site;
+		}
+		public function getMailSite(){
+			return $this->mail_site;
+		}
+		public function getGerantSite(){
+			return $this->gerant_site;
+		}
+		public function getUrlSite(){
+			return $this->url_site;
+		}
+		public function getMailAdministrateur(){
+			return $this->mail_administrateur;
+		}
+		public function getLastSave(){
+			return $this->last_save;
+		}
 		public function getAccesAdmin() {
 			return $this->acces_admin;
 		}
@@ -97,11 +97,11 @@ 
 		public function getActiverConnexion() {
 			return $this->activer_connexion;
 		}
-        //-------------------------- FIN GETTER ----------------------------------------------------------------------------//
+		//-------------------------- FIN GETTER ----------------------------------------------------------------------------//
 
 
 
-        //-------------------------- SETTER ----------------------------------------------------------------------------//
+		//-------------------------- SETTER ----------------------------------------------------------------------------//
 		/**
 		 * fonction qui permet de mettre à jour la date de la derniere save de la bdd
 		 * + supprimer la sauverde ancienne d'il y a 1 mois
@@ -125,5 +125,5 @@ 
 				unlink(ROOT."bdd_backup/".$nom_save);
 			}
 		}
-        //-------------------------- FIN SETTER ----------------------------------------------------------------------------//
-    }
\ No newline at end of file
+		//-------------------------- FIN SETTER ----------------------------------------------------------------------------//
+	}
\ No newline at end of file

Spacing +6 added lines, -6 removed lines

@@ -56,22 +56,22 @@
 
         //-------------------------- GETTER ----------------------------------------------------------------------------//
 		//pour la configuration générale du site
-        public function getNomSite(){
+        public function getNomSite() {
             return $this->nom_site;
         }
-        public function getMailSite(){
+        public function getMailSite() {
             return $this->mail_site;
         }
-        public function getGerantSite(){
+        public function getGerantSite() {
             return $this->gerant_site;
         }
-        public function getUrlSite(){
+        public function getUrlSite() {
             return $this->url_site;
         }
-        public function getMailAdministrateur(){
+        public function getMailAdministrateur() {
             return $this->mail_administrateur;
         }
-        public function getLastSave(){
+        public function getLastSave() {
             return $this->last_save;
         }
 		public function getAccesAdmin() {

core/contenus/Contenus.class.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 	namespace core\contenus;
 
-	use core\functions\ChaineCaractere;
 	use core\RedirectError;
 
 

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
 
 
 		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
-		public function __construct($init_all=0) {
+		public function __construct($init_all = 0) {
 			$dbc = \core\App::getDb();
 
 			if ($init_all == 1) {
@@ -71,7 +71,7 @@ 
 		 * pour récupérer l'en tete d'une page (balise title ++ meta description)
 		 * @param $id_page
 		 */
-		public function getHeadPage($id_page, $url=null) {
+		public function getHeadPage($id_page, $url = null) {
 			$dbc = \core\App::getDb();
 
 			if ($id_page != 0) {
@@ -94,7 +94,7 @@ 
 		 * pour récupérer une page en particulier
 		 * @param $id_page
 		 */
-		public function getContenuPage($id_page=null) {
+		public function getContenuPage($id_page = null) {
 			$dbc = \core\App::getDb();
 
 			if ($id_page == null) {
@@ -120,7 +120,7 @@ 
 
 
 		//-------------------------- SETTER ----------------------------------------------------------------------------//
-		protected function setMenu($id_page, $titre, $balise_title, $url, $parent ){
+		protected function setMenu($id_page, $titre, $balise_title, $url, $parent) {
 			$this->id_page = $id_page;
 			$this->titre = $titre;
 			$this->balise_title = $balise_title;

Braces +1 added lines, -2 removed lines

@@ -76,8 +76,7 @@
 
 			if ($id_page != 0) {
 				$query = $dbc->query("SELECT balise_title, meta_description, ID_page FROM page WHERE ID_page=".$id_page);
-			}
-			else {
+			} else {
 				$query = $dbc->query("SELECT balise_title, meta_description, ID_page FROM page WHERE url LIKE '$url'");
 			}
 

core/Database.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 		 * @param $table
 		 * @param $champ
 		 * @param $value
-		 * @return bool
+		 * @return boolean|null
 		 */
 		public function rechercherEgalite($table, $champ, $value, $id_table=null, $id=null) {
 			if ($id == null) {

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
 		 * @param $req -> la req a executer
 		 * @param $value -> le ou les tableaux de valeurs
 		 */
-		public function prepare($req, $value=null) {
+		public function prepare($req, $value = null) {
 			//si pas de donnees
 			if ($value === null) {
 				$this->getPdo()->query($req);
@@ -97,7 +97,7 @@ 
 		 * @param $value
 		 * @return bool
 		 */
-		public function rechercherEgalite($table, $champ, $value, $id_table=null, $id=null) {
+		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'");
 			}

Braces +8 added lines, -11 removed lines

@@ -51,8 +51,7 @@ 
 			if ($query) {
 				$obj = $query->fetchAll(PDO::FETCH_OBJ);
 				return $obj;
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("Une erreur est survenue en executant cette requette : ".$req);
 			}
 		}
@@ -66,8 +65,7 @@ 
 			//si pas de donnees
 			if ($value === null) {
 				$this->getPdo()->query($req);
-			}
-			else {
+			} else {
 				$query = $this->getPdo()->prepare($req);
 				//si on a plusieurs tableaux
 				if (array_key_exists(0, $value)) {
@@ -76,8 +74,7 @@ 
 							$err = true;
 						}
 					}
-				}
-				else {
+				} else {
 					if (!$query->execute($value)) {
 						$err = true;
 					}
@@ -100,8 +97,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");
 			}
 
@@ -109,7 +105,9 @@ 
 				$nb = $obj["nb"];
 			}
 
-			if ($nb == 1) return true;
+			if ($nb == 1) {
+				return true;
+			}
 		}
 		//-------------------------- FIN FUNCTION QUI FONT DES REQUETES SUR LA BDD --------------------------------------------//
 
@@ -124,8 +122,7 @@ 
 
 			if ($query->rowCount() > 0) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}