Php Inspections (EA Extended) - static code analysis includes:

kalessil · kalessil · commit ae3065f8ede1 · 2015-03-07T20:12:23.000+01:00
Reduce couple count calls in [Yaml]
Modernize type casting, fix several strict comparisons
Unsets merged
Elvis operator usage
Short syntax for applied operations
diff --git a/Extension/Core/Type/ChoiceType.php b/Extension/Core/Type/ChoiceType.php
@@ -131,7 +131,7 @@ public function buildView(FormView $view, FormInterface $form, array $options)
             // Add "[]" to the name in case a select tag with multiple options is
             // displayed. Otherwise only one of the selected options is sent in the
             // POST request.
-            $view->vars['full_name'] = $view->vars['full_name'].'[]';
+            $view->vars['full_name'] .= '[]';
         }
     }
 
diff --git a/Util/ServerParams.php b/Util/ServerParams.php
@@ -35,7 +35,7 @@ public function getPostMaxSize()
         } elseif (0 === strpos($max, '0')) {
             $max = intval($max, 8);
         } else {
-            $max = intval($max);
+            $max = (int) $max;
         }
 
         switch (substr($iniMax, -1)) {

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ public function buildView(FormView $view, FormInterface $form, array $options)`
`131`	`131`	`// Add "[]" to the name in case a select tag with multiple options is`
`132`	`132`	`// displayed. Otherwise only one of the selected options is sent in the`
`133`	`133`	`// POST request.`
`134`		`- $view->vars['full_name'] = $view->vars['full_name'].'[]';`
	`134`	`+ $view->vars['full_name'] .= '[]';`
`135`	`135`	`}`
`136`	`136`	`}`
`137`	`137`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ public function getPostMaxSize()`
`35`	`35`	`} elseif (0 === strpos($max, '0')) {`
`36`	`36`	`$max = intval($max, 8);`
`37`	`37`	`} else {`
`38`		`- $max = intval($max);`
	`38`	`+ $max = (int) $max;`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`switch (substr($iniMax, -1)) {`