Skip to content

Commit

Permalink
Merge pull request #229 from totten/master-php84
Browse files Browse the repository at this point in the history
PHP 8.4 - Fix some more warnings
  • Loading branch information
totten authored Nov 27, 2024
2 parents 4e923f5 + 621e01e commit 573ec28
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lib/src/Util/CvArgvInput.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ class CvArgvInput extends ArgvInput {

protected $originalArgv;

public function __construct(array $argv = NULL, InputDefinition $definition = NULL) {
public function __construct(?array $argv = NULL, ?InputDefinition $definition = NULL) {
$argv = $argv ?? $_SERVER['argv'] ?? [];
$this->originalArgv = $argv;
parent::__construct($argv, $definition);
Expand Down
2 changes: 1 addition & 1 deletion tests/Util/OptionalOptionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ public function testParse($inputArgv, $expectValue) {
* @return array
* [0 => InputInterface, 1 => OutputInterface]
*/
protected function createInputOutput(array $argv = NULL): array {
protected function createInputOutput(?array $argv = NULL): array {
$input = new ArgvInput($argv);
$input->setInteractive(FALSE);
$output = new NullOutput();
Expand Down

0 comments on commit 573ec28

Please # to comment.