Skip to content

Commit 95b0e62

Browse files
committed
Flush in controller
1 parent c68921c commit 95b0e62

File tree

2 files changed

+12
-1
lines changed

2 files changed

+12
-1
lines changed

src/Identity/Controller/AdminAttributeValueController.php

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
namespace PhpList\RestBundle\Identity\Controller;
66

7+
use Doctrine\ORM\EntityManagerInterface;
78
use OpenApi\Attributes as OA;
89
use PhpList\Core\Domain\Identity\Model\Filter\AdminAttributeValueFilter;
910
use PhpList\Core\Domain\Identity\Model\Administrator;
@@ -27,18 +28,21 @@ class AdminAttributeValueController extends BaseController
2728
private AdminAttributeManager $attributeManager;
2829
private AdminAttributeValueNormalizer $normalizer;
2930
private PaginatedDataProvider $paginatedDataProvider;
31+
private EntityManagerInterface $entityManager;
3032

3133
public function __construct(
3234
Authentication $authentication,
3335
RequestValidator $validator,
3436
AdminAttributeManager $attributeManager,
3537
AdminAttributeValueNormalizer $normalizer,
36-
PaginatedDataProvider $paginatedDataProvider
38+
PaginatedDataProvider $paginatedDataProvider,
39+
EntityManagerInterface $entityManager,
3740
) {
3841
parent::__construct($authentication, $validator);
3942
$this->attributeManager = $attributeManager;
4043
$this->normalizer = $normalizer;
4144
$this->paginatedDataProvider = $paginatedDataProvider;
45+
$this->entityManager = $entityManager;
4246
}
4347

4448
#[Route(
@@ -122,6 +126,7 @@ public function createOrUpdate(
122126
definition: $definition,
123127
value: $request->toArray()['value'] ?? null
124128
);
129+
$this->entityManager->flush();
125130
$json = $this->normalizer->normalize($attributeDefinition, 'json');
126131

127132
return $this->json($json, Response::HTTP_CREATED);

src/Subscription/Controller/SubscriptionController.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
namespace PhpList\RestBundle\Subscription\Controller;
66

7+
use Doctrine\ORM\EntityManagerInterface;
78
use OpenApi\Attributes as OA;
89
use PhpList\Core\Domain\Subscription\Model\SubscriberList;
910
use PhpList\Core\Domain\Subscription\Service\Manager\SubscriptionManager;
@@ -28,16 +29,19 @@ class SubscriptionController extends BaseController
2829
{
2930
private SubscriptionManager $subscriptionManager;
3031
private SubscriptionNormalizer $subscriptionNormalizer;
32+
private EntityManagerInterface $entityManager;
3133

3234
public function __construct(
3335
Authentication $authentication,
3436
RequestValidator $validator,
3537
SubscriptionManager $subscriptionManager,
3638
SubscriptionNormalizer $subscriptionNormalizer,
39+
EntityManagerInterface $entityManager,
3740
) {
3841
parent::__construct($authentication, $validator);
3942
$this->subscriptionManager = $subscriptionManager;
4043
$this->subscriptionNormalizer = $subscriptionNormalizer;
44+
$this->entityManager = $entityManager;
4145
}
4246

4347
#[Route('/{listId}/subscribers', name: 'create', requirements: ['listId' => '\d+'], methods: ['POST'])]
@@ -127,6 +131,7 @@ public function createSubscription(
127131
/** @var SubscriptionRequest $subscriptionRequest */
128132
$subscriptionRequest = $this->validator->validate($request, SubscriptionRequest::class);
129133
$subscriptions = $this->subscriptionManager->createSubscriptions($list, $subscriptionRequest->emails);
134+
$this->entityManager->flush();
130135
$normalized = array_map(fn($item) => $this->subscriptionNormalizer->normalize($item), $subscriptions);
131136

132137
return $this->json($normalized, Response::HTTP_CREATED);
@@ -193,6 +198,7 @@ public function deleteSubscriptions(
193198
/** @var SubscriptionRequest $subscriptionRequest */
194199
$subscriptionRequest = $this->validator->validateDto($subscriptionRequest);
195200
$this->subscriptionManager->deleteSubscriptions($list, $subscriptionRequest->emails);
201+
$this->entityManager->flush();
196202

197203
return $this->json(null, Response::HTTP_NO_CONTENT);
198204
}

0 commit comments

Comments
 (0)