chore: Clean up permission service

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Author: juliusknorr

lib/Service/PermissionService.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * SPDX-FileCopyrightText: 2016 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: AGPL-3.0-or-later
@@ -20,57 +22,27 @@
 use OCP\Cache\CappedMemoryCache;
 use OCP\IConfig;
 use OCP\IGroupManager;
-use OCP\ILogger;
 use OCP\IUserManager;
 use OCP\Share\IManager;
+use Psr\Log\LoggerInterface;
 
 class PermissionService {
-
-	/** @var CirclesService */
-	private $circlesService;
-	/** @var BoardMapper */
-	private $boardMapper;
-	/** @var AclMapper */
-	private $aclMapper;
-	/** @var ILogger */
-	private $logger;
-	/** @var IUserManager */
-	private $userManager;
-	/** @var IGroupManager */
-	private $groupManager;
-	/** @var IConfig */
-	private $config;
-	/** @var IManager */
-	private $shareManager;
-	/** @var string */
-	private $userId;
-	/** @var array */
-	private $users = [];
+	private array $users = [];
 
 	private CappedMemoryCache $boardCache;
 	private CappedMemoryCache $permissionCache;
 
 	public function __construct(
-		ILogger $logger,
-		CirclesService $circlesService,
-		AclMapper $aclMapper,
-		BoardMapper $boardMapper,
-		IUserManager $userManager,
-		IGroupManager $groupManager,
-		IManager $shareManager,
-		IConfig $config,
-		$userId
+		private LoggerInterface $logger,
+		private CirclesService $circlesService,
+		private AclMapper $aclMapper,
+		private BoardMapper $boardMapper,
+		private IUserManager $userManager,
+		private IGroupManager $groupManager,
+		private IManager $shareManager,
+		private IConfig $config,
+		private ?string $userId
 	) {
-		$this->circlesService = $circlesService;
-		$this->aclMapper = $aclMapper;
-		$this->boardMapper = $boardMapper;
-		$this->logger = $logger;
-		$this->userManager = $userManager;
-		$this->groupManager = $groupManager;
-		$this->shareManager = $shareManager;
-		$this->config = $config;
-		$this->userId = $userId;
-
 		$this->boardCache = new CappedMemoryCache();
 		$this->permissionCache = new CappedMemoryCache();
 	}
@@ -99,7 +71,7 @@ public function getPermissions($boardId, ?string $userId = null) {
 			$owner = false;
 			$acls = [];
 		}
-		
+
 		$permissions = [
 			Acl::PERMISSION_READ => $owner || $this->userCan($acls, Acl::PERMISSION_READ, $userId),
 			Acl::PERMISSION_EDIT => $owner || $this->userCan($acls, Acl::PERMISSION_EDIT, $userId),

tests/unit/Service/PermissionServiceTest.php

@@ -34,39 +34,28 @@
 use OCP\IConfig;
 use OCP\IGroup;
 use OCP\IGroupManager;
-use OCP\ILogger;
 use OCP\IRequest;
 use OCP\IUser;
 use OCP\IUserManager;
 use OCP\Share\IManager;
+use PHPUnit\Framework\MockObject\MockObject;
+use Psr\Log\LoggerInterface;
 
 class PermissionServiceTest extends \Test\TestCase {
-
-	/** @var IRequest */
-	private $request;
-	private $circlesService;
-	/** @var PermissionService*/
-	private $service;
-	/** @var ILogger */
-	private $logger;
-	/** @var AclMapper */
-	private $aclMapper;
-	/** @var BoardMapper */
-	private $boardMapper;
-	/** @var IUserManager */
-	private $userManager;
-	/** @var IGroupManager */
-	private $groupManager;
-	/** @var IManager */
-	private $shareManager;
-	/** @var IConfig */
-	private $config;
-	/** @var string */
-	private $userId = 'admin';
+	private IRequest|MockObject $request;
+	private CirclesService|MockObject $circlesService;
+	private PermissionService $service;
+	private LoggerInterface|MockObject $logger;
+	private AclMapper|MockObject $aclMapper;
+	private BoardMapper|MockObject $boardMapper;
+	private IUserManager $userManager;
+	private IGroupManager|MockObject $groupManager;
+	private MockObject|IManager $shareManager;
+	private IConfig|MockObject $config;
 
 	public function setUp(): void {
 		parent::setUp();
-		$this->logger = $this->createMock(ILogger::class);
+		$this->logger = $this->createMock(LoggerInterface::class);
 		$this->request = $this->createMock(IRequest::class);
 		$this->circlesService = $this->createMock(CirclesService::class);
 		$this->aclMapper = $this->createMock(AclMapper::class);