refactor: Eliminate \OC::$server->get calls

Signed-off-by: Christoph Wurst <[email protected]>
nextcloud · Sep 10, 2024 · 6133774 · 6133774
1 parent 9854ce5
commit 6133774
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 4 deletions.
diff --git a/rector.php b/rector.php
@@ -20,4 +20,10 @@
 	->withPreparedSets(
 		phpunitCodeQuality: true,
 		phpunit: true,
-	);
+	)
+	->withBootstrapFiles([
+		__DIR__ . '/../../lib/composer/autoload.php',
+	])
+	->withRules([
+		\ChristophWurst\Nextcloud\Rector\Rector\OcServerToOcpServerRector::class,
+	]);
diff --git a/tests/Integration/Service/Phishing/PhishingDetectionServiceIntegrationTest.php b/tests/Integration/Service/Phishing/PhishingDetectionServiceIntegrationTest.php
@@ -40,7 +40,7 @@ protected function setUp(): void {
 		$this->l10n = $this->createMock(IL10N::class);
 		$this->contactCheck = new ContactCheck($this->contactsIntegration, $this->l10n);
 		$this->customEmailCheck = new CustomEmailCheck($this->l10n);
-		$this->dateCheck = new DateCheck($this->l10n, \OC::$server->get(ITimeFactory::class));
+		$this->dateCheck = new DateCheck($this->l10n, \OCP\Server::get(ITimeFactory::class));
 		$this->replyToCheck = new ReplyToCheck($this->l10n);
 		$this->linkCheck = new LinkCheck($this->l10n);
 		$this->service = new PhishingDetectionService($this->contactCheck, $this->customEmailCheck, $this->dateCheck, $this->replyToCheck, $this->linkCheck);

diff --git a/vendor-bin/rector/composer.json b/vendor-bin/rector/composer.json
@@ -1,5 +1,9 @@
 {
+	"config": {
+		"sort-packages": true
+	},
     "require-dev": {
-        "rector/rector": "^1.2.4"
-    }
+		"christophwurst/nextcloud-rector": "^0.0.4",
+		"rector/rector": "^1.2.4"
+	}
 }