fix: Use Server::get in files_sharing DeletedShareAPIController

In this case we do not want the application DI container because we are
 requesting classes from other applications, so it’s better to ask the
 server container. We use \OCP\Server::get for this.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
This commit is contained in:
Côme Chilliet 2025-07-01 10:52:50 +02:00
parent 7bdfd94246
commit 90a2c77abf
No known key found for this signature in database
GPG key ID: A3E2F658B28C760A

View file

@ -24,11 +24,11 @@ use OCP\Files\NotFoundException;
use OCP\IGroupManager;
use OCP\IRequest;
use OCP\IUserManager;
use OCP\Server;
use OCP\Share\Exceptions\GenericShareException;
use OCP\Share\Exceptions\ShareNotFound;
use OCP\Share\IManager as ShareManager;
use OCP\Share\IShare;
use Psr\Container\ContainerInterface;
/**
* @psalm-import-type Files_SharingDeletedShare from ResponseDefinitions
@ -44,7 +44,6 @@ class DeletedShareAPIController extends OCSController {
private IGroupManager $groupManager,
private IRootFolder $rootFolder,
private IAppManager $appManager,
private ContainerInterface $serverContainer,
) {
parent::__construct($appName, $request);
}
@ -202,7 +201,7 @@ class DeletedShareAPIController extends OCSController {
throw new QueryException();
}
return $this->serverContainer->get('\OCA\Talk\Share\Helper\DeletedShareAPIController');
return Server::get('\OCA\Talk\Share\Helper\DeletedShareAPIController');
}
/**
@ -219,7 +218,7 @@ class DeletedShareAPIController extends OCSController {
throw new QueryException();
}
return $this->serverContainer->get('\OCA\Deck\Sharing\ShareAPIHelper');
return Server::get('\OCA\Deck\Sharing\ShareAPIHelper');
}
/**
@ -236,6 +235,6 @@ class DeletedShareAPIController extends OCSController {
throw new QueryException();
}
return $this->serverContainer->get('\OCA\ScienceMesh\Sharing\ShareAPIHelper');
return Server::get('\OCA\ScienceMesh\Sharing\ShareAPIHelper');
}
}