Skip to content

Commit 241a9be

Browse files
authored
Merge pull request #23 from SpearDevs/extend-usersubscription
Extend usersubscription
2 parents df8f1b3 + 2f33b0f commit 241a9be

File tree

4 files changed

+8
-9
lines changed

4 files changed

+8
-9
lines changed

assets/admin/controllers/push-notifications-generate_controller.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export default class extends Controller {
3030

3131
this.handleValueChange();
3232

33-
this.receiverInput.addEventListener('input', () => {
33+
this.receiverInput.addEventListener('change', () => {
3434
this.handleValueChange();
3535
});
3636
}

src/Controller/Admin/SendPushNotificationAction.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public function __invoke(Request $request): Response
4242
$pushContent = $data['body'] ?? '';
4343
$customerGroup = $data['groups']?->getName() ?? '';
4444
$receiver = $data['receiver'] ?? '';
45-
$user = $data['user']?->getEmail() ?? '';
45+
$user = $data['user']?->getUsername() ?? '';
4646

4747
$this->channelContext->setChannelCode($data['channel']->getCode());
4848

src/Repository/MySQLUserSubscriptionRepository.php

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public function getSubscriptionsForUsersInGroup(string $groupName, ChannelInterf
3434
public function getSubscriptionsForUserByEmail(string $email, ChannelInterface $channel): iterable
3535
{
3636
return $this->getQueryToGetUserSubscriptions()
37-
->where('user.username = :email')
37+
->where('customer.email = :email')
3838
->setParameter('email', $email)
3939
->andWhere('userSubscription.channel = :channel')
4040
->setParameter('channel', $channel)
@@ -47,12 +47,11 @@ private function getQueryToGetUserSubscriptions(): QueryBuilder
4747
return $this->createQueryBuilder('userSubscription')
4848
->select('userSubscription')
4949
->leftJoin(
50-
'userSubscription.user',
51-
'user',
50+
'userSubscription.customer',
51+
'customer',
5252
'WITH',
53-
'user.id = userSubscription.user',
53+
'customer.id = userSubscription.customer',
5454
)
55-
->where('userSubscription.user is not null')
56-
->join('user.customer', 'customer');
55+
->where('userSubscription.customer is not null');
5756
}
5857
}

tests/Application/assets/controllers/admin/push-notifications-generate_controller.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export default class extends Controller {
3030

3131
this.handleValueChange();
3232

33-
this.receiverInput.addEventListener('input', () => {
33+
this.receiverInput.addEventListener('change', () => {
3434
this.handleValueChange();
3535
});
3636
}

0 commit comments

Comments
 (0)