Inspection of "lots of correction bug and coding style" - Piou-piou/ribs-framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 8543ab...7db5d1 )

by Anthony

created 2016-02-26 17:58 UTC

Status

Braces +20 added lines, -40 removed lines patch added patch discarded remove patch

@@ -19,8 +19,7 @@  discard block
 block discarded – undo
 
 				if (isset($datas[$i][2])) {
 					$this->$function($datas[$i][1], $datas[$i][2]);
-				}
-				else {
+				} else {
 					$this->$function($datas[$i][1]);
 				}
 			}
@@ -53,8 +52,7 @@  discard block
 block discarded – undo
 		protected function getTestRequired($value) {
 			if (($value == "") || ($value == null) || (strlen($value) == 0)) {
 				return false;
-			}
-			else {
+			} else {
 				return true;
 			}
 		}
@@ -67,8 +65,7 @@  discard block
 block discarded – undo
 		protected function getTestLongueur($value, $longeur) {
 			if (strlen($value) > $longeur) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -80,8 +77,7 @@  discard block
 block discarded – undo
 		protected function getTestInt($value) {
 			if (is_numeric($value)) {
 				return true;
-			}
-			else {
+			} else {
 				return false;
 			}
 		}
@@ -101,17 +97,14 @@  discard block
 block discarded – undo
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>Le champs nom ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
 					$this->nom = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
 					$this->erreur .= "<li>Le champs nom soit être supérieur à deux caractères</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -128,17 +121,14 @@  discard block
 block discarded – undo
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>Le champs prénom ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
 					$this->prenom = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
 					$this->erreur .= "<li>Le champs prénom soit être supérieur à deux caractères</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -155,17 +145,14 @@  discard block
 block discarded – undo
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>Le champs mot de passe ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
 					$this->mdp = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
 					$this->erreur .= "<li>Le champs mot de passe soit être supérieur à deux caractères</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -180,8 +167,7 @@  discard block
 block discarded – undo
 		protected function setVerifRetapeMdp($value, $required=null) {
 			if ($this->mdp == $value) {
 				return true;
-			}
-			else {
+			} else {
 				$this->erreur .= "<li>Les mots de passent ne correspondent pas</li>";
 				$this->mdp = null;
 				return false;
@@ -199,17 +185,14 @@  discard block
 block discarded – undo
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>Le champs pseudo ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
 					$this->pseudo = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
+				} else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
 					$this->erreur .= "<li>Le champs pseudo soit être supérieur à deux caractères</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}
@@ -226,17 +209,14 @@  discard block
 block discarded – undo
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>Le champs E-mail ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) != false)) {
 					$this->mail = $value;
 					return true;
-				}
-				else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
+				} else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
 					$this->erreur .= "<li>Le champs E-mail doit être une adresse E-mail valide</li>";
 					return false;
-				}
-				else {
+				} else {
 					return true;
 				}
 			}

Please login to merge, or discard this patch.

core/admin/inscription/AdminInscription.class.php 1 patch

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

@@ -36,17 +36,14 @@  discard block
 block discarded – undo
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>Le champs accès administration ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestInt($value) === true)) {
 					$this->acces_administration = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestInt($value) === false)) {
+				} else if (($value != "") && ($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;
 				}
 			}
@@ -64,17 +61,14 @@  discard block
 block discarded – undo
 			if (($required != null) && ($this->getTestRequired($value) === false)) {
 				$this->erreur .= "<li>Le champs accès administration ne peut pas être vide</li>";
 				return false;
-			}
-			else {
+			} else {
 				if (($value != "") && ($this->getTestInt($value) === true)) {
 					$this->id_liste_droit_acces = $value;
 					return true;
-				}
-				else if (($value != "") && ($this->getTestInt($value) === false)) {
+				} else if (($value != "") && ($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;
 				}
 			}

Please login to merge, or discard this patch.

		@@ -19,8 +19,7 @@ discard block
		block discarded – undo
19	19
20	20	if (isset($datas[$i][2])) {
21	21	$this->$function($datas[$i][1], $datas[$i][2]);
22		- }
23		- else {
	22	+ } else {
24	23	$this->$function($datas[$i][1]);
25	24	}
26	25	}
		@@ -53,8 +52,7 @@ discard block
		block discarded – undo
53	52	protected function getTestRequired($value) {
54	53	if (($value == "") \|\| ($value == null) \|\| (strlen($value) == 0)) {
55	54	return false;
56		- }
57		- else {
	55	+ } else {
58	56	return true;
59	57	}
60	58	}
		@@ -67,8 +65,7 @@ discard block
		block discarded – undo
67	65	protected function getTestLongueur($value, $longeur) {
68	66	if (strlen($value) > $longeur) {
69	67	return true;
70		- }
71		- else {
	68	+ } else {
72	69	return false;
73	70	}
74	71	}
		@@ -80,8 +77,7 @@ discard block
		block discarded – undo
80	77	protected function getTestInt($value) {
81	78	if (is_numeric($value)) {
82	79	return true;
83		- }
84		- else {
	80	+ } else {
85	81	return false;
86	82	}
87	83	}
		@@ -101,17 +97,14 @@ discard block
		block discarded – undo
101	97	if (($required != null) && ($this->getTestRequired($value) === false)) {
102	98	$this->erreur .= "<li>Le champs nom ne peut pas être vide</li>";
103	99	return false;
104		- }
105		- else {
	100	+ } else {
106	101	if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
107	102	$this->nom = $value;
108	103	return true;
109		- }
110		- else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
	104	+ } else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
