Skip to content

Commit

Permalink
Add comments to new enable methods
Browse files Browse the repository at this point in the history
  • Loading branch information
mattstauffer committed Jul 6, 2022
1 parent 8219219 commit 912f677
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion app/Commands/EnableCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,11 @@ public function handle(Environment $environment, Services $services): void
$this->enable($option, $useDefaults, $passthroughOptions);
}

/**
* Since we're pulling the *full* list of server arguments, not just relying on
* $this->argument, we have to do own manual overriding for testing scenarios,
* because pulling $_SERVER['argv'] won't give the right results in testing.
*/
public function serverArguments(): array
{
if (App::environment() === 'testing') {
Expand Down Expand Up @@ -80,7 +85,8 @@ public function extractPassthroughOptions(array $arguments): array
}

/**
* Remove any options or passthrough options from the parameters list
* Remove any options or passthrough options from the parameters list, returning
* just the parameters passed to `enable`
*
* @param array $arguments
* @return array
Expand Down

0 comments on commit 912f677

Please sign in to comment.