getByPage data

Author: tatevikg1

src/Domain/Subscription/Repository/SubscriberPageDataRepository.php

@@ -18,4 +18,10 @@ public function findByPageAndName(SubscribePage $page, string $name): ?Subscribe
     {
         return $this->findOneBy(['id' => $page->getId(), 'name' => $name]);
     }
+
+    /** @return SubscribePageData[] */
+    public function getByPage(SubscribePage $page): array
+    {
+        return $this->findBy(['id' => $page->getId()]);
+    }
 }

src/Domain/Subscription/Repository/SubscriberPageRepository.php

@@ -7,8 +7,24 @@
 use PhpList\Core\Domain\Common\Repository\AbstractRepository;
 use PhpList\Core\Domain\Common\Repository\CursorPaginationTrait;
 use PhpList\Core\Domain\Common\Repository\Interfaces\PaginatableRepositoryInterface;
+use PhpList\Core\Domain\Subscription\Model\SubscribePage;
+use PhpList\Core\Domain\Subscription\Model\SubscribePageData;
 
 class SubscriberPageRepository extends AbstractRepository implements PaginatableRepositoryInterface
 {
     use CursorPaginationTrait;
+
+    /** @return array{page: SubscribePage, data: SubscribePageData}[] */
+    public function findPagesWithData(int $pageId): array
+    {
+        return $this->createQueryBuilder('p')
+            ->select('p AS page, d AS data')
+            ->from(SubscribePage::class, 'p')
+            ->from(SubscribePageData::class, 'd')
+            ->where('p.id = :id')
+            ->andWhere('d.id = p.id')
+            ->setParameter('id', $pageId)
+            ->getQuery()
+            ->getResult();
+    }
 }

src/Domain/Subscription/Service/Manager/SubscribePageManager.php

@@ -79,11 +79,10 @@ public function deletePage(SubscribePage $page): void
         $this->pageRepository->remove($page);
     }
 
-    public function getPageData(SubscribePage $page, string $name): ?string
+    /** @return SubscribePageData[] */
+    public function getPageData(SubscribePage $page): array
     {
-        /** @var SubscribePageData|null $data */
-        $data = $this->pageDataRepository->findByPageAndName($page, $name);
-        return $data?->getData();
+        return $this->pageDataRepository->getByPage($page,);
     }
 
     public function setPageData(SubscribePage $page, string $name, ?string $value): SubscribePageData

tests/Unit/Domain/Subscription/Service/Manager/SubscribePageManagerTest.php

@@ -153,12 +153,13 @@ public function testGetPageDataReturnsStringWhenFound(): void
 
         $this->pageDataRepository
             ->expects($this->once())
-            ->method('findByPageAndName')
-            ->with($page, 'key')
-            ->willReturn($data);
+            ->method('getByPage')
+            ->with($page)
+            ->willReturn([$data]);
 
-        $result = $this->manager->getPageData($page, 'key');
-        $this->assertSame('value', $result);
+        $result = $this->manager->getPageData($page);
+        $this->assertIsArray($result);
+        $this->assertSame('value', $result[0]->getData());
     }
 
     public function testGetPageDataReturnsNullWhenNotFound(): void
@@ -167,12 +168,12 @@ public function testGetPageDataReturnsNullWhenNotFound(): void
 
         $this->pageDataRepository
             ->expects($this->once())
-            ->method('findByPageAndName')
-            ->with($page, 'missing')
-            ->willReturn(null);
+            ->method('getByPage')
+            ->with($page)
+            ->willReturn([]);
 
-        $result = $this->manager->getPageData($page, 'missing');
-        $this->assertNull($result);
+        $result = $this->manager->getPageData($page);
+        $this->assertEmpty($result);
     }
 
     public function testSetPageDataUpdatesExistingDataAndFlushes(): void