Inspection of "New mail function add a facade to simplify it" - Piou-piou/ribs-framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b01865...b01865 )

by Anthony

created 2016-06-24 14:23 UTC

Status

Braces +1 added lines, -2 removed lines patch added patch discarded remove patch

@@ -24,8 +24,7 @@
 block discarded – undo
 		$_SESSION['acces_admin'] = $inscription->getAccesAdministration();
 		//$id_liste_droit_acces = $_SESSION['id_liste_droit_acces'];
 		\core\HTML\flashmessage\FlashMessage::setFlash("<ul>".$inscription->getErreur()."</ul>");
-	}
-	else {
+	} else {
 		\core\HTML\flashmessage\FlashMessage::setFlash("L'utilisateur ".$inscription->getNom()." ".$inscription->getPrenom()."a bien été inscrit", "success");
 		$inscription->setInscrireUtilisateur();
 	}

Please login to merge, or discard this patch.

core/admin/contenus/ParentTexte.php 1 patch

Braces +3 added lines, -1 removed lines patch added patch discarded remove patch

@@ -16,7 +16,9 @@
 block discarded – undo
 			$query = $dbc->select("titre")->from("page")->where("ID_page", "=", $parent)->get();
 
 			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;

Please login to merge, or discard this patch.

admin/views/template/principal.php 1 patch

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -9,7 +9,7 @@  discard block
 block discarded – undo
 		<link rel="stylesheet" type="text/css" href="<?=LIBSWEBROOT?>reset_css/reset.css">
 		<link rel="stylesheet" type="text/css" href="<?=WEBROOT?>admin/views/template/css/style.css">
 		<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.0/jquery.min.js"></script>
-		<?php require_once(ROOT."admin/views/template/js/menu.php");?>
+		<?php require_once(ROOT."admin/views/template/js/menu.php"); ?>
 
 		<!-- Les librairies utlisées -->
 		<link rel="stylesheet" type="text/css" href="<?=LIBSWEBROOT?>popup/css/style.css">
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
 	</head>
 	<?=\core\HTML\flashmessage\FlashMessage::getFlash(); ?>
 	<body>
-		<nav class="menu <?php if (($_SESSION["menu_plie".CLEF_SITE] == "deplie") || (!isset($_SESSION["menu_plie".CLEF_SITE]))):?>active<?php endif;?>">
+		<nav class="menu <?php if (($_SESSION["menu_plie".CLEF_SITE] == "deplie") || (!isset($_SESSION["menu_plie".CLEF_SITE]))):?>active<?php endif; ?>">
 			<div class="titre">
 				<h1>Ribs V2.3.5.3</h1>
 				<i class="fa fa-bars"></i>
@@ -41,14 +41,14 @@  discard block
 block discarded – undo
 										<a href="<?=ADMWEBROOT?>notifications"><i class="fa fa-bell  <?php if ($admin->getNotification() == 1):?> animated infinite swing<?php endif; ?>"></i></a>
 									</div>
 								</div>
-							<?php endif;?>
+							<?php endif; ?>
 							<?php if (($droit_acces->getSuperAdmin() == 1) || ($droit_acces->getDroitAccesPage("configuration/mon-compte"))):?>
 								<div class="colonne">
 									<div class="config">
 										<a href="<?=ADMWEBROOT?>configuration/index"><i class="fa fa-gear"></i></a>
 									</div>
 								</div>
-							<?php endif;?>
+							<?php endif; ?>
 							<div class="colonne">
 								<div class="logout">
 									<a href="<?=WEBROOT?>administrator/controller/core/auth/connexion/logout"><i class="fa fa-sign-out animated activate swing infinite"></i></a>
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
 							<ul>
 								<?php if ($droit_acces->getDroitAccesAction("CREATION COMPTE ADMIN")):?>
 									<li><i class="fa fa-user"></i><a href="<?=ADMWEBROOT?>gestion-comptes/creer-utilisateur">Créer un utilisateur</a></li>
