Merge pull request #302 from nextcloud/debt/noid/lock-backend-propert…

…y-promotion

Fix some deprecation warnings

lib/DAV/LockBackend.php

@@ -39,28 +39,15 @@
 use OCP\Files\NotFoundException;
 use Sabre\DAV\Locks\Backend\BackendInterface;
 use Sabre\DAV\Locks\LockInfo;
-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

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);
 	}

tests/Feature/LockFeatureTest.php

@@ -30,6 +30,7 @@
 use OCP\Files\Lock\ILockManager;
 use OCP\Files\Lock\LockContext;
 use OCP\Lock\ManuallyLockedException;
+use OCP\Share\IManager as IShareManager;
 use OCP\Share\IShare;
 use Test\TestCase;
 use Test\Util\User\Dummy;
@@ -43,6 +44,8 @@ class LockFeatureTest extends TestCase {
 
 	private LockManager $lockManager;
 	private IRootFolder $rootFolder;
+	private ITimeFactory $timeFactory;
+	private IShareManager $shareManager;
 	private ?int $time = null;
 
 	public static function setUpBeforeClass(): void {