Piou-piou / ribs-framework

core/admin/droitsacces/DroitAcces.php

Spacing +8 added lines, -8 removed lines

@@ -41,25 +41,25 @@ 
     
     
 		//-------------------------- GETTER ----------------------------------------------------------------------------//
-		public function getLogged(){
+		public function getLogged() {
 			return $this->logged;
 		}
-		public function getSuperAdmin(){
+		public function getSuperAdmin() {
 			return $this->super_admin;
 		}
-		public function getIdListeDroitAcces(){
+		public function getIdListeDroitAcces() {
 			return $this->id_liste_droit_acces;
 		}
-		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;
 		}
 
@@ -141,7 +141,7 @@ 
 			//page sans droit dans admin
 			$all_access = array("gestion-comptes/mon-compte", "index");
 
-			if (($this->super_admin == 1) || (in_array($this->getListeDroitPage($page), $this->getListeDroitAcces()))|| (in_array($page, $all_access))) {
+			if (($this->super_admin == 1) || (in_array($this->getListeDroitPage($page), $this->getListeDroitAcces())) || (in_array($page, $all_access))) {
 				return true;
 			}
 			else {

Braces +9 added lines, -8 removed lines

@@ -78,7 +78,9 @@ 
 				");
 
 			if ((is_array($query)) && (count($query) > 0)) {
-				foreach ($query as $obj) $liste_droit_acces[] = $obj->droit_acces;
+				foreach ($query as $obj) {
+					$liste_droit_acces[] = $obj->droit_acces;
+				}
 			}
 
 			return $liste_droit_acces;
@@ -93,7 +95,9 @@ 
 
 			$query = $dbc->query("SELECT droit_acces FROM droit_acces WHERE page LIKE '%$page%'");
 			if ((is_array($query)) && (count($query) > 0)) {
-				foreach ($query as $obj) $droit_acces = $obj->droit_acces;
+				foreach ($query as $obj) {
+					$droit_acces = $obj->droit_acces;
+				}
 			}
 
 			return $droit_acces;
@@ -143,8 +147,7 @@ 
 
 			if (($this->super_admin == 1) || (in_array($this->getListeDroitPage($page), $this->getListeDroitAcces()))|| (in_array($page, $all_access))) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -169,8 +172,7 @@ 
 			//si les trois sont différent de 0 on renvoit true soinon false
 			if (($this->super_admin == 1) || ((in_array($droit, $liste_droit_acces)) && in_array(1, $array_modif))) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -185,8 +187,7 @@ 
 
 			if (($this->super_admin == 1) || (in_array($droit_acces, $liste_droit_acces))) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}

core/admin/Admin.php

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/admin/droitsacces/GestionDroitAcces.php

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/Database.php

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);
 			}
 		}
@@ -71,8 +70,7 @@ 
 						$err = true;
 					}
 				}
-			}
-			else {
+			} else {
 				if (!$query->execute($value)) {
 					$err = true;
 				}
@@ -94,8 +92,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");
 			}
 
@@ -104,9 +101,10 @@ 
 					$nb = $obj["nb"];
 				}
 
-				if ((isset($nb)) && ($nb != 0)) return true;
-			}
-			else {
+				if ((isset($nb)) && ($nb != 0)) {
+					return true;
+				}
+			} else {
 				return false;
 			}
 		}
@@ -123,8 +121,7 @@ 
 
 			if ($query->rowCount() > 0) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}

core/database/Querybuilder.php

Spacing +8 added lines, -8 removed lines

@@ -111,7 +111,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 {
@@ -127,10 +127,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();
@@ -145,15 +145,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);
@@ -164,10 +164,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

@@ -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'");
 			}
 

Doc Comments +5 added lines, -5 removed lines

