Merge pull request #38 from niconoe-/patch-1

rskuipers · web-flow · commit 0e569b8ac503 · 2022-02-15T20:28:15.000+01:00
Fix #37
diff --git a/src/PhpAssumptions/Cli.php b/src/PhpAssumptions/Cli.php
@@ -102,7 +102,7 @@ public function handle(array $args)
                 break;
         }
 
-        $excludes = $this->getPathsFromList($this->cli->arguments->get('exclude'));
+        $excludes = $this->getPathsFromList((string) $this->cli->arguments->get('exclude'));
 
         $nodeTraverser = new NodeTraverser();
 
@@ -135,7 +135,7 @@ public function handle(array $args)
     private function getPathsFromList($list)
     {
         $paths = [];
-        if (strlen($list) > 0) {
+        if ($list !== '') {
             $items = explode(',', $list);
             foreach ($items as $item) {
                 $paths = array_merge($paths, $this->getPaths($item));

Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,7 @@ public function handle(array $args)`
`102`	`102`	`break;`
`103`	`103`	`}`
`104`	`104`
`105`		`- $excludes = $this->getPathsFromList($this->cli->arguments->get('exclude'));`
	`105`	`+ $excludes = $this->getPathsFromList((string) $this->cli->arguments->get('exclude'));`
`106`	`106`
`107`	`107`	`$nodeTraverser = new NodeTraverser();`
`108`	`108`
`@@ -135,7 +135,7 @@ public function handle(array $args)`
`135`	`135`	`private function getPathsFromList($list)`
`136`	`136`	`{`
`137`	`137`	`$paths = [];`
`138`		`- if (strlen($list) > 0) {`
	`138`	`+ if ($list !== '') {`
`139`	`139`	`$items = explode(',', $list);`
`140`	`140`	`foreach ($items as $item) {`
`141`	`141`	`$paths = array_merge($paths, $this->getPaths($item));`