Passed
Push — master ( 053c59...ec0ca6 )
by Anthony
06:21 queued 02:59
created
core/admin/Admin.class.php 1 patch
Braces   +3 added lines, -6 removed lines patch added patch discarded remove patch
@@ -51,8 +51,7 @@  discard block
 block discarded – undo
51 51
 
52 52
 			if ($archiver == null) {
53 53
 				$query = $dbc->query("SELECT * FROM identite WHERE archiver IS NULL AND ID_identite > 1");
54
-			}
55
-			else {
54
+			} else {
56 55
 				$query = $dbc->query("SELECT * FROM identite WHERE archiver IS NOT NULL AND ID_identite > 1");
57 56
 			}
58 57
 
@@ -77,8 +76,7 @@  discard block
 block discarded – undo
77 76
 
78 77
 					if (($config->getValiderInscription() == 1) && ($obj->valide == 0)) {
79 78
 						$valide[] = "<a href=".ADMWEBROOT."controller/core/admin/comptes/valider_compte?id_identite=$obj->ID_identite>Valider cet utilisateur</a>";
80
-					}
81
-					else {
79
+					} else {
82 80
 						$valide[] = "Utilisateur validé";
83 81
 					}
84 82
 				}
@@ -181,8 +179,7 @@  discard block
 block discarded – undo
181 179
 
182 180
 				$mail = new Mail($this->mail);
183 181
 				$mail->setEnvoyerMail("Réinitialisation de votre E-mail effectuée", "Votre mot de passe a été réinitialisé");
184
-			}
185
-			else {
182
+			} else {
186 183
 				FlashMessage::setFlash("le mot de passe de $this->pseudo ne peu pas être réinitialisé car il ne possède pas d'E-mail");
187 184
 				$this->erreur = true;
188 185
 			}
Please login to merge, or discard this patch.
core/auth/Connexion.class.php 2 patches
Braces   +8 added lines, -11 removed lines patch added patch discarded remove patch
@@ -49,8 +49,7 @@  discard block
 block discarded – undo
49 49
 			if (!isset($id)) {
50 50
 				FlashMessage::setFlash("Vos identifiants de connexions sont incorrects");
51 51
 				header("location:$page_retour_err");
52
-			}
53
-			else {
52
+			} else {
54 53
 				$config = new Configuration();
55 54
 
56 55
 				//cela veut dire que l'utilisateur doit obligatoirement etre valider pour avoir acces au site
@@ -75,8 +74,7 @@  discard block
 block discarded – undo
75 74
 
76 75
 					FlashMessage::setFlash("Vous êtes maintenant connecté", "info");
77 76
 					header("location:$page_retour");
78
-				}
79
-				else {
77
+				} else {
80 78
 					FlashMessage::setFlash("Vos identifiants de connexions sont incorrects");
81 79
 					header("location:$page_retour_err");
82 80
 				}
@@ -104,8 +102,7 @@  discard block
 block discarded – undo
104 102
 						if ($obj->archiver == 1) {
105 103
 							setcookie("auth".CLEF_SITE, NULL, -1);
106 104
 							self::setDeconnexion($page_retour);
107
-						}
108
-						else {
105
+						} else {
109 106
 							$key = sha1($obj->pseudo.$obj->mdp);
110 107
 
111 108
 							if ($key == $auth[1]) {
@@ -113,15 +110,13 @@  discard block
 block discarded – undo
113 110
 								$_SESSION["idlogin".CLEF_SITE] = $obj->ID_identite;
114 111
 
115 112
 								setcookie("auth".CLEF_SITE, $obj->ID_identite."-----".$key, time() + 3600 * 24 * 3, "/", "", false, true);
116
-							}
117
-							else if ($obj_connecte == 1) {
113
+							} else if ($obj_connecte == 1) {
118 114
 								self::setDeconnexion($page_retour);
119 115
 							}
120 116
 						}
121 117
 					}
122 118
 				}
123
-			}
124
-			else if ((!isset($_SESSION["idlogin".CLEF_SITE])) && ($obj_connecte == 1)) {
119
+			} else if ((!isset($_SESSION["idlogin".CLEF_SITE])) && ($obj_connecte == 1)) {
125 120
 				FlashMessage::setFlash("Vous devez être connecté pour accéder à cette page");
126 121
 				header("location:".$page_retour);
127 122
 			}
@@ -167,7 +162,9 @@  discard block
 block discarded – undo
167 162
 
168 163
 			$query = $dbc->query("SELECT last_change_mdp FROM identite WHERE ID_identite=".$id_identite);
169 164
 			if ((is_array($query)) && (count($query) > 0)) {
170
-				foreach ($query as $obj) return $obj->last_change_mdp;
165
+				foreach ($query as $obj) {
166
+					return $obj->last_change_mdp;
167
+				}
171 168
 			}