@@ -80,7 +80,7 @@ 
 		}
 
 		/**
-		 * @param $table
+		 * @param string $table
 		 * @return $this
 		 *
 		 * pour initialiser la les listes des tables ou il faudra aler chercher les données
@@ -92,7 +92,7 @@ 
 		}
 
 		/**
-		 * @param $table
+		 * @param string $table
 		 *
 		 * pour initialiser la table dans laquelle on va insérer les données
 		 */
@@ -104,9 +104,9 @@ 
 
 		/**
 		 * @param $champ
-		 * @param $cond
+		 * @param string $cond
 		 * @param $champ_test
-		 * @param null $closure
+		 * @param string $closure
 		 * @return $this
 		 *
 		 * pour intialiser la ou les clauses where d'une requete
@@ -123,7 +123,7 @@ 
 		}
 
 		/**
-		 * @param $order
+		 * @param string $order
 		 */
 		public function orderBy($order) {
 			$this->order_by = " ORDER BY ".$order;

core/Navigation.php

Braces +1 added lines, -2 removed lines

@@ -132,8 +132,7 @@
 				foreach ($query as $obj) {
 					if ($obj->activer == 1) {
 						return true;
-					}
-					else {
+					} else {
 						return false;
 					}
 				}

Spacing +3 added lines, -3 removed lines

@@ -68,7 +68,7 @@ 
 
 			if (is_array($query) && (count($query) > 0)) {
 				foreach ($query as $obj) {
-					$sous_menu[] = [$obj->ID_page, $obj->titre, $obj->url, $obj->balise_title,];
+					$sous_menu[] = [$obj->ID_page, $obj->titre, $obj->url, $obj->balise_title, ];
 				}
 			}
 
@@ -89,7 +89,7 @@ 
 
 			if (is_array($query) && (count($query) > 0)) {
 				foreach ($query as $obj) {
-					return [$obj->nom_module,$obj->url];
+					return [$obj->nom_module, $obj->url];
 				}
 			}
 		}
@@ -106,7 +106,7 @@ 
 			$dbc = App::getDb();
 
 			if ($afficher != null) {
-				$dbc->insert($id, $value_id)->insert("ordre", $this->last_ordre+1)->into("navigation")->set();
+				$dbc->insert($id, $value_id)->insert("ordre", $this->last_ordre + 1)->into("navigation")->set();
 			}
 		}
 

core/admin/contenus/GestionContenus.php

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
 
 					$dbc->prepare("INSERT INTO page (titre, contenu, url, meta_description, balise_title, ordre, parent, affiche) VALUES (:titre, :contenu, :url, :meta_description, :balise_title, :ordre, :parent, :affiche)", $value);
 					$this->id_page = $dbc->lastInsertId();
-					$this->setAjoutLienNavigation("ID_page", $this->id_page , 1);
+					$this->setAjoutLienNavigation("ID_page", $this->id_page, 1);
 				}
 				else {
 					FlashMessage::setFlash("Impossible de créer cette page, veuillez réeseyer dans un moment. Si le problème persiste contactez votre administrateur.");

Doc Comments +12 added lines

@@ -71,6 +71,14 @@ 
 
 
 		//-------------------------- SETTER ----------------------------------------------------------------------------//
+
+		/**
+		 * @param string $url
+		 * @param string $err_balise_title
+		 * @param string $err_url
+		 * @param string $err_meta_description
+		 * @param string $err_titre_page
+		 */
 		private function setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $contenu, $err_balise_title, $err_url, $err_meta_description, $err_titre_page) {
 			$_SESSION['balise_title'] = $balise_title;
 			$_SESSION['url'] = $url;
@@ -265,6 +273,10 @@ 
 			}
 		}
 
