From 3341c53e9a502650309b48c0d69f97b9e0767df0 Mon Sep 17 00:00:00 2001 From: grnd-alt Date: Mon, 18 May 2026 16:05:58 +0200 Subject: [PATCH] fix: use unknownActivityException Signed-off-by: grnd-alt --- lib/Activity/DeckProvider.php | 9 +++++---- tests/unit/Activity/DeckProviderTest.php | 3 ++- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/lib/Activity/DeckProvider.php b/lib/Activity/DeckProvider.php index e81c6a2e1f..78017f3eb5 100644 --- a/lib/Activity/DeckProvider.php +++ b/lib/Activity/DeckProvider.php @@ -10,6 +10,7 @@ use OCA\Deck\Db\Acl; use OCA\Deck\Service\CardService; use OCA\Deck\Service\CirclesService; +use OCP\Activity\Exceptions\UnknownActivityException; use OCP\Activity\IEvent; use OCP\Activity\IProvider; use OCP\Comments\IComment; @@ -61,12 +62,12 @@ public function __construct(IURLGenerator $urlGenerator, ActivityManager $activi * To do so, simply use setChildEvent($previousEvent) after setting the * combined subject on the current event. * @return IEvent - * @throws \InvalidArgumentException Should be thrown if your provider does not know this event + * @throws UnknownActivityException Should be thrown if your provider does not know this event * @since 11.0.0 */ public function parse($language, IEvent $event, ?IEvent $previousEvent = null): IEvent { if ($event->getApp() !== 'deck') { - throw new \InvalidArgumentException(); + throw new UnknownActivityException(); } $event = $this->getIcon($event); @@ -107,7 +108,7 @@ public function parse($language, IEvent $event, ?IEvent $previousEvent = null): } if ($event->getObjectType() === ActivityManager::DECK_OBJECT_BOARD) { if (!$this->activityManager->canSeeBoardActivity($event->getObjectId(), $event->getAffectedUser())) { - throw new \InvalidArgumentException(); + throw new UnknownActivityException(); } if (isset($subjectParams['board']) && $event->getObjectName() === '') { $event->setObject($event->getObjectType(), $event->getObjectId(), $subjectParams['board']['title']); @@ -124,7 +125,7 @@ public function parse($language, IEvent $event, ?IEvent $previousEvent = null): if (isset($subjectParams['card']) && $event->getObjectType() === ActivityManager::DECK_OBJECT_CARD) { if (!$this->activityManager->canSeeCardActivity($event->getObjectId(), $event->getAffectedUser())) { - throw new \InvalidArgumentException(); + throw new UnknownActivityException(); } if ($event->getObjectName() === '') { $event->setObject($event->getObjectType(), $event->getObjectId(), $subjectParams['card']['title']); diff --git a/tests/unit/Activity/DeckProviderTest.php b/tests/unit/Activity/DeckProviderTest.php index 1a99e9e4d6..18399bc633 100644 --- a/tests/unit/Activity/DeckProviderTest.php +++ b/tests/unit/Activity/DeckProviderTest.php @@ -29,6 +29,7 @@ use OCA\Deck\Db\Card; use OCA\Deck\Service\CardService; use OCA\Deck\Service\CirclesService; +use OCP\Activity\Exceptions\UnknownActivityException; use OCP\Activity\IEvent; use OCP\Comments\IComment; use OCP\Comments\ICommentsManager; @@ -122,7 +123,7 @@ private function mockEvent($objectType, $objectId, $objectName, $subject, $subje } public function testParseFailureApp() { - $this->expectException(\InvalidArgumentException::class); + $this->expectException(UnknownActivityException::class); $event = $this->createMock(IEvent::class); $event->expects($this->once())->method('getApp')->willReturn('notdeck'); $this->provider->parse('en_US', $event, $event);