Passed
Push — master ( 855a75...d1b130 )
by Anthony
02:47
created
core/modules/RouterModule.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -59,12 +59,10 @@
 block discarded – undo
59 59
 				if (in_array($explode[$i], $this->getAllModules())) {
60 60
 					$this->module = $explode[$i];
61 61
 					$debut_url = $explode[$i];
62
-				}
63
-				else if ($i == 1) {
62
+				} else if ($i == 1) {
64 63
 					$centre_url = $explode[$i];
65 64
 					$this->page = $explode[$i];
66
-				}
67
-				else {
65
+				} else {
68 66
 					$this->parametre = $explode[$i];
69 67
 				}
70 68
 			}
Please login to merge, or discard this patch.
core/App.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -48,8 +48,7 @@
 block discarded – undo
48 48
 			if (strlen(utf8_decode($value)) > $limit_char) {
49 49
 				self::$erreur = true;
50 50
 				return "<li>$err_char</li>";
51
-			}
52
-			else if ($dbc->rechercherEgalite($nom_table, $champ, $value, $nom_id_table, $value_id_table) == true) {
51
+			} else if ($dbc->rechercherEgalite($nom_table, $champ, $value, $nom_id_table, $value_id_table) == true) {
53 52
 				self::$erreur = true;
54 53
 				return "<li>$err_egalite</li>";
55 54
 			}
Please login to merge, or discard this patch.
core/RedirectError.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -76,8 +76,7 @@
 block discarded – undo
76 76
 
77 77
 			if ($id_page != 0) {
78 78
 				$query = $dbc->query("SELECT balise_title, meta_description, ID_page FROM page WHERE ID_page=".$id_page);
79
-			}
80
-			else {
79
+			} else {
81 80
 				$query = $dbc->query("SELECT balise_title, meta_description, ID_page FROM page WHERE url LIKE '$url'");
82 81
 			}
83 82
 
Please login to merge, or discard this patch.
core/auth/Inscription.php 1 patch
Braces   +9 added lines, -18 removed lines patch added patch discarded remove patch
@@ -20,8 +20,7 @@  discard block
 block discarded – undo
20 20
 
21 21
 				if (isset($datas[$i][2])) {
22 22
 					$this->$function($datas[$i][1], $datas[$i][2]);
23
-				}
24
-				else {
23
+				} else {
25 24
 					$this->$function($datas[$i][1]);
26 25
 				}
27 26
 			}
@@ -54,8 +53,7 @@  discard block
 block discarded – undo
54 53
 		protected function getTestRequired($value) {
55 54
 			if (($value == "") || ($value == null) || (strlen($value) == 0)) {
56 55
 				return false;
57
-			}
58
-			else {
56
+			} else {
59 57
 				return true;
60 58
 			}
61 59
 		}
@@ -68,8 +66,7 @@  discard block
 block discarded – undo
68 66
 		protected function getTestLongueur($value, $longeur) {
69 67
 			if (strlen($value) > $longeur) {
70 68
 				return true;
71
-			}
72
-			else {
69
+			} else {
73 70
 				return false;
74 71
 			}
75 72
 		}
@@ -81,8 +78,7 @@  discard block
 block discarded – undo
81 78
 		protected function getTestInt($value) {
82 79
 			if (is_numeric($value)) {
83 80
 				return true;
84
-			}
85
-			else {
81
+			} else {
86 82
 				return false;
87 83
 			}
88 84
 		}
@@ -100,12 +96,10 @@  discard block
 block discarded – undo
100 96
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
101 97
 				$this->erreur .= "<li>$erreur</li>";
102 98
 				return false;
103
-			}
104
-			else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
99
+			} else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
105 100
 				$this->erreur .= "<li>$erreur_long</li>";
106 101
 				return false;
107
-			}
108
-			else {
102
+			} else {
109 103
 				$this->$varaible = $value;
110 104
 				return true;
111 105
 			}
@@ -153,8 +147,7 @@  discard block
 block discarded – undo
153 147
 		protected function setVerifRetapeMdp($value) {
154 148
 			if ($this->mdp == $value) {
155 149
 				return true;
156
-			}
157
-			else {
150
+			} else {
158 151
 				$this->erreur .= "<li>Les mots de passent ne correspondent pas</li>";
159 152
 				$this->mdp = null;
160 153
 				return false;
@@ -182,12 +175,10 @@  discard block
 block discarded – undo
182 175
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
183 176
 				$this->erreur .= "<li>Le champs E-mail ne peut pas être vide</li>";
184 177
 				return false;
185
-			}
186
-			else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
178
+			} else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
187 179
 				$this->erreur .= "<li>Le champs E-mail doit être une adresse E-mail valide</li>";