+		/**
+		 * @param string $id
+		 * @param integer $affiche
+		 */
 		private function setModifierLienNavigation($id, $id_page, $parent, $affiche) {
 			if ($parent != "") {
 				$nav = new Navigation();

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 	use core\App;
 	use core\contenus\Contenus;
-	use core\admin\droitsacces\DroitAcces;
 	use core\functions\ChaineCaractere;
 	use core\HTML\flashmessage\FlashMessage;
 	use core\Navigation;

Braces +8 added lines, -11 removed lines

@@ -31,7 +31,9 @@ 
 
 			$query = $dbc->query("SELECT titre FROM page WHERE ID_page=".$parent);
 			if ((is_array($query)) && (count($query) > 0)) {
-				foreach ($query as $obj) $this->parent_texte = $obj->titre;
+				foreach ($query as $obj) {
+					$this->parent_texte = $obj->titre;
+				}
 			}
 
 			return $this->parent_texte;
@@ -137,13 +139,11 @@ 
 					$dbc->prepare("INSERT INTO page (titre, contenu, url, meta_description, balise_title, ordre, parent, affiche) VALUES (:titre, :contenu, :url, :meta_description, :balise_title, :ordre, :parent, :affiche)", $value);
 					$this->id_page = $dbc->lastInsertId();
 					$this->setAjoutLienNavigation("ID_page", $this->id_page , 1);
-				}
-				else {
+				} else {
 					FlashMessage::setFlash("Impossible de créer cette page, veuillez réeseyer dans un moment. Si le problème persiste contactez votre administrateur.");
 					$this->erreur = true;
 				}
-			}
-			else {
+			} else {
 				$this->setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $contenu, $err_balise_title, $err_url, $err_meta_description, $err_titre_page);
 			}
 		}
@@ -207,8 +207,7 @@ 
 
 					$dbc->prepare("UPDATE page SET titre=:titre_page, contenu=:contenu, url=:url, meta_description=:meta_description, balise_title=:balise_title, parent=:parent, affiche=:affiche WHERE ID_page=:id_page", $value);
 					$this->setModifierLienNavigation("ID_page", $id_page, $this->getParentId($parent), $affiche);
-				}
-				else {
+				} else {
 					$this->setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $contenu, $err_balise_title, $err_url, $err_meta_description, $err_titre_page);
 				}
 			}
@@ -251,8 +250,7 @@ 
 					FlashMessage::setFlash("Impossible de supprimer cette page, veuillez contacter votre administrateur pour corriger ce problème");
 					$this->erreur = true;
 				}
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("Impossible de supprimer cette page, veuillez contacter votre administrateur pour corriger ce problème");
 				$this->erreur = true;
 			}
@@ -269,8 +267,7 @@ 
 			if ($parent != "") {
 				$nav = new Navigation();
 				$nav->setSupprimerLien($id, $id_page);
-			}
-			else if (($affiche == 1) && ($parent == "")) {
+			} else if (($affiche == 1) && ($parent == "")) {
 				$this->setAjoutLienNavigation($id, $id_page, $affiche);
 			}
 		}

admin/views/gestion-contenus/header.php

Spacing +4 added lines, -4 removed lines

@@ -1,17 +1,17 @@
 <nav class="nav-page">
 	<div class="inner">
 		<ul>
-			<?php $nav = new \core\Navigation();foreach ($nav->getNavigation() as $nav):?>
+			<?php $nav = new \core\Navigation(); foreach ($nav->getNavigation() as $nav):?>
 				<li><a href="<?=ADMWEBROOT?>gestion-contenus/modifier-contenu?id=<?=$nav[0]?>" title="<?=$nav[3]?>"><?=$nav[1]?></a>
 					<?php if (count($nav[4]) > 0):?>
 						<ul>
 							<?php foreach ($nav[4] as $snav):?>
 								<li><a href="<?=ADMWEBROOT?>gestion-contenus/modifier-contenu?id=<?=$snav[0]?>" title="<?=$snav[3]?>"><?=$snav[1]?></a></li>
-							<?php endforeach;?>
+							<?php endforeach; ?>
 						</ul>
-					<?php endif;?>
+					<?php endif; ?>
 				</li>
-			<?php endforeach;?>
+			<?php endforeach; ?>
 		</ul>
 	</div>
 </nav>