Resolve risky truth test

Bastian Schwarz · Bastian Schwarz · commit ee4b378b2eba · 2024-01-21T00:18:36.000+01:00
diff --git a/src/Task/AbstractCrontabCommand.php b/src/Task/AbstractCrontabCommand.php
@@ -47,12 +47,12 @@ public function __construct(
    */
   final public function getOptionsWithUser() : array {
     $options = $this instanceof HasOptionsInterface ? $this->getOptions() : [];
-    !$this->user ?: $options[] = "-u $this->user";
+    (string) $this->user !== ''  && $options[] = "-u $this->user";
     return $options;
   }
 
   final public function __invoke() : void {
     $output = $this->commandRunner->run($this->crontabCommandFactory->build($this->getOptionsWithUser()));
     !$this instanceof HasOutputInteface ?: $this->writeln->writeln(PHP_EOL . $output);
   }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -47,12 +47,12 @@ public function __construct(`
`47`	`47`	`*/`
`48`	`48`	`final public function getOptionsWithUser() : array {`
`49`	`49`	`$options = $this instanceof HasOptionsInterface ? $this->getOptions() : [];`
`50`		`- !$this->user ?: $options[] = "-u $this->user";`
	`50`	`+ (string) $this->user !== '' && $options[] = "-u $this->user";`
`51`	`51`	`return $options;`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`final public function __invoke() : void {`
`55`	`55`	`$output = $this->commandRunner->run($this->crontabCommandFactory->build($this->getOptionsWithUser()));`
`56`	`56`	`!$this instanceof HasOutputInteface ?: $this->writeln->writeln(PHP_EOL . $output);`
`57`	`57`	`}`
`58`		`-}`
	`58`	`+}`