Skip to content

Commit

Permalink
[stable30] fix(command): Ensure that writeln() argument is string (#409)
Browse files Browse the repository at this point in the history
Backport of PR #408

Signed-off-by: Joas Schilling <coding@schilljs.com>
Co-authored-by: Joas Schilling <coding@schilljs.com>
  • Loading branch information
backportbot[bot] and nickvergessen authored Oct 3, 2024
1 parent 0600821 commit 090445e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/Command/ExAppConfig/GetConfig.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
$defaultValue = $input->getOption('default-value');
if ($exAppConfig === null) {
if (isset($defaultValue)) {
$output->writeln($defaultValue);
$output->writeln((string)$defaultValue);
return 0;
}
$output->writeln(sprintf('ExApp %s config %s not found', $appId, $configKey));
Expand All @@ -58,7 +58,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int

$value = $exAppConfig->getConfigvalue() ?? $defaultValue;

$output->writeln($value);
$output->writeln((string)$value);
return 0;
}
}

0 comments on commit 090445e

Please sign in to comment.