Skip to content

Commit

Permalink
Merged PR 56475: Fix incorrect publisher topic for subscriptions on c…
Browse files Browse the repository at this point in the history
…ontact reset

## What's being changed

We have corrected a publisher topic for contact subscriptions.

## Why it's being changed

When the cron for order clean-up runs, the `resetContact` was publishing a subscription message to an invalid queue.

## How to review / test this change

- Ensure you have DB consumers configured.
- Trigger the `sales_order_save_after` event.
- Observe no error in the message queue report.

Related work items: #263931
  • Loading branch information
pvpcookie authored and sta1r committed Jul 29, 2024
1 parent 50bc8bb commit a26a4f8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Observer/Sales/OrderSaveAfter.php
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ private function resetContact($contact)
$subscriptionData->setId($contact->getId());
$subscriptionData->setEmail($contact->getEmail());
$subscriptionData->setWebsiteId($contact->getWebsiteId());
$this->publisher->publish('ddg.newsletter.subscribe', $subscriptionData);
$this->publisher->publish('ddg.newsletter.subscription', $subscriptionData);
}
}

Expand Down

0 comments on commit a26a4f8

Please sign in to comment.