fix: Remove uses of deprecated service string names

Use the class/interface name instead to avoid logging.
This was breaking in some cases as logging would trigger a deprecation
 warning, resulting in an infinite loop.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Author: come-nc

lib/private/Memcache/Redis.php

@@ -210,7 +210,7 @@ public function compareSetTTL(string $key, mixed $value, int $ttl): bool {
 
 	#[\Override]
 	public static function isAvailable(): bool {
-		return Server::get('RedisFactory')->isAvailable();
+		return Server::get(RedisFactory::class)->isAvailable();
 	}
 
 	protected function evalLua(string $scriptName, array $keys, array $args) {

tests/lib/Lockdown/Filesystem/NoFSTest.php

@@ -11,6 +11,7 @@
 use OC\Files\Filesystem;
 use OC\Lockdown\Filesystem\NullStorage;
 use OCP\Authentication\Token\IToken;
+use OCP\Lockdown\ILockdownManager;
 use OCP\Server;
 use Test\Traits\UserTrait;
 
@@ -24,7 +25,7 @@ protected function tearDown(): void {
 		$token->setScope([
 			IToken::SCOPE_FILESYSTEM => true
 		]);
-		Server::get('LockdownManager')->setToken($token);
+		Server::get(ILockdownManager::class)->setToken($token);
 		parent::tearDown();
 	}
 
@@ -36,7 +37,7 @@ protected function setUp(): void {
 			IToken::SCOPE_FILESYSTEM => false
 		]);
 
-		Server::get('LockdownManager')->setToken($token);
+		Server::get(ILockdownManager::class)->setToken($token);
 		$this->createUser('foo', 'var');
 	}