fix appstoreenabled check in Fetcher so it's coherent with AppSetting…

…sController Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
nextcloud · Oct 15, 2021 · 70379ab · 70379ab
1 parent 1119197
commit 70379ab
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/private/App/AppStore/Fetcher/Fetcher.php b/lib/private/App/AppStore/Fetcher/Fetcher.php
@@ -84,7 +84,7 @@ public function __construct(Factory $appDataFactory,
 	 * @return array
 	 */
 	protected function fetch($ETag, $content) {
-		$appstoreenabled = $this->config->getSystemValue('appstoreenabled', true);
+		$appstoreenabled = $this->config->getSystemValue('appstoreenabled', true) === true;
 		if ((int)$this->config->getAppValue('settings', 'appstore-fetcher-lastFailure', '0') > time() - self::RETRY_AFTER_FAILURE_SECONDS) {
 			return [];
 		}
@@ -136,7 +136,7 @@ protected function fetch($ETag, $content) {
 	 * @return array
 	 */
 	public function get($allowUnstable = false) {
-		$appstoreenabled = $this->config->getSystemValue('appstoreenabled', true);
+		$appstoreenabled = $this->config->getSystemValue('appstoreenabled', true) === true;
 		$internetavailable = $this->config->getSystemValue('has_internet_connection', true);
 
 		if (!$appstoreenabled || !$internetavailable) {