188 180
 				return false;
189
-			}
190
-			else {
181
+			} else {
191 182
 				$this->mail = $value;
192 183
 				return true;
193 184
 			}
Please login to merge, or discard this patch.
core/auth/Membre.php 1 patch
Braces   +5 added lines, -10 removed lines patch added patch discarded remove patch
@@ -87,12 +87,10 @@  discard block
 block discarded – undo
87 87
 			if ((strlen($new_pseudo) < 5) || (strlen($new_pseudo) > 15)) {
88 88
 				$err = "Votre pseudo doit être entre 5 et 15 caractères";
89 89
 				$this->erreur = $err;
90
-			}
91
-			else if ($dbc->rechercherEgalite("identite", "pseudo", $new_pseudo) == false) {
90
+			} else if ($dbc->rechercherEgalite("identite", "pseudo", $new_pseudo) == false) {
92 91
 				$err = "Ce pseudo est déjà utilisé, veuillez en choisir un autre";
93 92
 				$this->erreur = $err;
94
-			}
95
-			else {
93
+			} else {
96 94
 				$dbc->query("UPDATE identite set pseudo=$new_pseudo WHERE ID_identite=".$_SESSION["idlogin".CLEF_SITE]);
97 95
 				$this->pseudo = $new_pseudo;
98 96
 			}
@@ -112,16 +110,13 @@  discard block
 block discarded – undo
