Merge pull request #20952 from nextcloud/test/noid/fix-lint-exit-code

Respect exit code of lint run - changed from -exec to xargs as this exits properly

Author: MorrisJobke

composer.json

@@ -22,6 +22,6 @@
     "scripts": {
         "cs:fix": "php-cs-fixer fix",
         "cs:check": "php-cs-fixer fix --dry-run --diff",
-        "lint": "find . -name \\*.php -not -path './lib/composer/*' -not -path './build/.phan/*' -exec php -l \"{}\" \\;"
+        "lint": "find . -name \\*.php -not -path './lib/composer/*' -not -path './build/.phan/*' -print0 | xargs -0 -n1 php -l"
     }
 }

lib/private/AppFramework/Services/AppConfig.php

@@ -70,6 +70,6 @@ public function getUserValue(string $userId, string $key, string $default = ''):
 	}
 
 	public function deleteUserValue(string $userId, string $key): void {
-		return $this->config->deleteUserValue($userId, $this->appName, $key);
+		$this->config->deleteUserValue($userId, $this->appName, $key);
 	}
 }