diff --git a/lib/Db/ShareRequestMapper.php b/lib/Db/ShareRequestMapper.php index a836fd2f..9a5a86ab 100644 --- a/lib/Db/ShareRequestMapper.php +++ b/lib/Db/ShareRequestMapper.php @@ -74,15 +74,14 @@ class ShareRequestMapper extends QBMapper { * @return Entity[] * @throws Exception */ - public function getRequestsByItemGuidGroupedByUser(string $item_guid) { + public function getRequestsByItemGuid(string $item_guid) { if (strtolower($this->db->getDatabasePlatform()->getName()) === 'mysql') { $this->db->executeQuery("SET sql_mode = '';"); } $qb = $this->db->getQueryBuilder(); $qb->select('*') ->from(self::TABLE_NAME) - ->where($qb->expr()->eq('item_guid', $qb->createNamedParameter($item_guid, IQueryBuilder::PARAM_STR))) - ->groupBy('target_user_id'); + ->where($qb->expr()->eq('item_guid', $qb->createNamedParameter($item_guid, IQueryBuilder::PARAM_STR))); return $this->findEntities($qb); } diff --git a/lib/Service/ShareService.php b/lib/Service/ShareService.php index 1827674c..7577f72d 100644 --- a/lib/Service/ShareService.php +++ b/lib/Service/ShareService.php @@ -290,7 +290,7 @@ class ShareService { * @throws Exception */ public function getCredentialPendingAclList(string $item_guid) { - return $this->shareRequest->getRequestsByItemGuidGroupedByUser($item_guid); + return $this->shareRequest->getRequestsByItemGuid($item_guid); } /** diff --git a/tests/unit/lib/Db/ShareRequestMapperTest.php b/tests/unit/lib/Db/ShareRequestMapperTest.php index c6ff88be..7a666710 100644 --- a/tests/unit/lib/Db/ShareRequestMapperTest.php +++ b/tests/unit/lib/Db/ShareRequestMapperTest.php @@ -112,7 +112,7 @@ class ShareRequestMapperTest extends DatabaseHelperTest { } /** - * @covers ::getRequestsByItemGuidGroupedByUser + * @covers ::getRequestsByItemGuid */ public function testGetRequestsByItemGuidGroupedByUser() { $dataset = $this->findInDataset( @@ -121,7 +121,7 @@ class ShareRequestMapperTest extends DatabaseHelperTest { $this->dataset->getRow(0)['item_guid'] ); - $result = $this->mapper->getRequestsByItemGuidGroupedByUser($dataset[0]['item_guid']); + $result = $this->mapper->getRequestsByItemGuid($dataset[0]['item_guid']); $this->assertCount(count($dataset), $result);