112 110
 			if (md5($old_mdp) != $mdp) {
113 111
 				$err = "Votre mot de passe est incorrect";
114 112
 				$this->erreur = $err;
115
-			}
116
-			else if ($new_mdp != $verif_new_mdp) {
113
+			} else if ($new_mdp != $verif_new_mdp) {
117 114
 				$err = "Vos mots de passe sont différents";
118 115
 				$this->erreur = $err;
119
-			}
120
-			else if ($this->testpassword($new_mdp) == false) {
116
+			} else if ($this->testpassword($new_mdp) == false) {
121 117
 				$err = "Votre mot de passe est trop simple, il doit contenir 5 caractères et au moins un chiffre";
122 118
 				$this->erreur = $err;
123
-			}
124
-			else {
119
+			} else {
125 120
 				$mdpok = Encrypt::setEncryptMdp($new_mdp, $this->id_identite);
126 121
 				//le nouveau mdp est bon on update
127 122
 				$dbc->query("UPDATE identite SET mdp='$mdpok' WHERE ID_identite=".$this->id_identite);
Please login to merge, or discard this patch.
core/admin/droitsacces/DroitAcces.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -41,25 +41,25 @@  discard block
 block discarded – undo
41 41
     
42 42
     
43 43
 		//-------------------------- GETTER ----------------------------------------------------------------------------//
44
-		public function getLogged(){
44
+		public function getLogged() {
45 45
 			return $this->logged;
46 46
 		}
47
-		public function getSuperAdmin(){
47
+		public function getSuperAdmin() {
48 48
 			return $this->super_admin;
49 49
 		}
50
-		public function getIdListeDroitAcces(){
50
+		public function getIdListeDroitAcces() {
51 51
 			return $this->id_liste_droit_acces;
52 52
 		}
53
-		public function getModifSeo(){
53
+		public function getModifSeo() {
54 54
 			return $this->modif_seo;
55 55
 		}
56
-		public function getModifContenu(){
56
+		public function getModifContenu() {
57 57
 			return $this->modif_contenu;
58 58
 		}
59
-		public function getModifNavigation(){
59
+		public function getModifNavigation() {
60 60
 			return $this->modif_navigation;
61 61
 		}
62
-		public function getSupprimerPage(){
62
+		public function getSupprimerPage() {
63 63
 			return $this->supprimer_page;
64 64
 		}
65 65
 
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
 			//page sans droit dans admin
142 142
 			$all_access = array("gestion-comptes/mon-compte", "index");
143 143
 
144
-			if (($this->super_admin == 1) || (in_array($this->getListeDroitPage($page), $this->getListeDroitAcces()))|| (in_array($page, $all_access))) {
144
+			if (($this->super_admin == 1) || (in_array($this->getListeDroitPage($page), $this->getListeDroitAcces())) || (in_array($page, $all_access))) {
145 145
 				return true;
146 146
 			}
147 147
 			else {
Please login to merge, or discard this patch.
Braces   +9 added lines, -8 removed lines patch added patch discarded remove patch
@@ -78,7 +78,9 @@  discard block
 block discarded – undo
78 78
 				");
79 79
 
80 80
 			if ((is_array($query)) && (count($query) > 0)) {
81
-				foreach ($query as $obj) $liste_droit_acces[] = $obj->droit_acces;
81
+				foreach ($query as $obj) {
82
+					$liste_droit_acces[] = $obj->droit_acces;
83
+				}
82 84
 			}
83 85
 
84 86
 			return $liste_droit_acces;
@@ -93,7 +95,9 @@  discard block
 block discarded – undo
93 95
 
94 96
 			$query = $dbc->query("SELECT droit_acces FROM droit_acces WHERE page LIKE '%$page%'");
95 97
 			if ((is_array($query)) && (count($query) > 0)) {
96
-				foreach ($query as $obj) $droit_acces = $obj->droit_acces;
98
+				foreach ($query as $obj) {
99
+					$droit_acces = $obj->droit_acces;
100
+				}
97 101
 			}
98 102
 
99 103
 			return $droit_acces;
@@ -143,8 +147,7 @@  discard block
 block discarded – undo
143 147
 
144 148
 			if (($this->super_admin == 1) || (in_array($this->getListeDroitPage($page), $this->getListeDroitAcces()))|| (in_array($page, $all_access))) {
145 149
 				return true;
146
-			}
147
-			else {
150
+			} else {
148 151
 				return false;
149 152
 			}
150 153
 		}
@@ -169,8 +172,7 @@  discard block
 block discarded – undo
169 172
 			//si les trois sont différent de 0 on renvoit true soinon false
170 173
 			if (($this->super_admin == 1) || ((in_array($droit, $liste_droit_acces)) && in_array(1, $array_modif))) {
171 174
 				return true;
172
-			}
173
-			else {
175
+			} else {
174 176
 				return false;
175 177
 			}
176 178
 		}
@@ -185,8 +187,7 @@  discard block
 block discarded – undo
185 187
 
186 188
 			if (($this->super_admin == 1) || (in_array($droit_acces, $liste_droit_acces))) {
187 189
 				return true;
188
-			}
189
-			else {
190
+			} else {
190 191
 				return false;
191 192
 			}
192 193
 		}
Please login to merge, or discard this patch.
core/admin/Admin.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,8 +21,7 @@  discard block
 block discarded – undo
21 21
 				foreach ($query as $obj) {
22 22
 					$params = $obj->mdp_params;
23 23
 				}
24
-			}
25
-			else {
24
+			} else {
26 25
 				$params = self::$params;
27 26
 			}
28 27
 
@@ -74,8 +73,7 @@  discard block
 block discarded – undo
74 73
 
75 74
 			if ($id_identite != null) {
76 75
 				self::setSaveParams("$taille_bout1, $debut_bout2, $nb_aleatoire_mdp, $taille_bout2, $debut_bout1, ".ChaineCaractere::random(20), $id_identite);
77
-			}
78
-			else {
76
+			} else {
79 77
 				self::$params = "$taille_bout1, $debut_bout2, $nb_aleatoire_mdp, $taille_bout2, $debut_bout1, ".ChaineCaractere::random(20);
80 78
 			}
81 79
 
Please login to merge, or discard this patch.
core/admin/droitsacces/GestionDroitAcces.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -76,8 +76,7 @@
 block discarded – undo
76 76
 
77 77
 			if ($id_page != 0) {
78 78
 				$query = $dbc->query("SELECT balise_title, meta_description, ID_page FROM page WHERE ID_page=".$id_page);
79
-			}
80
-			else {
79
+			} else {
81 80
 				$query = $dbc->query("SELECT balise_title, meta_description, ID_page FROM page WHERE url LIKE '$url'");
82 81
 			}
83 82
 
Please login to merge, or discard this patch.
core/admin/contenus/GestionContenus.php 4 patches
Doc Comments   +12 added lines patch added patch discarded remove patch
@@ -71,6 +71,14 @@  discard block
 block discarded – undo
71 71
 
72 72
 
73 73
 		//-------------------------- SETTER ----------------------------------------------------------------------------//
74
+
75
+		/**
76
+		 * @param string $url
77
+		 * @param string $err_balise_title
78
+		 * @param string $err_url
79
+		 * @param string $err_meta_description
80
+		 * @param string $err_titre_page
81
+		 */
74 82
 		private function setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $contenu, $err_balise_title, $err_url, $err_meta_description, $err_titre_page) {
75 83
 			$_SESSION['balise_title'] = $balise_title;
76 84
 			$_SESSION['url'] = $url;
@@ -265,6 +273,10 @@  discard block
 block discarded – undo
265 273
 			}
266 274
 		}
267 275
 
276
+		/**
277
+		 * @param string $id
278
+		 * @param integer $affiche
279
+		 */
268 280
 		private function setModifierLienNavigation($id, $id_page, $parent, $affiche) {
269 281
 			if ($parent != "") {
270 282
 				$nav = new Navigation();
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 
4 4
 	use core\App;
5 5
 	use core\contenus\Contenus;
6
-	use core\admin\droitsacces\DroitAcces;
7 6
 	use core\functions\ChaineCaractere;
8 7
 	use core\HTML\flashmessage\FlashMessage;
9 8
 	use core\Navigation;
Please login to merge, or discard this patch.
Braces   +8 added lines, -11 removed lines patch added patch discarded remove patch
@@ -31,7 +31,9 @@  discard block
 block discarded – undo
31 31
 
32 32
 			$query = $dbc->query("SELECT titre FROM page WHERE ID_page=".$parent);
33 33
 			if ((is_array($query)) && (count($query) > 0)) {
34
-				foreach ($query as $obj) $this->parent_texte = $obj->titre;
34
+				foreach ($query as $obj) {
35
+					$this->parent_texte = $obj->titre;
36
+				}
35 37
 			}
36 38
 
37 39
 			return $this->parent_texte;
@@ -137,13 +139,11 @@  discard block
 block discarded – undo
137 139
 					$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);
138 140
 					$this->id_page = $dbc->lastInsertId();
139 141
 					$this->setAjoutLienNavigation("ID_page", $this->id_page , 1);
140
-				}
141
-				else {
142
+				} else {
142 143
 					FlashMessage::setFlash("Impossible de créer cette page, veuillez réeseyer dans un moment. Si le problème persiste contactez votre administrateur.");
143 144
 					$this->erreur = true;
144 145
 				}
145
-			}
146
-			else {
146
+			} else {
147 147
 				$this->setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $contenu, $err_balise_title, $err_url, $err_meta_description, $err_titre_page);
148 148
 			}
149 149
 		}
@@ -207,8 +207,7 @@  discard block
 block discarded – undo
207 207
 
208 208
 					$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);