-								<?php endif;?>
+								<?php endif; ?>
 							</ul>
 						</li>
 					<?php endif; ?>

Please login to merge, or discard this patch.

core/auth/Connexion.php 1 patch

Braces +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -71,8 +71,7 @@  discard block
 block discarded – undo
 			if (!isset($id)) {
 				FlashMessage::setFlash("Vos identifiants de connexions sont incorrects");
 				header("location:$page_retour_err");
-			}
-			else {
+			} else {
 				self::setTestParamCompte($valide, $archiver, $page_retour_err);
 
 				//si les mdp sont egaux on redirige ver esace membre sinon ver login avec un mess d'erreur
@@ -84,8 +83,7 @@  discard block
 block discarded – undo
 
 					FlashMessage::setFlash("Vous êtes maintenant connecté", "info");
 					header("location:$page_retour");
-				}
-				else {
+				} else {
 					FlashMessage::setFlash("Vos identifiants de connexions sont incorrects");
 					header("location:$page_retour_err");
 				}
@@ -171,7 +169,9 @@  discard block
 block discarded – undo
 			$dbc = App::getDb();
 			$query = $dbc->select("last_change_mdp")->from("identite")->where("ID_identite", "=", $id_identite)->get();
 			if ((is_array($query)) && (count($query) > 0)) {
-				foreach ($query as $obj) return $obj->last_change_mdp;
+				foreach ($query as $obj) {
+					return $obj->last_change_mdp;
+				}
 			}
 		}
 

Please login to merge, or discard this patch.

core/admin/contenus/GestionContenus.php 2 patches

Doc Comments +9 added lines, -6 removed lines patch added patch discarded remove patch

@@ -33,6 +33,9 @@  discard block
 block discarded – undo
 			return $bloc_editable;
 		}
 
