refactor: use constructor property promotion

Fix "Creation of dynamic property OCA\\FilesLock\\DAV\\LockBackend::$server is deprecated" by removing it. Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
nextcloud · May 2, 2024 · c285083 · c285083
1 parent 87bd6b3
commit c285083
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 16 deletions.
diff --git a/lib/DAV/LockBackend.php b/lib/DAV/LockBackend.php
@@ -42,25 +42,13 @@
 use Sabre\DAV\Server;
 
 class LockBackend implements BackendInterface {
-	/** @var FileService */
-	private $fileService;
-
-	/** @var LockService */
-	private $lockService;
-
-	/** @var bool */
-	private $absolute = false;
-
 	public function __construct(
-		Server $server, FileService $fileService, LockService $lockService, bool $absolute
+		private FileService $fileService,
+		private LockService $lockService,
+		private bool $absolute
 	) {
-		$this->server = $server;
-		$this->fileService = $fileService;
-		$this->lockService = $lockService;
-		$this->absolute = $absolute;
 	}
 
-
 	/**
 	 * @param string $uri
 	 * @param bool $returnChildLocks

diff --git a/lib/DAV/LockPlugin.php b/lib/DAV/LockPlugin.php
@@ -58,7 +58,7 @@ public function initialize(Server $server) {
 				$absolute = true;
 				break;
 		}
-		$this->locksBackend = new LockBackend($server, $this->fileService, $this->lockService, $absolute);
+		$this->locksBackend = new LockBackend($this->fileService, $this->lockService, $absolute);
 		$server->on('propFind', [$this, 'customProperties']);
 		parent::initialize($server);
 	}