Piou-piou / ribs-framework

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/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);
 			}
 		}

core/Navigation.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 			$navigation = [];
 			$last_ordre = "";
 
-			if($no_module === null) {
+			if ($no_module === null) {
 				$query = $dbc->select()->from("navigation")->orderBy("ordre")->get();
 			}
 			else {
@@ -74,7 +74,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, ];
 				}
 			}
 
@@ -111,7 +111,7 @@ 
 		public function setAjoutLien($id, $value_id) {
 			$dbc = App::getDb();
 
-			$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();
 		}
 
 		public function setSupprimerLien($id, $value_id) {

Braces +2 added lines, -4 removed lines

@@ -14,8 +14,7 @@ 
 
 			if($no_module === null) {
 				$query = $dbc->select()->from("navigation")->orderBy("ordre")->get();
-			}
-			else {
+			} else {
 				$query = $dbc->select()->from("navigation")->where("ID_page", " IS NOT ", "NULL")->orderBy("ordre")->get();
 			}
 
@@ -23,8 +22,7 @@ 
 				foreach ($query as $obj) {
 					if ($obj->ID_page === null) {
 						$navigation[] = $this->getLienNavigationModule($obj->ID_module);
-					}
-					else {
+					} else {
 						$navigation[] = $this->getLienNavigationPage($obj->ID_page);
 					}
 					$last_ordre = $obj->ordre;

core/modules/GestionModule.php

Braces +2 added lines, -4 removed lines

@@ -130,8 +130,7 @@ 
 				foreach ($query as $obj) {
 					if ($obj->activer == 1) {
 						return true;
-					}
-					else {
+					} else {
 						return false;
 					}
 				}
@@ -183,8 +182,7 @@ 
 
 			if ($activer == 1) {
 				$nav->setAjoutLien("ID_module", self::getUrlToId($url));
-			}
-			else {
+			} else {
 				$nav->setSupprimerLien("ID_module", self::getUrlToId($url));
 			}
 		}

admin/views/gestion-contenus/header.php

Spacing +3 added lines, -3 removed lines

@@ -7,11 +7,11 @@
 						<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>

core/admin/inscription/AdminInscription.php

Braces +4 added lines, -8 removed lines

@@ -36,12 +36,10 @@ 
 			if ($this->getTestInt($value) === true) {
 				$this->acces_administration = $value;
 				return true;
-			}
-			else if ($this->getTestInt($value) === false) {
+			} else if ($this->getTestInt($value) === false) {
 				$this->erreur .= "<li>Le champs accès administration n'est pas au bon format</li>";
 				return false;
-			}
-			else {
+			} else {
 				return true;
 			}
 
@@ -59,12 +57,10 @@ 
 			if ($this->getTestInt($value) === true) {
 				$this->id_liste_droit_acces = $value;
 				return true;
-			}
-			else if ($this->getTestInt($value) === false) {
+			} else if ($this->getTestInt($value) === false) {
 				$this->erreur .= "<li>Le champs accès administration n'est pas au bon format</li>";
 				return false;
-			}
-			else {
+			} else {
 				return true;
 			}
 

core/mail/Mail.php

Braces +4 added lines, -3 removed lines

@@ -24,8 +24,7 @@ 
 		public function setVerifierMail($email) {
 			if (filter_var($email, FILTER_VALIDATE_EMAIL)) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -42,7 +41,9 @@ 
 			//on récupere le mail du site
 			$config = new \core\Configuration();
 
-			if ($from == null) $from = $config->getMailSite();
+			if ($from == null) {
+				$from = $config->getMailSite();
+			}
 
 			$headers = 'Content-type: text/html; charset=utf-8'."\r\n";
 			$headers .= "From: ".$from;

admin/views/gestion-contenus/modifier-contenu.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 		<button type="submit" class="submit-contenu" type="submit"><i class="fa fa-check"></i>Valider</button>
 	<?php endif; ?>
 	<input type="hidden" name="id_page" value="<?=$id_page_courante?>">
-	<?php if (($_GET['id'] != 1) && (($droit_acces->getSupprimerPage() == 1)  || ($droit_acces->getSuperAdmin() == 1))):?>
+	<?php if (($_GET['id'] != 1) && (($droit_acces->getSupprimerPage() == 1) || ($droit_acces->getSuperAdmin() == 1))):?>
 		<button id="supprimer-page-contenu" type="button" class="submit-contenu supprimer-page supprimer open-popup" popup="supprimer-page" href="<?=ADMWEBROOT?>controller/core/admin/contenus/gestion/supprimer_page?id=<?=$id_page_courante?>"><i class="fa fa-times"></i>Supprimer cette page</button>
 	<?php endif; ?>