[stable33] chore(psalm): Bump psalm and fix new micro issues

lib/Service/MiscService.php

@@ -335,7 +335,9 @@ public function asyncAndLeaveClientOutOfThis($result = '') {
 		ob_start();
 		echo(json_encode($result));
 		$size = ob_get_length();
-		header('Content-Length: ' . $size);
+		if ($size !== false) {
+			header('Content-Length: ' . $size);
+		}
 		ob_end_flush();
 		flush();
 	}

lib/Service/ShareWrapperService.php

@@ -354,7 +354,7 @@ private function generateSharedWithByPathCacheKey(
 		$pathHash = \md5($path);
 		return $federatedUser->getSingleId() . '#'
 			. $pathHash . '#'
-			. $forChildren . '#'
+			. (int)$forChildren . '#'
 			. $probeSum;
 	}
 

lib/Tools/Traits/TAsync.php

@@ -41,7 +41,9 @@ public function async(string $result = ''): void {
 		echo($result);
 
 		$size = ob_get_length();
-		header('Content-Length: ' . $size);
+		if ($size !== false) {
+			header('Content-Length: ' . $size);
+		}
 		ob_end_flush();
 		flush();
 	}

lib/Tools/Traits/TNCLogger.php

@@ -30,7 +30,6 @@ trait TNCLogger {
 	public static $INFO = 1;
 	public static $DEBUG = 0;
 
-
 	/**
 	 * @param Throwable $t
 	 * @param array $serializable
@@ -172,7 +171,8 @@ public function log(int $level, string $message, bool $trace = false, array $ser
 	 * @return LoggerInterface
 	 */
 	public function logger(): LoggerInterface {
-		if (isset($this->logger)) {
+		/** @psalm-suppress UndefinedThisPropertyFetch Ugly but works */
+		if (isset($this->logger) && $this->logger instanceof LoggerInterface) {
 			return $this->logger;
 		} else {
 			return Server::get(LoggerInterface::class);

lib/Tools/Traits/TNCRequest.php

@@ -24,7 +24,6 @@
 trait TNCRequest {
 	use TNCLogger;
 
-
 	/**
 	 * @param int $size
 	 */
@@ -86,6 +85,7 @@ public function doRequest(NCRequest $request, bool $exceptionOnIssue = true): vo
 	 * @return IClientService
 	 */
 	public function clientService(): IClientService {
+		/** @psalm-suppress UndefinedThisPropertyFetch Ugly but works */
 		if (isset($this->clientService) && $this->clientService instanceof IClientService) {
 			return $this->clientService;
 		} else {
@@ -140,19 +140,14 @@ private function generationClientOptions(NCRequest $request) {
 	 */
 	private function useClient(NCRequest $request): IResponse {
 		$client = $request->getClient();
-		switch ($request->getType()) {
-			case Request::TYPE_POST:
-				return $client->post($request->getCompleteUrl(), $request->getClientOptions());
-			case Request::TYPE_PUT:
-				return $client->put($request->getCompleteUrl(), $request->getClientOptions());
-			case Request::TYPE_DELETE:
-				return $client->delete($request->getCompleteUrl(), $request->getClientOptions());
-			case Request::TYPE_GET:
-				return $client->get(
-					$request->getCompleteUrl() . $request->getQueryString(), $request->getClientOptions()
-				);
-			default:
-				throw new Exception('unknown request type ' . json_encode($request));
-		}
+		return match ($request->getType()) {
+			Request::TYPE_POST => $client->post($request->getCompleteUrl(), $request->getClientOptions()),
+			Request::TYPE_PUT => $client->put($request->getCompleteUrl(), $request->getClientOptions()),
+			Request::TYPE_DELETE => $client->delete($request->getCompleteUrl(), $request->getClientOptions()),
+			Request::TYPE_GET => $client->get(
+				$request->getCompleteUrl() . $request->getQueryString(), $request->getClientOptions()
+			),
+			default => throw new Exception('unknown request type ' . json_encode($request)),
+		};
 	}
 }

lib/Tools/Traits/TStringTools.php

@@ -181,7 +181,7 @@ public function humanReadable(int $bytes): string {
 		$s = ['B', 'KB', 'MB', 'GB', 'TB', 'PB'];
 		$e = floor(log($bytes, 1024));
 
-		return round($bytes / pow(1024, $e), 2) . ' ' . $s[$e];
+		return (string)round((float)$bytes / pow(1024, $e), 2) . ' ' . $s[$e];
 	}
 
 

tests/psalm-baseline.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="6.5.0@38fc8444edf0cebc9205296ee6e30e906ade783b">
+<files psalm-version="6.14.3@d0b040a91f280f071c1abcb1b77ce3822058725a">
   <file src="lib/Activity/ProviderParser.php">
     <TypeDoesNotContainType>
       <code><![CDATA[str_starts_with($sK, '_')]]></code>

tests/stubs/oc_db_querybuilder_querybuilder.php

@@ -27,6 +27,17 @@
 use OCP\IDBConnection;
 use Psr\Log\LoggerInterface;
 
+enum ConflictResolutionMode {
+	/**
+	 * Wait for the row to be unlocked.
+	 */
+	case Ordinary;
+	/**
+	 * Skip the row if it is locked.
+	 */
+	case SkipLocked;
+}
+
 class QueryBuilder implements IQueryBuilder {
 	/** @var string */
 	protected $lastInsertedTable;
@@ -981,4 +992,6 @@ public function runAcrossAllShards(): self
  {
  }
 
+
+public function forUpdate(ConflictResolutionMode $conflictResolutionMode = ConflictResolutionMode::Ordinary): self {}
 }