diff --git a/lib/Controller/MentionController.php b/lib/Controller/MentionController.php index 3269acc76d..6f4aa57ef5 100644 --- a/lib/Controller/MentionController.php +++ b/lib/Controller/MentionController.php @@ -16,6 +16,7 @@ use OCP\AppFramework\Utility\ITimeFactory; use OCP\Files\IRootFolder; use OCP\IRequest; +use OCP\IUserManager; use OCP\Notification\IManager; class MentionController extends Controller { @@ -25,6 +26,7 @@ public function __construct( private IRootFolder $rootFolder, private IManager $manager, private ITimeFactory $timeFactory, + private IUserManager $userManager, private ?string $userId, ) { parent::__construct($appName, $request); @@ -39,14 +41,22 @@ public function mention(int $fileId, string $mention): DataResponse { return new DataResponse([], Http::STATUS_NOT_FOUND); } - $userFolder = $this->rootFolder->getUserFolder($mention); + // Reverse the array of users to pop off the first user later + $userResults = array_reverse($this->userManager->searchDisplayName($mention, 1)); + if (count($userResults) < 1) { + return new DataResponse([], Http::STATUS_NOT_FOUND); + } + + // Get the first user returned in the array + $user = array_pop($userResults); + $userFolder = $this->rootFolder->getUserFolder($user->getUID()); $file = $userFolder->getById($fileId)[0]; if ($file === null) { return new DataResponse([], Http::STATUS_NOT_FOUND); } $notification = $this->manager->createNotification(); - $notification->setUser($mention) + $notification->setUser($user->getUID()) ->setApp(Application::APPNAME) ->setSubject(Notifier::TYPE_MENTIONED, [ Notifier::SUBJECT_MENTIONED_SOURCE_USER => $this->userId, @@ -58,7 +68,6 @@ public function mention(int $fileId, string $mention): DataResponse { $notification->setDateTime(\DateTime::createFromImmutable($this->timeFactory->now())); $this->manager->notify($notification); return new DataResponse([], Http::STATUS_OK); - } return new DataResponse([], Http::STATUS_NOT_FOUND);