Merge pull request #48790 from nextcloud/refactor/apps/constructor-property-promotion

1 files changed, 8 insertions, 27 deletions

diff --git a/apps/testing/lib/Controller/LockingController.php b/apps/testing/lib/Controller/LockingController.php
index 14135a4929d..e226126630f 100644
--- a/apps/testing/lib/Controller/LockingController.php
+++ b/apps/testing/lib/Controller/LockingController.php
@@ -23,21 +23,6 @@ use OCP\Lock\LockedException;
 
 class LockingController extends OCSController {
 
-	/** @var ILockingProvider */
-	protected $lockingProvider;
-
-	/** @var FakeDBLockingProvider */
-	protected $fakeDBLockingProvider;
-
-	/** @var IDBConnection */
-	protected $connection;
-
-	/** @var IConfig */
-	protected $config;
-
-	/** @var IRootFolder */
-	protected $rootFolder;
-
 	/**
 	 * @param string $appName
 	 * @param IRequest $request
@@ -47,20 +32,16 @@ class LockingController extends OCSController {
 	 * @param IConfig $config
 	 * @param IRootFolder $rootFolder
 	 */
-	public function __construct($appName,
+	public function __construct(
+		$appName,
 		IRequest $request,
-		ILockingProvider $lockingProvider,
-		FakeDBLockingProvider $fakeDBLockingProvider,
-		IDBConnection $connection,
-		IConfig $config,
-		IRootFolder $rootFolder) {
+		protected ILockingProvider $lockingProvider,
+		protected FakeDBLockingProvider $fakeDBLockingProvider,
+		protected IDBConnection $connection,
+		protected IConfig $config,
+		protected IRootFolder $rootFolder,
+	) {
 		parent::__construct($appName, $request);
-
-		$this->lockingProvider = $lockingProvider;
-		$this->fakeDBLockingProvider = $fakeDBLockingProvider;
-		$this->connection = $connection;
-		$this->config = $config;
-		$this->rootFolder = $rootFolder;
 	}
 
 	/**