From: John Molakvoæ <skjnldsv@protonmail.com>
Date: Fri, 29 Dec 2023 09:16:08 +0000 (+0100)
Subject: fix(psalm): update baseline and suppress unnecessary issues
X-Git-Tag: v29.0.0beta1~542^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fpull%2F32400%2Fhead;p=nextcloud-server.git

fix(psalm): update baseline and suppress unnecessary issues

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
---

diff --git a/apps/dav/appinfo/v2/publicremote.php b/apps/dav/appinfo/v2/publicremote.php
index f9a3b23e0b8..0ef9bda8dda 100644
--- a/apps/dav/appinfo/v2/publicremote.php
+++ b/apps/dav/appinfo/v2/publicremote.php
@@ -109,13 +109,20 @@ $server = $serverFactory->createServer($baseuri, $requestUri, $authPlugin, funct
 	$fileId = $share->getNodeId();
 
 	// FIXME: should not add storage wrappers outside of preSetup, need to find a better way
+	/** @psalm-suppress InternalMethod */
 	$previousLog = Filesystem::logWarningWhenAddingStorageWrapper(false);
+
+	/** @psalm-suppress MissingClosureParamType */
 	Filesystem::addStorageWrapper('sharePermissions', function ($mountPoint, $storage) use ($share) {
 		return new PermissionsMask(['storage' => $storage, 'mask' => $share->getPermissions() | \OCP\Constants::PERMISSION_SHARE]);
 	});
+
+	/** @psalm-suppress MissingClosureParamType */
 	Filesystem::addStorageWrapper('shareOwner', function ($mountPoint, $storage) use ($share) {
 		return new PublicOwnerWrapper(['storage' => $storage, 'owner' => $share->getShareOwner()]);
 	});
+
+	/** @psalm-suppress InternalMethod */
 	Filesystem::logWarningWhenAddingStorageWrapper($previousLog);
 
 	OC_Util::tearDownFS();
diff --git a/build/psalm-baseline.xml b/build/psalm-baseline.xml
index 4af64e0bcab..ba529091482 100644
--- a/build/psalm-baseline.xml
+++ b/build/psalm-baseline.xml
@@ -105,12 +105,6 @@
       <code>$baseuri</code>
     </UndefinedGlobalVariable>
   </file>
-  <file src="apps/dav/appinfo/v2/publicremote.php">
-    <InternalMethod>
-      <code>Filesystem::logWarningWhenAddingStorageWrapper($previousLog)</code>
-      <code>Filesystem::logWarningWhenAddingStorageWrapper(false)</code>
-    </InternalMethod>
-  </file>
   <file src="apps/dav/lib/AppInfo/Application.php">
     <InvalidArgument>
       <code>CalendarDeletionDefaultUpdaterListener::class</code>
diff --git a/public.php b/public.php
index 5954da7e3a0..9f3ffcec5e9 100644
--- a/public.php
+++ b/public.php
@@ -36,7 +36,7 @@ require_once __DIR__ . '/lib/versioncheck.php';
  * @param $service
  * @return string
  */
-function resolveService($service) {
+function resolveService(string $service): string {
 	$services = [
 		'webdav' => 'dav/appinfo/v1/publicwebdav.php',
 		'dav' => 'dav/appinfo/v2/publicremote.php',
@@ -74,7 +74,7 @@ try {
 
 	$file = resolveService($service);
 
-	if (is_null($file)) {
+	if (!$file) {
 		throw new RemoteException('Path not found', 404);
 	}