Inspection of "merge master" - thinktomorrow/chief - Measure and Improve Code Quality continuously with Scrutinizer

chief

Test Setup Failed

Push — ft/sitemap ( 226c6d...869597 )

by Philippe

created 2020-03-11 12:47 UTC

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

@@ -6,7 +6,9 @@
 block discarded – undo
 
     $hasErrors = false;
     foreach($fields as $field){
-        if($errors->has($field)) $hasErrors = true;
+        if($errors->has($field)) {
+            $hasErrors = true;
+        }
     }
 
 ?>

Please login to merge, or discard this patch.

		@@ -6,7 +6,9 @@
		block discarded – undo
6	6
7	7	$hasErrors = false;
8	8	foreach($fields as $field){
9		- if($errors->has($field)) $hasErrors = true;
	9	+ if($errors->has($field)) {
	10	+ $hasErrors = true;
	11	+ }
10	12	}
11	13
12	14	?>

thinktomorrow / chief

Push — ft/sitemap ( 226c6d...869597 )

Category

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