Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Unify counting dynamic subscribers

Browse Source 
[MAILPOET-3219]
This commit is contained in:
Jan Lysý
2021-03-13 16:55:29 +01:00
committed by Veljko V
parent 6b55cf08e0
commit c7eb9df3da
1 changed files with 29 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

51
lib/Segments/SegmentSubscribersRepository.php
View File

@@ -41,12 +41,7 @@ class SegmentSubscribersRepository {
public function getSubscribersCount(int $segmentId, string $status = null): int {
$segment = $this->getSegment($segmentId);
$subscribersTable = $this->entityManager->getClassMetadata(SubscriberEntity::class)->getTableName();
$queryBuilder = $this->entityManager
->getConnection()
->createQueryBuilder()
->select("count(DISTINCT $subscribersTable.id)")
->from($subscribersTable);
$queryBuilder = $this->createCountQueryBuilder();
if ($segment->isStatic()) {
$queryBuilder = $this->filterSubscribersInStaticSegment($queryBuilder, $segment, $status);
@@ -62,11 +57,7 @@ class SegmentSubscribersRepository {
$subscribersTable = $this->entityManager->getClassMetadata(SubscriberEntity::class)->getTableName();
$subscribersSegmentsTable = $this->entityManager->getClassMetadata(SubscriberSegmentEntity::class)->getTableName();
$queryBuilder = $this->entityManager
->getConnection()
->createQueryBuilder()
->select("count(DISTINCT $subscribersTable.id)")
->from($subscribersTable)
$queryBuilder = $this->createCountQueryBuilder()
->join(
$subscribersTable,
$subscribersSegmentsTable,
@@ -86,18 +77,23 @@ class SegmentSubscribersRepository {
return (int)$result;
}
public function getDynamicSubscribersCount(DynamicSegmentFilterData $data): int {
$subscribersTable = $this->entityManager->getClassMetadata(SubscriberEntity::class)->getTableName();
$queryBuilder = $this->entityManager->getConnection()->createQueryBuilder();
$queryBuilder
->select("COUNT(DISTINCT {$subscribersTable}.id) AS count")
->from($subscribersTable, $subscribersTable);
$queryBuilder = $this->filterHandler->apply($queryBuilder, $data);
public function getDynamicSubscribersCount(DynamicSegmentFilterData $filter): int {
$queryBuilder = $this->createCountQueryBuilder();
$queryBuilder = $this->filterSubscribersInDynamicSegment($queryBuilder, null, null, [$filter]);
$statement = $this->executeQuery($queryBuilder);
$result = $statement->fetchColumn();
return (int)$result;
}
private function createCountQueryBuilder(): QueryBuilder {
$subscribersTable = $this->entityManager->getClassMetadata(SubscriberEntity::class)->getTableName();
return $this->entityManager
->getConnection()
->createQueryBuilder()
->select("count(DISTINCT $subscribersTable.id)")
->from($subscribersTable);
}
public function getSubscribersWithoutSegmentCount(): int {
$queryBuilder = $this->getSubscribersWithoutSegmentCountQuery();
return (int)$queryBuilder->getQuery()->getSingleScalarResult();
@@ -172,19 +168,30 @@ class SegmentSubscribersRepository {
return $queryBuilder;
}
/**
* @param DynamicSegmentFilterData[] $filters
*/
private function filterSubscribersInDynamicSegment(
QueryBuilder $queryBuilder,
SegmentEntity $segment,
string $status = null
?SegmentEntity $segment,
string $status = null,
array $filters = []
): QueryBuilder {
$filters = $segment->getDynamicFilters();
if ($segment) {
$filters = [];
$dynamicFilters = $segment->getDynamicFilters();
foreach ($dynamicFilters as $dynamicFilter) {
$filters[] = $dynamicFilter->getFilterData();
}
}
// We don't allow dynamic segment without filers since it would return all subscribers
// For BC compatibility fetching an empty result
if (count($filters) === 0) {
return $queryBuilder->andWhere('0 = 1');
}
foreach ($filters as $filter) {
$queryBuilder = $this->filterHandler->apply($queryBuilder, $filter->getFilterData());
$queryBuilder = $this->filterHandler->apply($queryBuilder, $filter);
}
$subscribersTable = $this->entityManager->getClassMetadata(SubscriberEntity::class)->getTableName();
$queryBuilder = $queryBuilder->andWhere("$subscribersTable.deleted_at IS NULL");