Simplify

Author: kylekatarnls

.codeclimate.yml

@@ -11,10 +11,15 @@ engines:
     enabled: true
   phpmd:
     enabled: true
-    checks:
-      Controversial/CamelCaseMethodName:
-        exclude_paths:
-        - "src/JsPhpize/Stream/ExpressionStream.php"
+    exclude_fingerprints:
+    # Ignore Stream checks
+    - 862522819408aad57533cfe39d74fde2
+    - 9fe2454c6fe08317f5451a330cff5305
+    - ea57ee5054e53ca5768b1f49cd5ef638
+    - 7986fd32c99587cde451688d5b55808d
+    - 41ada8ab27cb7d56ab534d0b2abc4702
+    - eb746d53b9a8dabef2c1041d8e37158d
+    - ef7cef7244578b6906e8785db37f6f48
 ratings:
   paths:
   - "**.php"

src/JsPhpize/Parser/ExpressionParser.php

@@ -117,13 +117,8 @@ protected function handlePlus(&$expression, $token, $handlePlus)
                 $before = implode(' ', array_reverse($before));
                 $after = array(',', $token, ',');
                 while ($this->next() && ($next = $this->current())) {
-                    switch ($next->type) {
-                        case '(':
-                        case '{':
-                        case '[':
-                        case 'variable':
-                            $next = $this->getExpression(false);
-                            break;
+                    if (in_array($next->type, array('(', '{', '[', 'variable')))) {
+                        $next = $this->getExpression(false);
                     }
                     if ($next instanceof Token && !in_array($next->type, $priorToPlusOperators)) {
                         break;