172 169
 		}
173 170
 
Please login to merge, or discard this patch.
Doc Comments   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 		/**
18 18
 		 * @param $valide
19 19
 		 * @param $archiver
20
-		 * @param $page_retour_err
20
+		 * @param string $page_retour_err
21 21
 		 */
22 22
 		private function setTestParamCompte($valide, $archiver, $page_retour_err) {
23 23
 			$config = new Configuration();
@@ -181,7 +181,8 @@  discard block
 block discarded – undo
181 181
 
182 182
 		/**
183 183
 		 * @param $id_identite
184
-		 * @param null $remember
184
+		 * @param integer $remember
185
+		 * @param string $mdp_nonencrypt_tape
185 186
 		 */
186 187
 		private static function setTestChangerMdp($id_identite, $mdp_nonencrypt_tape, $remember) {
187 188
 			$membre = new Membre($id_identite);
Please login to merge, or discard this patch.
core/HTML/flashmessage/FlashMessage.class.php 1 patch
Braces   +3 added lines, -6 removed lines patch added patch discarded remove patch
@@ -17,11 +17,9 @@  discard block
 block discarded – undo
17 17
 
18 18
 			if ($type == "error") {
19 19
 				$icone = "<i class='fa fa-close'></i>";
20
-			}
21
-			else if ($type == "success") {
20
+			} else if ($type == "success") {
22 21
 				$icone = "<i class='fa fa-check'></i>";
23
-			}
24
-			else {
22
+			} else {
25 23
 				$icone = "<i class='fa fa-info'></i>";
26 24
 			}
27 25
 
@@ -43,8 +41,7 @@  discard block
 block discarded – undo
43 41
 				if (strstr($_SERVER['SCRIPT_NAME'], "index.php")) {
44 42
 					//definit le chemin sachant que l'on part de index.php
45 43
 					$chemin = str_replace("\\", "/", str_replace("index.php", "", $_SERVER['SCRIPT_NAME']).__NAMESPACE__."/view/");
46
-				}
47
-				else {
44
+				} else {
48 45
 					//definit le chemin sachant que l'on part de admin.php
49 46
 					$chemin = str_replace("\\", "/", str_replace("admin.php", "", $_SERVER['SCRIPT_NAME']).__NAMESPACE__."/view/");
50 47
 				}
Please login to merge, or discard this patch.
core/RedirectError.class.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/Database.class.php 1 patch
Braces   +8 added lines, -11 removed lines patch added patch discarded remove patch
@@ -51,8 +51,7 @@  discard block
 block discarded – undo
51 51
 			if ($query) {
52 52
 				$obj = $query->fetchAll(PDO::FETCH_OBJ);
53 53
 				return $obj;
54
-			}
55
-			else {
54
+			} else {
56 55
 				FlashMessage::setFlash("Une erreur est survenue en executant cette requette : ".$req);
57 56
 			}
58 57
 		}
@@ -71,8 +70,7 @@  discard block
 block discarded – undo
71 70
 						$err = true;
72 71
 					}
73 72
 				}
74
-			}
75
-			else {
73
+			} else {
76 74
 				if (!$query->execute($value)) {
77 75
 					$err = true;
78 76
 				}
@@ -94,8 +92,7 @@  discard block
 block discarded – undo
94 92
 		public function rechercherEgalite($table, $champ, $value, $id_table = null, $id = null) {
95 93
 			if ($id == null) {
96 94
 				$query = $this->getPdo()->query("SELECT COUNT($champ) as nb FROM $table WHERE $champ LIKE '$value'");
97
-			}
98
-			else {
95
+			} else {
99 96
 				$query = $this->getPdo()->query("SELECT COUNT($champ) as nb FROM $table WHERE $champ LIKE '$value' AND $id_table != $id");
100 97
 			}
101 98
 
@@ -104,9 +101,10 @@  discard block
 block discarded – undo
104 101
 					$nb = $obj["nb"];
105 102
 				}
106 103
 
107
-				if ((isset($nb)) && ($nb != 0)) return true;
108
-			}
109
-			else {
104
+				if ((isset($nb)) && ($nb != 0)) {
105
+					return true;
106
+				}
107
+			} else {
110 108
 				return false;
111 109
 			}
112 110
 		}
@@ -123,8 +121,7 @@  discard block
 block discarded – undo
123 121
 
124 122
 			if ($query->rowCount() > 0) {
125 123
 				return true;
126
-			}
127
-			else {
124
+			} else {
128 125
 				return false;
129 126
 			}
130 127
 		}
Please login to merge, or discard this patch.
core/images/Images.class.php 1 patch
Braces   +7 added lines, -15 removed lines patch added patch discarded remove patch
@@ -58,8 +58,7 @@  discard block
 block discarded – undo
