Skip to content

Commit f3ced1c

Browse files
committed
chore: Port away from QueryException
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
1 parent 7c83e69 commit f3ced1c

3 files changed

Lines changed: 4 additions & 7 deletions

File tree

lib/Collaboration/Resources/ResourceProvider.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,13 @@
1212
use OCA\Deck\Service\PermissionService;
1313
use OCP\AppFramework\Db\DoesNotExistException;
1414
use OCP\AppFramework\Db\MultipleObjectsReturnedException;
15-
use OCP\AppFramework\QueryException;
1615
use OCP\Collaboration\Resources\IManager;
1716
use OCP\Collaboration\Resources\IProvider;
1817
use OCP\Collaboration\Resources\IResource;
1918
use OCP\IURLGenerator;
2019
use OCP\IUser;
2120
use OCP\Server;
21+
use Psr\Container\ContainerExceptionInterface;
2222

2323
class ResourceProvider implements IProvider {
2424
public const RESOURCE_TYPE = 'deck';
@@ -106,7 +106,7 @@ public function invalidateAccessCache($boardId = null) {
106106
try {
107107
/** @var IManager $resourceManager */
108108
$resourceManager = Server::get(IManager::class);
109-
} catch (QueryException $e) {
109+
} catch (ContainerExceptionInterface $e) {
110110
}
111111
if ($boardId !== null) {
112112
$resource = $resourceManager->getResourceForUser(self::RESOURCE_TYPE, (string)$boardId, null);

lib/Collaboration/Resources/ResourceProviderCard.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,14 @@
1414
use OCA\Deck\Service\PermissionService;
1515
use OCP\AppFramework\Db\DoesNotExistException;
1616
use OCP\AppFramework\Db\MultipleObjectsReturnedException;
17-
use OCP\AppFramework\QueryException;
1817
use OCP\Collaboration\Resources\IManager;
1918
use OCP\Collaboration\Resources\IProvider;
2019
use OCP\Collaboration\Resources\IResource;
2120
use OCP\Collaboration\Resources\ResourceException;
2221
use OCP\IURLGenerator;
2322
use OCP\IUser;
2423
use OCP\Server;
24+
use Psr\Container\ContainerExceptionInterface;
2525

2626
class ResourceProviderCard implements IProvider {
2727
public const RESOURCE_TYPE = 'deck-card';
@@ -127,7 +127,7 @@ public function invalidateAccessCache($cardId = null) {
127127
try {
128128
/** @var IManager $resourceManager */
129129
$resourceManager = Server::get(IManager::class);
130-
} catch (QueryException $e) {
130+
} catch (ContainerExceptionInterface $e) {
131131
}
132132
if ($cardId !== null) {
133133
$resource = $resourceManager->getResourceForUser(self::RESOURCE_TYPE, (string)$cardId, null);

tests/unit/Service/AttachmentServiceTest.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,6 @@ class AttachmentServiceTest extends TestCase {
9898
*/
9999
private $attachmentServiceValidator;
100100

101-
/**
102-
* @throws \OCP\AppFramework\QueryException
103-
*/
104101
public function setUp(): void {
105102
parent::setUp();
106103

0 commit comments

Comments
 (0)