It seems like $label defined by parameter $label on line 17 can also be of type string; however, Crummy\Phlack\Common\For...r\Sequencer::sequence() does only seem to accept null, maybe add an additional type check?
This check looks at variables that have been passed in as parameters and are passed out again
to other methods.
If the outgoing method call has stricter type requirements than the method itself, an issue is raised.
An additional type check may prevent trouble.
Loading history...
20
}
21
22
/**
23
* @param string $text
24
* @param null $label
25
*
26
* @return string
27
*/
28
12
public static function sequence($text, $label = null)
29
{
30
12
$text = $label ? $text.'|'.$label : $text;
31
32
12
return sprintf(static::SEQUENCE, $text);
33
}
34
35
/**
36
* @param CommandInterface $command
37
*
38
* @return array
39
*/
40
6
public static function command(CommandInterface $command)
mcrumm /
phlack
This check looks at variables that have been passed in as parameters and are passed out again to other methods.
If the outgoing method call has stricter type requirements than the method itself, an issue is raised.
An additional type check may prevent trouble.