Refactor code duplication

[MAILPOET-3168]
This commit is contained in:
Pavel Dohnal
2021-01-07 13:09:27 +01:00
committed by Veljko V
parent b64525ecc5
commit 5db5a7edbc

View File

@ -29,10 +29,7 @@ class SegmentsResponseBuilder {
$this->segmentSubscriberRepository = $segmentSubscriberRepository;
}
/**
* @return array
*/
public function build(SegmentEntity $segment) {
public function build(SegmentEntity $segment): array {
return [
'id' => (string)$segment->getId(), // (string) for BC
'name' => $segment->getName(),
@ -58,20 +55,14 @@ class SegmentsResponseBuilder {
}
private function buildListingItem(SegmentEntity $segment, array $scheduledNewsletterSubjectsMap, array $automatedNewsletterSubjectsMap): array {
return [
'id' => (string)$segment->getId(), // (string) for BC
'name' => $segment->getName(),
'type' => $segment->getType(),
'description' => $segment->getDescription(),
'created_at' => $segment->getCreatedAt()->format(self::DATE_FORMAT),
'updated_at' => $segment->getUpdatedAt()->format(self::DATE_FORMAT),
'deleted_at' => ($deletedAt = $segment->getDeletedAt()) ? $deletedAt->format(self::DATE_FORMAT) : null,
'automated_emails_subjects' => $automatedNewsletterSubjectsMap[$segment->getId()] ?? [],
'scheduled_emails_subjects' => $scheduledNewsletterSubjectsMap[$segment->getId()] ?? [],
'subscribers_count' => $this->segmentSubscriberRepository->getSubscribersStatisticsCount($segment),
'subscribers_url' => $this->wp->adminUrl(
'admin.php?page=mailpoet-subscribers#/filter[segment=' . $segment->getId() . ']'
),
];
$data = $this->build($segment);
$data['automated_emails_subjects'] = $automatedNewsletterSubjectsMap[$segment->getId()] ?? [];
$data['scheduled_emails_subjects'] = $scheduledNewsletterSubjectsMap[$segment->getId()] ?? [];
$data['subscribers_count'] = $this->segmentSubscriberRepository->getSubscribersStatisticsCount($segment);
$data['subscribers_url'] = $this->wp->adminUrl(
'admin.php?page=mailpoet-subscribers#/filter[segment=' . $segment->getId() . ']'
);
return $data;
}
}