chore: update composer to fix psalm

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

Author: skjnldsv

composer.json

@@ -1,16 +1,19 @@
 {
 	"name": "nextcloud/guests",
 	"config": {
-		"optimize-autoloader": true,
-		"classmap-authoritative": true,
 		"platform": {
-			"php": "8.1.0"
+			"php": "8.1"
+		},
+		"sort-packages": true,
+		"allow-plugins": {
+			"bamarni/composer-bin-plugin": true
 		},
-		"sort-packages": true
+		"optimize-autoloader": true,
+		"autoloader-suffix": "Guests"
 	},
-	"autoload-dev": {
+	"autoload": {
 		"psr-4": {
-			"OCP\\": "vendor/nextcloud/ocp/OCP"
+			"OCA\\Guests\\": "lib/"
 		}
 	},
 	"scripts": {
@@ -20,14 +23,17 @@
 		"psalm": "psalm --threads=1",
 		"psalm:update-baseline": "psalm --threads=1 --update-baseline",
 		"psalm:clear": "psalm --clear-cache && psalm --clear-global-cache",
-		"psalm:fix": "psalm --alter --issues=InvalidReturnType,InvalidNullableReturnType,MissingParamType,InvalidFalsableReturnType",
+		"psalm:fix": "psalm --alter --issues=InvalidReturnType,InvalidNullableReturnType,MismatchingDocblockParamType,MismatchingDocblockReturnType,MissingParamType,InvalidFalsableReturnType",
 		"test:unit": "vendor/bin/phpunit -c phpunit.xml"
 	},
+	"require": {
+		"php": ">=8.1 <=8.4"
+	},
 	"require-dev": {
 		"nextcloud/coding-standard": "^1.3.2",
 		"nextcloud/ocp": "dev-master",
 		"php-parallel-lint/php-parallel-lint": "^1.2",
 		"phpunit/phpunit": "^9",
-		"vimeo/psalm": "^5.14"
+		"vimeo/psalm": "^5.26.1"
 	}
 }

composer.lock

@@ -22,7 +22,7 @@ public function setOverwrite(array $overwrite): void {
 	/**
 	 * @psalm-suppress MethodSignatureMustProvideReturnType
 	 */
-	public function getValue($app, $key, $default = null) {
+	public function getValue($app, $key, $default = null): string {
 		if (isset($this->overWrite[$app]) && isset($this->overWrite[$app][$key])) {
 			return $this->overWrite[$app][$key];
 		}

lib/AppConfigOverwrite.php

@@ -51,7 +51,7 @@ public function __construct(
 		$this->baseUrlLength = strlen($this->baseUrl);
 	}
 
-	public function isAppWhitelisted($appId): bool {
+	public function isAppWhitelisted(string $appId): bool {
 		$whitelist = $this->config->getAppWhitelist();
 		$alwaysEnabled = explode(',', self::WHITELIST_ALWAYS);
 
@@ -62,7 +62,10 @@ public function isWhitelistEnabled(): bool {
 		return $this->config->useWhitelist();
 	}
 
-	public function isUrlAllowed(IUser $user, $url): bool {
+	/**
+	 * @param false|string $url
+	 */
+	public function isUrlAllowed(IUser $user, string|false $url): bool {
 		if ($this->guestManager->isGuest($user) && $this->isWhitelistEnabled()) {
 			$app = $this->getRequestedApp($url);
 
@@ -95,8 +98,10 @@ public function verifyAccess(IUser $user, IRequest $request): void {
 	/**
 	 * Core has \OC::$REQUESTEDAPP but it isn't set until the routes are matched
 	 * taken from \OC\Route\Router::match()
+	 *
+	 * @param false|string $url
 	 */
-	private function getRequestedApp($url): string {
+	private function getRequestedApp(string|false $url): string {
 		if (substr($url, 0, $this->baseUrlLength) === $this->baseUrl) {
 			$url = substr($url, $this->baseUrlLength);
 		}

lib/AppWhitelist.php

@@ -31,7 +31,7 @@ public function allowExternalStorage(): bool {
 	/**
 	 * @param string|bool $allow
 	 */
-	public function setAllowExternalStorage($allow) {
+	public function setAllowExternalStorage($allow): void {
 		$this->appConfig->setAppValueBool('allow_external_storage', $allow === true || $allow === 'true') ;
 	}
 
@@ -57,7 +57,7 @@ public function useWhitelist(): bool {
 	/**
 	 * @param string|bool $use
 	 */
-	public function setUseWhitelist($use) {
+	public function setUseWhitelist($use): void {
 		$this->appConfig->setAppValueBool('usewhitelist', $use === true || $use === 'true') ;
 	}
 

lib/Config.php

@@ -28,10 +28,16 @@ private function isSettingAllowed(string $setting): bool {
 		return $this->appWhitelist->isAppWhitelisted($appId);
 	}
 
+	/**
+	 * @return void
+	 */
 	public function registerSection(string $type, string $section) {
 		$this->manager->registerSection($type, $section);
 	}
 
+	/**
+	 * @return void
+	 */
 	public function registerSetting(string $type, string $setting) {
 		if (!$this->isSettingAllowed($setting)) {
 			return;

lib/FilteredSettingsManager.php

@@ -73,7 +73,7 @@ public function inGroup($uid, $gid): bool {
 	 * Get all groups a user belongs to
 	 *
 	 * @param string $uid Name of the user
-	 * @return array an array of group names
+	 * @return list<string> an array of group names
 	 * @since 4.5.0
 	 *
 	 * This function fetches all groups a user belongs to. It does not check
@@ -120,7 +120,7 @@ public function groupExists($gid): bool {
 	 * @param string $search
 	 * @param int $limit
 	 * @param int $offset
-	 * @return array an array of user ids
+	 * @return array<int, string> an array of user ids
 	 * @since 4.5.0
 	 */
 	public function usersInGroup($gid, $search = '', $limit = -1, $offset = 0): array {

lib/GroupBackend.php

@@ -149,7 +149,7 @@ private function getShareCountForUsers(array $guests): array {
 		return $data;
 	}
 
-	public function getGuestInfo($userId): array {
+	public function getGuestInfo(string $userId): array {
 		$shares = array_merge(
 			$this->shareManager->getSharedWith($userId, IShare::TYPE_USER, null, -1, 0),
 			$this->shareManager->getSharedWith($userId, IShare::TYPE_GROUP, null, -1, 0),

lib/GuestManager.php

@@ -18,6 +18,9 @@
  * @template-implements IEventListener<BeforeTemplateRenderedEvent>
  */
 class BeforeUserManagementRenderedListener implements IEventListener {
+	/**
+	 * @param Event $event
+	 */
 	public function handle(Event $event): void {
 		if (!($event instanceof BeforeTemplateRenderedEvent)) {
 			return;

lib/Listener/BeforeUserManagementRenderedListener.php

@@ -26,6 +26,9 @@ public function __construct(
 	) {
 	}
 
+	/**
+	 * @param Event $event
+	 */
 	public function handle(Event $event): void {
 		// If the user cannot create guests, we don't need to load the script
 		if (!$this->config->canCreateGuests()) {