fix(psalm): update baseline and suppress unnecessary issues

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
nextcloud · Jan 9, 2024 · 9e4c9b9 · 9e4c9b9
1 parent 18399fc
commit 9e4c9b9
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 8 deletions.
diff --git a/apps/dav/appinfo/v2/publicremote.php b/apps/dav/appinfo/v2/publicremote.php
@@ -109,13 +109,20 @@
 	$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
@@ -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
@@ -36,7 +36,7 @@
  * @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 @@ function resolveService($service) {
 
 	$file = resolveService($service);
 
-	if (is_null($file)) {
+	if (!$file) {
 		throw new RemoteException('Path not found', 404);
 	}