fix: Ensure we only pass string to writeln()

Signed-off-by: Joas Schilling <coding@schilljs.com>
nextcloud · Oct 2, 2024 · 9b38338 · 9b38338
1 parent 6968906
commit 9b38338
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/apps/files_external/lib/Command/Config.php b/apps/files_external/lib/Command/Config.php
@@ -73,7 +73,7 @@ protected function getOption(StorageConfig $mount, $key, OutputInterface $output
 		if (!is_string($value) && json_decode(json_encode($value)) === $value) { // show bools and objects correctly
 			$value = json_encode($value);
 		}
-		$output->writeln($value);
+		$output->writeln((string) $value);
 	}
 
 	/**

diff --git a/apps/files_external/lib/Command/Option.php b/apps/files_external/lib/Command/Option.php
@@ -35,10 +35,10 @@ protected function configure(): void {
 	 */
 	protected function getOption(StorageConfig $mount, $key, OutputInterface $output): void {
 		$value = $mount->getMountOption($key);
-		if (!is_string($value)) { // show bools and objects correctly
+		if (!is_string($value) && json_decode(json_encode($value)) === $value) { // show bools and objects correctly
 			$value = json_encode($value);
 		}
-		$output->writeln($value);
+		$output->writeln((string) $value);
 	}
 
 	/**