58 58
 				$explode = explode("/", $this->chemin_image);
59 59
 				$this->image = end($explode);
60 60
 				return $this->image;
61
-			}
62
-			else {
61
+			} else {
63 62
 				$this->erreur = "Impossible de trouver votre image, vuellez réessayer dans un instant";
64 63
 				return false;
65 64
 			}
@@ -87,8 +86,7 @@  discard block
 block discarded – undo
87 86
 			if ((empty($_FILES[$name]['name'])) && ($autorize_empty == 0)) {
88 87
 				$this->erreur = "Vous devez obligatoirement ajouter une image";
89 88
 				return false;
90
-			}
91
-			else {
89
+			} else {
92 90
 				//test si il y a deja une img
93 91
 				if ($old_image_req != null) {
94 92
 					$query = $dbc->query($old_image_req);
@@ -106,9 +104,7 @@  discard block
 block discarded – undo
106 104
 				if ((!in_array(substr($image, -3), $this->autorized_extention)) || (($infos_img[0] >= $this->width_max) && ($infos_img[1] >= $this->height_max) && ($_FILES[$name]['size'] >= $this->poid_max))) {
107 105
 					$this->erreur = "Problème dans les dimensions, taille ou format (extension) de l'image.";
108 106
 					return false;
109
-				}
110
-
111
-				else if (move_uploaded_file($_FILES[$name]['tmp_name'], $this->dossier_image."/".$uniqid.substr($image, -4))) {
107
+				} else if (move_uploaded_file($_FILES[$name]['tmp_name'], $this->dossier_image."/".$uniqid.substr($image, -4))) {
112 108
 					$imageok = $uniqid.substr($image, -4);
113 109
 
114 110
 					$urlimg = $this->dossier_image."/$imageok";
@@ -120,8 +116,7 @@  discard block
 block discarded – undo
120 116
 					}
121 117
 
122 118
 					return true;
123
-				}
124
-				else {
119
+				} else {
125 120
 					$this->erreur = "Impossible d'envoyer votre image sur le serveur, veuillez réessayer dans une instant, si l'erreur se reproduit, contactez votre administrateur";
126 121
 				}
127 122
 			}
@@ -146,8 +141,7 @@  discard block
 block discarded – undo
146 141
 				$resize->saveImage($this->dossier_image."/".$img_resize, 100);
147 142
 
148 143
 				$this->nom_image = $img_resize;
149
-			}
150
-			else {
144
+			} else {
151 145
 				$this->nom_image = null;
152 146
 			}
153 147
 
@@ -171,11 +165,9 @@  discard block
 block discarded – undo
171 165
 					unlink($this->dossier_image."/".end($old_image));
172 166
 					return true;
173 167
 				}
174
-			}
175
-			else if (($nom_image !== null) && (unlink($this->dossier_image."/".$nom_image))) {
168
+			} else if (($nom_image !== null) && (unlink($this->dossier_image."/".$nom_image))) {
176 169
 				return true;
177
-			}
178
-			else {
170
+			} else {
179 171
 				$this->erreur = "Impossible de supprimer cette image, veuillez réesayer dans un instant, sinon contacter l'administrateur de votre site";
180 172
 				return false;
181 173
 			}
Please login to merge, or discard this patch.
core/modules/GestionModule.class.php 1 patch
Braces   +6 added lines, -12 removed lines patch added patch discarded remove patch
@@ -140,8 +140,7 @@  discard block
 block discarded – undo
140 140
 				}
141 141
 
142 142
 				return $installer;
143
-			}
144
-			else {
143
+			} else {
145 144
 				return false;
146 145
 			}
147 146
 		}
@@ -160,8 +159,7 @@  discard block
 block discarded – undo
160 159
 				foreach ($query as $obj) {
161 160
 					if ($obj->activer == 1) {
162 161
 						return true;
163
-					}
164
-					else {
162
+					} else {
165 163
 						return false;
166 164
 					}
167 165
 				}
@@ -183,8 +181,7 @@  discard block
 block discarded – undo
183 181
 				foreach ($query as $obj) {
184 182
 					if ($obj->mettre_jour == 1) {
185 183
 						return false;
186
-					}
187
-					else {
184
+					} else {
188 185
 						return true;
189 186
 					}
190 187
 				}
@@ -208,8 +205,7 @@  discard block
 block discarded – undo
208 205
 						//si pas de version a checker, cela veut dire qu'on vient d'installer le module
209 206
 						//du coup on met le next_check aa la semaine pro
210 207
 						$set_next = true;