209 209
 					$this->setModifierLienNavigation("ID_page", $id_page, $this->getParentId($parent), $affiche);
210
-				}
211
-				else {
210
+				} else {
212 211
 					$this->setErreurContenus($balise_title, $url, $meta_description, $titre_page, $parent, $contenu, $err_balise_title, $err_url, $err_meta_description, $err_titre_page);
213 212
 				}
214 213
 			}
@@ -251,8 +250,7 @@  discard block
 block discarded – undo
251 250
 					FlashMessage::setFlash("Impossible de supprimer cette page, veuillez contacter votre administrateur pour corriger ce problème");
252 251
 					$this->erreur = true;
253 252
 				}
254
-			}
255
-			else {
253
+			} else {
256 254
 				FlashMessage::setFlash("Impossible de supprimer cette page, veuillez contacter votre administrateur pour corriger ce problème");
257 255
 				$this->erreur = true;
258 256
 			}
@@ -269,8 +267,7 @@  discard block
 block discarded – undo
269 267
 			if ($parent != "") {
270 268
 				$nav = new Navigation();
271 269
 				$nav->setSupprimerLien($id, $id_page);
272
-			}
273
-			else if (($affiche == 1) && ($parent == "")) {
270
+			} else if (($affiche == 1) && ($parent == "")) {
274 271
 				$this->setAjoutLienNavigation($id, $id_page, $affiche);
275 272
 			}
276 273
 		}
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
 						->set();
137 137
 
138 138
 					$this->id_page = $dbc->lastInsertId();
139
-					$this->setAjoutLienNavigation("ID_page", $this->id_page , 1);
139
+					$this->setAjoutLienNavigation("ID_page", $this->id_page, 1);
140 140
 				}
141 141
 				else {
142 142
 					FlashMessage::setFlash("Impossible de créer cette page, veuillez réeseyer dans un moment. Si le problème persiste contactez votre administrateur.");
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
 						->update("parent", $parent)
204 204
 						->update("affiche", $affiche)
205 205
 						->from("page")
206
-						->where("ID_page", "=", $id_page,"",true)
206
+						->where("ID_page", "=", $id_page, "", true)
207 207
 						->set();
208 208
 
209 209
 					$this->setModifierLienNavigation("ID_page", $id_page, $this->getParentId($parent), $affiche);
Please login to merge, or discard this patch.