+		/**
+		 * @param integer $parent
+		 */
 		private function getOrdrePage($parent) {
 			if (($parent != "") || ($parent != 0)) {
 				$dbc = \core\App::getDb();
@@ -66,13 +69,13 @@  discard block
 block discarded – undo
 		}
 
 		/**
-		 * @param $nom_table
-		 * @param $nom_id_table
-		 * @param $champ
+		 * @param string $nom_table
+		 * @param string $nom_id_table
+		 * @param string $champ
 		 * @param $value
-		 * @param $limit_char
-		 * @param $err_char
-		 * @param $err_egalite
+		 * @param integer $limit_char
+		 * @param string $err_char
+		 * @param string $err_egalite
 		 * @param null $value_id_table
 		 * @return string
 		 * fonction qui permet de vérifier qu'il n'y ait pas d'erreur dans le champ spécifié ni de doublons

Please login to merge, or discard this patch.

Braces +9 added lines, -13 removed lines patch added patch discarded remove patch

@@ -52,7 +52,9 @@  discard block
 block discarded – undo
 		private function getParentId($parent) {
 			$dbc = \core\App::getDb();
 
-			if ($parent == "") return 0;
+			if ($parent == "") {
+				return 0;
+			}
 
 			$query = $dbc->select("ID_page")->from("page")->where("titre", " LIKE ", '"%'.$parent.'%"', "", true)->get();
 
@@ -83,8 +85,7 @@  discard block
 block discarded – undo
 			if (strlen(utf8_decode($value)) > $limit_char) {
 				$this->erreur = true;
 				return "<li>$err_char</li>";
-			}
-			else if ($dbc->rechercherEgalite($nom_table, $champ, $value, $nom_id_table, $value_id_table) == true) {
+			} else if ($dbc->rechercherEgalite($nom_table, $champ, $value, $nom_id_table, $value_id_table) == true) {
 				$this->erreur = true;
 				return "<li>$err_egalite</li>";
 			}
@@ -168,13 +169,11 @@  discard block
 block discarded – undo
 					if ($parent == "") {
 						$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, $err_balise_title, $err_url, $err_meta_description, $err_titre_page);
 				$this->erreur = true;
 			}
@@ -238,8 +237,7 @@  discard block
 block discarded – undo
 						->set();
 
 					$this->setModifierLienNavigation("ID_page", $id_page, $this->getParentId($parent), $affiche);
-				}
-				else {
+				} else {
 					$this->setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $err_balise_title, $err_url, $err_meta_description, $err_titre_page);
 				}
 			}
@@ -288,8 +286,7 @@  discard block
 block discarded – undo
 					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;
 			}
@@ -315,8 +312,7 @@  discard block
 block discarded – undo
 			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);
 			}
 		}

Please login to merge, or discard this patch.

core/functions/DateHeure.php 1 patch

Braces +6 added lines, -12 removed lines patch added patch discarded remove patch

@@ -28,8 +28,7 @@  discard block
 block discarded – undo
 				$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 @@  discard block
 block discarded – undo
 
 				$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 @@  discard block
 block discarded – undo
 				$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();
@@ -85,8 +82,7 @@  discard block
 block discarded – undo
 
 			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)) {
 				$explode = explode("/", $date);
 				$jour_d = $explode[0];
 				$mois_d = $explode[1];
@@ -130,8 +126,7 @@  discard block
 block discarded – undo
 				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();
@@ -153,8 +148,7 @@  discard block
 block discarded – undo
 				$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();

Please login to merge, or discard this patch.

core/modules/ImportModule.php 1 patch

Braces +8 added lines, -6 removed lines patch added patch discarded remove patch

@@ -21,8 +21,12 @@  discard block
 block discarded – undo
 		//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
 		public function __construct() {
 			//On test si dossier temporaire + modules a la racines existes bien sinon on les crées
-			if (!file_exists(ROOT."temp")) mkdir(ROOT."temp");
-			if (!file_exists(ROOT."modules")) mkdir(ROOT."modules");
+			if (!file_exists(ROOT."temp")) {
+				mkdir(ROOT."temp");
+			}
+			if (!file_exists(ROOT."modules")) {
+				mkdir(ROOT."modules");
+			}
 		}
 		//-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
 		
@@ -95,8 +99,7 @@  discard block
 block discarded – undo
 				}
 
 				FlashMessage::setFlash("Votre module a bien été ajouté au site.", "success");
-			}
-			else {
+			} else {
 				FlashMessage::setFlash("Le module n'a pas pu être correctement téléchargé et installé, veuillez réesseyer dans un instant");
 			}
 		}
@@ -108,8 +111,7 @@  discard block
 block discarded – undo
 		private function setSupprimerArchiveTemp() {
 			if (unlink(TEMPROOT.$this->nom_fichier) == true) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}

Please login to merge, or discard this patch.

core/App.php 1 patch

Braces +6 added lines, -3 removed lines patch added patch discarded remove patch

@@ -55,7 +55,11 @@  discard block
 block discarded – undo
 				$objects = scandir($url);
 				foreach ($objects as $object) {
 					if ($object != "." && $object != "..") {
-						if (filetype($url."/".$object) == "dir") App::supprimerDossier($url."/".$object); else unlink($url."/".$object);
+						if (filetype($url."/".$object) == "dir") {
+							App::supprimerDossier($url."/".$object);
+						} else {
+							unlink($url."/".$object);
+						}
 					}
 				}
 				reset($objects);
@@ -84,8 +88,7 @@  discard block
 block discarded – undo
 					'secure' => SMTP_SECURE,
 					'port' => SMTP_PORT
 				]);
-			}
-			else {
+			} else {
 				$mailer = new \Nette\Mail\SmtpMailer();
 			}
 

Please login to merge, or discard this patch.

		@@ -24,8 +24,7 @@
		block discarded – undo
24	24	$_SESSION['acces_admin'] = $inscription->getAccesAdministration();
25	25	//$id_liste_droit_acces = $_SESSION['id_liste_droit_acces'];
26	26	\core\HTML\flashmessage\FlashMessage::setFlash("<ul>".$inscription->getErreur()."</ul>");
27		- }
28		- else {
	27	+ } else {
29	28	\core\HTML\flashmessage\FlashMessage::setFlash("L'utilisateur ".$inscription->getNom()." ".$inscription->getPrenom()."a bien été inscrit", "success");
30	29	$inscription->setInscrireUtilisateur();
31	30	}

		@@ -16,7 +16,9 @@
		block discarded – undo
16	16	$query = $dbc->select("titre")->from("page")->where("ID_page", "=", $parent)->get();
17	17
18	18	if ((is_array($query)) && (count($query) > 0)) {
19		- foreach ($query as $obj) $this->parent_texte = $obj->titre;
	19	+ foreach ($query as $obj) {
	20	+ $this->parent_texte = $obj->titre;
	21	+ }
20	22	}
21	23
22	24	return $this->parent_texte;

		@@ -9,7 +9,7 @@ discard block
		block discarded – undo
9	9	<link rel="stylesheet" type="text/css" href="<?=LIBSWEBROOT?>reset_css/reset.css">
10	10	<link rel="stylesheet" type="text/css" href="<?=WEBROOT?>admin/views/template/css/style.css">
11	11	<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.0/jquery.min.js"></script>
12		- <?php require_once(ROOT."admin/views/template/js/menu.php");?>
	12	+ <?php require_once(ROOT."admin/views/template/js/menu.php"); ?>
13	13
14	14	<!-- Les librairies utlisées -->
15	15	<link rel="stylesheet" type="text/css" href="<?=LIBSWEBROOT?>popup/css/style.css">
		@@ -19,7 +19,7 @@ discard block
		block discarded – undo
19	19	</head>
20	20	<?=\core\HTML\flashmessage\FlashMessage::getFlash(); ?>
21	21	<body>
22		- <nav class="menu <?php if (($_SESSION["menu_plie".CLEF_SITE] == "deplie") \|\| (!isset($_SESSION["menu_plie".CLEF_SITE]))):?>active<?php endif;?>">
	22	+ <nav class="menu <?php if (($_SESSION["menu_plie".CLEF_SITE] == "deplie") \|\| (!isset($_SESSION["menu_plie".CLEF_SITE]))):?>active<?php endif; ?>">
23	23	<div class="titre">
24	24	<h1>Ribs V2.3.5.3</h1>
25	25	<i class="fa fa-bars"></i>
		@@ -41,14 +41,14 @@ discard block
		block discarded – undo
41	41	<a href="<?=ADMWEBROOT?>notifications"><i class="fa fa-bell <?php if ($admin->getNotification() == 1):?> animated infinite swing<?php endif; ?>"></i></a>
42	42	</div>
43	43	</div>
44		- <?php endif;?>
	44	+ <?php endif; ?>
45	45	<?php if (($droit_acces->getSuperAdmin() == 1) \|\| ($droit_acces->getDroitAccesPage("configuration/mon-compte"))):?>
46	46	<div class="colonne">
47	47	<div class="config">
48	48	<a href="<?=ADMWEBROOT?>configuration/index"><i class="fa fa-gear"></i></a>
49	49	</div>
50	50	</div>
51		- <?php endif;?>
	51	+ <?php endif; ?>
52	52	<div class="colonne">
53	53	<div class="logout">
54	54	<a href="<?=WEBROOT?>administrator/controller/core/auth/connexion/logout"><i class="fa fa-sign-out animated activate swing infinite"></i></a>
		@@ -67,7 +67,7 @@ discard block
		block discarded – undo
67	67	<ul>
68	68	<?php if ($droit_acces->getDroitAccesAction("CREATION COMPTE ADMIN")):?>
69	69	<li><i class="fa fa-user"></i><a href="<?=ADMWEBROOT?>gestion-comptes/creer-utilisateur">Créer un utilisateur</a></li>
70		- <?php endif;?>
	70	+ <?php endif; ?>
71	71	</ul>
72	72	</li>
73	73	<?php endif; ?>

		@@ -71,8 +71,7 @@ discard block
		block discarded – undo
71	71	if (!isset($id)) {
72	72	FlashMessage::setFlash("Vos identifiants de connexions sont incorrects");
73	73	header("location:$page_retour_err");
74		- }
75		- else {
	74	+ } else {
76	75	self::setTestParamCompte($valide, $archiver, $page_retour_err);
77	76
78	77	//si les mdp sont egaux on redirige ver esace membre sinon ver login avec un mess d'erreur
		@@ -84,8 +83,7 @@ discard block
		block discarded – undo
84	83
85	84	FlashMessage::setFlash("Vous êtes maintenant connecté", "info");
86	85	header("location:$page_retour");
87		- }
88		- else {
	86	+ } else {
89	87	FlashMessage::setFlash("Vos identifiants de connexions sont incorrects");
90	88	header("location:$page_retour_err");
91	89	}
		@@ -171,7 +169,9 @@ discard block
		block discarded – undo
171	169	$dbc = App::getDb();
172	170	$query = $dbc->select("last_change_mdp")->from("identite")->where("ID_identite", "=", $id_identite)->get();
173	171	if ((is_array($query)) && (count($query) > 0)) {
174		- foreach ($query as $obj) return $obj->last_change_mdp;
	172	+ foreach ($query as $obj) {
	173	+ return $obj->last_change_mdp;
	174	+ }
175	175	}
176	176	}
177	177

		@@ -33,6 +33,9 @@ discard block
		block discarded – undo
33	33	return $bloc_editable;
34	34	}
35	35
	36	+ /**
	37	+ * @param integer $parent
	38	+ */
36	39	private function getOrdrePage($parent) {
37	40	if (($parent != "") \|\| ($parent != 0)) {
38	41	$dbc = \core\App::getDb();
		@@ -66,13 +69,13 @@ discard block
		block discarded – undo
66	69	}
67	70
68	71	/**
69		- * @param $nom_table
70		- * @param $nom_id_table
71		- * @param $champ
	72	+ * @param string $nom_table
	73	+ * @param string $nom_id_table
	74	+ * @param string $champ
72	75	* @param $value
73		- * @param $limit_char
74		- * @param $err_char
75		- * @param $err_egalite
	76	+ * @param integer $limit_char
	77	+ * @param string $err_char
	78	+ * @param string $err_egalite
76	79	* @param null $value_id_table
77	80	* @return string
78	81	* fonction qui permet de vérifier qu'il n'y ait pas d'erreur dans le champ spécifié ni de doublons

		@@ -52,7 +52,9 @@ discard block
		block discarded – undo
52	52	private function getParentId($parent) {
53	53	$dbc = \core\App::getDb();
54	54
55		- if ($parent == "") return 0;
	55	+ if ($parent == "") {
	56	+ return 0;
	57	+ }
56	58
57	59	$query = $dbc->select("ID_page")->from("page")->where("titre", " LIKE ", '"%'.$parent.'%"', "", true)->get();
58	60
		@@ -83,8 +85,7 @@ discard block
		block discarded – undo
83	85	if (strlen(utf8_decode($value)) > $limit_char) {
84	86	$this->erreur = true;
85	87	return "<li>$err_char</li>";
86		- }
87		- else if ($dbc->rechercherEgalite($nom_table, $champ, $value, $nom_id_table, $value_id_table) == true) {
	88	+ } else if ($dbc->rechercherEgalite($nom_table, $champ, $value, $nom_id_table, $value_id_table) == true) {
88	89	$this->erreur = true;
89	90	return "<li>$err_egalite</li>";
90	91	}
		@@ -168,13 +169,11 @@ discard block
		block discarded – undo
168	169	if ($parent == "") {
169	170	$this->setAjoutLienNavigation("ID_page", $this->id_page, 1);
170	171	}
171		- }
172		- else {
	172	+ } else {
173	173	FlashMessage::setFlash("Impossible de créer cette page, veuillez réeseyer dans un moment. Si le problème persiste contactez votre administrateur.");
174	174	$this->erreur = true;
175	175	}
176		- }
177		- else {
	176	+ } else {
178	177	$this->setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $err_balise_title, $err_url, $err_meta_description, $err_titre_page);
179	178	$this->erreur = true;
180	179	}
		@@ -238,8 +237,7 @@ discard block
		block discarded – undo
238	237	->set();
239	238
240	239	$this->setModifierLienNavigation("ID_page", $id_page, $this->getParentId($parent), $affiche);
241		- }
242		- else {
	240	+ } else {
243	241	$this->setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $err_balise_title, $err_url, $err_meta_description, $err_titre_page);
244	242	}
245	243	}
		@@ -288,8 +286,7 @@ discard block
		block discarded – undo
288	286	FlashMessage::setFlash("Impossible de supprimer cette page, veuillez contacter votre administrateur pour corriger ce problème");
289	287	$this->erreur = true;
290	288	}
291		- }
292		- else {
	289	+ } else {
293	290	FlashMessage::setFlash("Impossible de supprimer cette page, veuillez contacter votre administrateur pour corriger ce problème");
294	291	$this->erreur = true;
295	292	}
		@@ -315,8 +312,7 @@ discard block
		block discarded – undo
315	312	if ($parent != "") {
316	313	$nav = new Navigation();
317	314	$nav->setSupprimerLien($id, $id_page);
318		- }
319		- else if (($affiche == 1) && ($parent == "")) {
	315	+ } else if (($affiche == 1) && ($parent == "")) {
320	316	$this->setAjoutLienNavigation($id, $id_page, $affiche);
321	317	}
322	318	}

		@@ -28,8 +28,7 @@ discard block
		block discarded – undo
28	28	$minutesec = $minute * 60;
29	29	$heureseconde = $heuresec + $minutesec;
30	30	return $heureseconde;
31		- }
32		- else {
	31	+ } else {
33	32	FlashMessage::setFlash("La/les valeurs entrée n'est/ne sont pas de type int");
34	33	FlashMessage::getFlash();
35	34	die();
		@@ -48,8 +47,7 @@ discard block
		block discarded – undo
48	47
49	48	$temps = $heure."h".$minute."m".$seconde;
50	49	return $temps;
51		- }
52		- else {
	50	+ } else {
53	51	FlashMessage::setFlash("La valeur entrée n'est pas de type int");
54	52	FlashMessage::getFlash();
55	53	die();
		@@ -66,8 +64,7 @@ discard block
		block discarded – undo
66	64	$chaine = str_replace(":", "h", $temps);
67	65
68	66	return $chaine;
69		- }
70		- else {
	67	+ } else {
71	68	FlashMessage::setFlash("La valeur entrée n'est pas de type h:m");
72	69	FlashMessage::getFlash();
73	70	die();
		@@ -85,8 +82,7 @@ discard block
		block discarded – undo
85	82
86	83	if ($date == 0) {
87	84	return $jours[date("w")]." ".date("j").(date("j") == 1 ? "er" : " ").$mois[date("n") - 1]." ".date("Y");
88		- }
89		- else if ((strpos($date, "-") > 0) \|\| (strpos($date, "/") > 0)) {
	85	+ } else if ((strpos($date, "-") > 0) \|\| (strpos($date, "/") > 0)) {
90	86	$explode = explode("/", $date);
91	87	$jour_d = $explode[0];
92	88	$mois_d = $explode[1];
		@@ -130,8 +126,7 @@ discard block
		block discarded – undo
130	126	self::$annee = $annee;
131	127
132	128	return $jour."/".$mois."/".$annee;
133		- }
134		- else {
	129	+ } else {
135	130	FlashMessage::setFlash("format de date passé en paramètre ne correspond pas à YYYY-mm-jj");
136	131	FlashMessage::getFlash();
137	132	die();
		@@ -153,8 +148,7 @@ discard block
		block discarded – undo
153	148	$annee = $explode[2];
154	149
155	150	return $annee."-".$mois."-".$jour;
156		- }
157		- else {
	151	+ } else {
158	152	FlashMessage::setFlash("Format de date passé en paramètre ne correspond pas à jj/mm/aaaa");
159	153	FlashMessage::getFlash();
160	154	die();

		@@ -21,8 +21,12 @@ discard block
		block discarded – undo
21	21	//-------------------------- CONSTRUCTEUR ----------------------------------------------------------------------------//
22	22	public function __construct() {
23	23	//On test si dossier temporaire + modules a la racines existes bien sinon on les crées
24		- if (!file_exists(ROOT."temp")) mkdir(ROOT."temp");
25		- if (!file_exists(ROOT."modules")) mkdir(ROOT."modules");
	24	+ if (!file_exists(ROOT."temp")) {
	25	+ mkdir(ROOT."temp");
	26	+ }
	27	+ if (!file_exists(ROOT."modules")) {
	28	+ mkdir(ROOT."modules");
	29	+ }
26	30	}
27	31	//-------------------------- FIN CONSTRUCTEUR ----------------------------------------------------------------------------//
28	32
		@@ -95,8 +99,7 @@ discard block
		block discarded – undo
95	99	}
96	100
97	101	FlashMessage::setFlash("Votre module a bien été ajouté au site.", "success");
98		- }
99		- else {
	102	+ } else {
100	103	FlashMessage::setFlash("Le module n'a pas pu être correctement téléchargé et installé, veuillez réesseyer dans un instant");
101	104	}
102	105	}
		@@ -108,8 +111,7 @@ discard block
		block discarded – undo
108	111	private function setSupprimerArchiveTemp() {
109	112	if (unlink(TEMPROOT.$this->nom_fichier) == true) {
110	113	return true;
111		- }
112		- else {
	114	+ } else {
113	115	return false;
114	116	}
115	117	}

		@@ -55,7 +55,11 @@ discard block
		block discarded – undo
55	55	$objects = scandir($url);
56	56	foreach ($objects as $object) {
57	57	if ($object != "." && $object != "..") {
58		- if (filetype($url."/".$object) == "dir") App::supprimerDossier($url."/".$object); else unlink($url."/".$object);
	58	+ if (filetype($url."/".$object) == "dir") {
	59	+ App::supprimerDossier($url."/".$object);
	60	+ } else {
	61	+ unlink($url."/".$object);
	62	+ }
59	63	}
60	64	}
61	65	reset($objects);
		@@ -84,8 +88,7 @@ discard block
		block discarded – undo
84	88	'secure' => SMTP_SECURE,
85	89	'port' => SMTP_PORT
86	90	]);
87		- }
88		- else {
	91	+ } else {
89	92	$mailer = new \Nette\Mail\SmtpMailer();
90	93	}
91	94

Piou-piou / ribs-framework

Push — master ( b01865...b01865 )

Status

Category

Braces +1 added lines, -2 removed lines patch added patch discarded remove patch

Braces +3 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

Braces +5 added lines, -5 removed lines patch added patch discarded remove patch

Doc Comments +9 added lines, -6 removed lines patch added patch discarded remove patch

Braces +9 added lines, -13 removed lines patch added patch discarded remove patch

Braces +6 added lines, -12 removed lines patch added patch discarded remove patch

Braces +8 added lines, -6 removed lines patch added patch discarded remove patch

Braces +6 added lines, -3 removed lines patch added patch discarded remove patch