111	105	$this->erreur .= "<li>Le champs nom soit être supérieur à deux caractères</li>";
112	106	return false;
113		- }
114		- else {
	107	+ } else {
115	108	return true;
116	109	}
117	110	}
		@@ -128,17 +121,14 @@ discard block
		block discarded – undo
128	121	if (($required != null) && ($this->getTestRequired($value) === false)) {
129	122	$this->erreur .= "<li>Le champs prénom ne peut pas être vide</li>";
130	123	return false;
131		- }
132		- else {
	124	+ } else {
133	125	if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
134	126	$this->prenom = $value;
135	127	return true;
136		- }
137		- else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
	128	+ } else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
138	129	$this->erreur .= "<li>Le champs prénom soit être supérieur à deux caractères</li>";
139	130	return false;
140		- }
141		- else {
	131	+ } else {
142	132	return true;
143	133	}
144	134	}
		@@ -155,17 +145,14 @@ discard block
		block discarded – undo
155	145	if (($required != null) && ($this->getTestRequired($value) === false)) {
156	146	$this->erreur .= "<li>Le champs mot de passe ne peut pas être vide</li>";
157	147	return false;
158		- }
159		- else {
	148	+ } else {
160	149	if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
161	150	$this->mdp = $value;
162	151	return true;
163		- }
164		- else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
	152	+ } else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
165	153	$this->erreur .= "<li>Le champs mot de passe soit être supérieur à deux caractères</li>";
166	154	return false;
167		- }
168		- else {
	155	+ } else {
169	156	return true;
170	157	}
171	158	}
		@@ -180,8 +167,7 @@ discard block
		block discarded – undo
180	167	protected function setVerifRetapeMdp($value, $required=null) {
181	168	if ($this->mdp == $value) {
182	169	return true;
183		- }
184		- else {
	170	+ } else {
185	171	$this->erreur .= "<li>Les mots de passent ne correspondent pas</li>";
186	172	$this->mdp = null;
187	173	return false;
		@@ -199,17 +185,14 @@ discard block
		block discarded – undo
199	185	if (($required != null) && ($this->getTestRequired($value) === false)) {
200	186	$this->erreur .= "<li>Le champs pseudo ne peut pas être vide</li>";
201	187	return false;
202		- }
203		- else {
	188	+ } else {
204	189	if (($value != "") && ($this->getTestLongueur($value, 2) === true)) {
205	190	$this->pseudo = $value;
206	191	return true;
207		- }
208		- else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
	192	+ } else if (($value != "") && ($this->getTestLongueur($value, 2) === false)) {
209	193	$this->erreur .= "<li>Le champs pseudo soit être supérieur à deux caractères</li>";
210	194	return false;
211		- }
212		- else {
	195	+ } else {
213	196	return true;
214	197	}
215	198	}
		@@ -226,17 +209,14 @@ discard block
		block discarded – undo
226	209	if (($required != null) && ($this->getTestRequired($value) === false)) {
227	210	$this->erreur .= "<li>Le champs E-mail ne peut pas être vide</li>";
228	211	return false;
229		- }
230		- else {
	212	+ } else {
231	213	if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) != false)) {
232	214	$this->mail = $value;
233	215	return true;
234		- }
235		- else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
	216	+ } else if (($value != "") && (filter_var($value, FILTER_VALIDATE_EMAIL) == false)) {
236	217	$this->erreur .= "<li>Le champs E-mail doit être une adresse E-mail valide</li>";
237	218	return false;
238		- }
239		- else {
	219	+ } else {
240	220	return true;
241	221	}
242	222	}

		@@ -36,17 +36,14 @@ discard block
		block discarded – undo
36	36	if (($required != null) && ($this->getTestRequired($value) === false)) {
37	37	$this->erreur .= "<li>Le champs accès administration ne peut pas être vide</li>";
38	38	return false;
39		- }
40		- else {
	39	+ } else {
41	40	if (($value != "") && ($this->getTestInt($value) === true)) {
42	41	$this->acces_administration = $value;
43	42	return true;
44		- }
45		- else if (($value != "") && ($this->getTestInt($value) === false)) {
	43	+ } else if (($value != "") && ($this->getTestInt($value) === false)) {
46	44	$this->erreur .= "<li>Le champs accès administration n'est pas au bon format</li>";
47	45	return false;
48		- }
49		- else {
	46	+ } else {
50	47	return true;
51	48	}
52	49	}
		@@ -64,17 +61,14 @@ discard block
		block discarded – undo
64	61	if (($required != null) && ($this->getTestRequired($value) === false)) {
65	62	$this->erreur .= "<li>Le champs accès administration ne peut pas être vide</li>";
66	63	return false;
67		- }
68		- else {
	64	+ } else {
69	65	if (($value != "") && ($this->getTestInt($value) === true)) {
70	66	$this->id_liste_droit_acces = $value;
71	67	return true;
72		- }
73		- else if (($value != "") && ($this->getTestInt($value) === false)) {
	68	+ } else if (($value != "") && ($this->getTestInt($value) === false)) {
74	69	$this->erreur .= "<li>Le champs accès administration n'est pas au bon format</li>";
75	70	return false;
76		- }
77		- else {
	71	+ } else {
78	72	return true;
79	73	}
80	74	}

Piou-piou / ribs-framework

Push — master ( 8543ab...7db5d1 )

Status

Category

Braces +20 added lines, -40 removed lines patch added patch discarded remove patch

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