Skip to content

Commit

Permalink
Fix array options bug
Browse files Browse the repository at this point in the history
  • Loading branch information
lorisleiva committed Apr 5, 2020
1 parent e8f166a commit 2315426
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions src/LaravelDeployer/Commands/BaseCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
class BaseCommand extends Command
{
use ParsesCliParameters;

protected $parameters;
protected $providedFile;
protected $providedStrategy;
Expand All @@ -37,7 +37,7 @@ public function __construct()
if ($this->useDeployerOptions) {
$this->signature .= $deployerOptions;
}

parent::__construct();
}

Expand All @@ -49,7 +49,7 @@ public function dep($command)
$this->providedStrategy = $this->parameters->pull('--strategy');

// Force Ansi mode if not specified.
if ($this->parameters->intersect(['--ansi', '--no-ansi'])->isEmpty()) {
if (! $this->parameters->contains('--ansi') && ! $this->parameters->contains('--no-ansi')) {
$this->parameters->push('--ansi');
}

Expand Down Expand Up @@ -94,7 +94,7 @@ public function getConfigFile()
);
}
}

public function getCustomDeployFile()
{
if (! $configFile = $this->getConfigFile()) {
Expand Down
2 changes: 1 addition & 1 deletion tests/Strategies/BasicTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -32,4 +32,4 @@ function a_basic_deployment_should_execute_all_default_tasks()
$this->assertStringContainsString('Successfully deployed', $output);
$this->assertSuccessfulDeployment();
}
}
}

0 comments on commit 2315426

Please sign in to comment.