Browse Source

Add no_useless_else in phpcs

Gabriel Caruso 7 years ago
parent
commit
50045a7536
2 changed files with 7 additions and 6 deletions
  1. 1 0
      .php_cs
  2. 6 6
      src/Composer/Util/Perforce.php

+ 1 - 0
.php_cs

@@ -39,6 +39,7 @@ return PhpCsFixer\Config::create()
         'no_leading_namespace_whitespace' => true,
         'no_trailing_comma_in_singleline_array' => true,
         'no_unused_imports' => true,
+        'no_useless_else' => true,
         'no_whitespace_in_blank_line' => true,
         'object_operator_without_whitespace' => true,
         'phpdoc_align' => true,

+ 6 - 6
src/Composer/Util/Perforce.php

@@ -243,13 +243,13 @@ class Perforce
             }
 
             return null;
-        } else {
-            $command = 'echo $' . $name;
-            $this->executeCommand($command);
-            $result = trim($this->commandResult);
-
-            return $result;
         }
+
+        $command = 'echo $' . $name;
+        $this->executeCommand($command);
+        $result = trim($this->commandResult);
+
+        return $result;
     }
 
     public function queryP4Password()