211
-					}
212
-					else if (($obj->next_check_version <= $today) && ($obj->mettre_jour != 1)) {
208
+					} else if (($obj->next_check_version <= $today) && ($obj->mettre_jour != 1)) {
213 209
 						//avant tout on regarde si on doit delete une vieille version
214 210
 						if ($obj->delete_old_version == 1) {
215 211
 							$import = new ImportModule();
@@ -246,8 +242,7 @@  discard block
 block discarded – undo
246 242
 								$dbc->prepare("UPDATE module SET mettre_jour=:update, online_version=:online_version WHERE ID_module=:id_module", $value);
247 243
 
248 244
 								$set_next = true;
249
-							}
250
-							else {
245
+							} else {
251 246
 								$set_next = true;
252 247
 							}
253 248
 						}
@@ -284,8 +279,7 @@  discard block
 block discarded – undo
284 279
 				$this->setListeModuleMettreJour($nom_module, $version, $online_version);
285 280
 
286 281
 				return true;
287
-			}
288
-			else {
282
+			} else {
289 283
 				return false;
290 284
 			}
291 285
 		}
Please login to merge, or discard this patch.
core/auth/Membre.class.php 1 patch
Braces   +8 added lines, -16 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
 			}
@@ -114,16 +112,13 @@  discard block
 block discarded – undo
114 112
 			if (md5($old_mdp) != $mdp) {
115 113
 				$err = "Votre mot de passe est incorrect";
116 114
 				$this->erreur = $err;
117
-			}
118
-			else if ($new_mdp != $verif_new_mdp) {
115
+			} else if ($new_mdp != $verif_new_mdp) {
119 116
 				$err = "Vos mots de passe sont différents";
120 117
 				$this->erreur = $err;
121
-			}
122
-			else if ((strlen($new_mdp) < 5) || ($testmdp < 40)) {
118
+			} else if ((strlen($new_mdp) < 5) || ($testmdp < 40)) {
123 119
 				$err = "Votre mot de passe est trop simple";
124 120
 				$this->erreur = $err;
125
-			}
126
-			else {
121
+			} else {
127 122
 				$mdpok = Encrypt::setEncryptMdp($new_mdp, $this->id_identite);
128 123
 				//le nouveau mdp est bon on update
129 124
 				$dbc->query("UPDATE identite SET mdp='$mdpok' WHERE ID_identite=".$this->id_identite);
@@ -155,16 +150,13 @@  discard block
 block discarded – undo
155 150
 				if ($lettre >= 'a' && $lettre <= 'z') {
156 151
 					$point = $point + 1;
157 152
 					$point_min = 1;
158
-				}
159
-				else if ($lettre >= 'A' && $lettre <= 'Z') {
153
+				} else if ($lettre >= 'A' && $lettre <= 'Z') {
160 154
 					$point = $point + 2;
161 155
 					$point_maj = 2;
162
-				}
163
-				else if ($lettre >= '0' && $lettre <= '9') {
156
+				} else if ($lettre >= '0' && $lettre <= '9') {
164 157
 					$point = $point + 3;
165 158
 					$point_chiffre = 3;
166
-				}
167
-				else {
159
+				} else {
168 160
 					$point = $point + 5;
169 161
 					$point_caracteres = 5;
170 162
 				}
Please login to merge, or discard this patch.
core/auth/Inscription.class.php 1 patch
Braces   +11 added lines, -22 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,17 +96,14 @@  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 {
99
+			} else {
105 100
 				if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
106 101
 					$this->$varaible = $value;
107 102
 					return true;
108
-				}
109
-				else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
103
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
110 104
 					$this->erreur .= "<li>$erreur_long</li>";
111 105
 					return false;
112
-				}
113
-				else {
106
+				} else {
114 107
 					return true;
115 108
 				}
116 109
 			}
@@ -158,8 +151,7 @@  discard block
 block discarded – undo
158 151
 		protected function setVerifRetapeMdp($value) {
159 152
 			if ($this->mdp == $value) {
160 153
 				return true;
161
-			}
162
-			else {
154
+			} else {
163 155
 				$this->erreur .= "<li>Les mots de passent ne correspondent pas</li>";
164 156
 				$this->mdp = null;
165 157
 				return false;
@@ -187,17 +179,14 @@  discard block
 block discarded – undo
187 179
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
188 180
 				$this->erreur .= "<li>Le champs E-mail ne peut pas être vide</li>";
189 181
 				return false;
190
-			}
191
-			else {
182
+			} else {
192 183
 				if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) != false)) {
193 184
 					$this->mail = $value;
194 185
 					return true;
195
-				}
196
-				else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
186
+				} else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
197 187
 					$this->erreur .= "<li>Le champs E-mail doit être une adresse E-mail valide</li>";
198 188
 					return false;
199
-				}
200
-				else {
189
+				} else {
201 190
 					return true;
202 191
 				}
203 192
 			}
Please login to merge, or discard this patch.