Merge pull request #15753 from owncloud/remove-app-version-from-disabled-list

Remove the app version from disabled app list

core/command/app/listapps.php

@@ -55,12 +55,12 @@ class ListApps extends Base {
 
 		sort($enabledApps);
 		foreach ($enabledApps as $app) {
-			$apps['enabled'][$app] = (isset($versions[$app])) ? $versions[$app] : '';
+			$apps['enabled'][$app] = (isset($versions[$app])) ? $versions[$app] : true;
 		}
 
 		sort($disabledApps);
 		foreach ($disabledApps as $app) {
-			$apps['disabled'][$app] = (isset($versions[$app])) ? $versions[$app] : '';
+			$apps['disabled'][$app] = null;
 		}
 
 		$this->writeAppList($input, $output, $apps);

core/command/base.php

@@ -54,9 +54,30 @@ class Base extends Command {
 				break;
 			default:
 				foreach ($items as $key => $item) {
-					$output->writeln(' - ' . (!is_int($key) ? $key . ': ' : '') . $item);
+					if (!is_int($key)) {
+						$value = $this->valueToString($item);
+						if (!is_null($value)) {
+							$output->writeln(' - ' . $key . ': ' . $value);
+						} else {
+							$output->writeln(' - ' . $key);
+						}
+					} else {
+						$output->writeln(' - ' . $this->valueToString($item));
+					}
 				}
 				break;
 		}
 	}
+
+	protected function valueToString($value) {
+		if ($value === false) {
+			return 'false';
+		} else if ($value === true) {
+			return 'true';
+		} else if ($value === null) {
+			null;
+		} else {
+			return $value;
+		}
+	}
 }

core/command/status.php

@@ -37,7 +37,7 @@ class Status extends Base {
 
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		$values = array(
-			'installed' => \OC_Config::getValue('installed') ? 'true' : 'false',
+			'installed' => (bool) \OC_Config::getValue('installed'),
 			'version' => implode('.', \OC_Util::getVersion()),
 			'versionstring' => \OC_Util::getVersionString(),
 			'edition' => \OC_Util::getEditionString(),