fix double slack notification

This commit is contained in:
Marko
2024-03-21 16:45:32 +01:00
parent fd9f86cecd
commit 0bc2024494

View File

@@ -63,14 +63,12 @@ class SlackNotifySubscriber implements EventSubscriberInterface
foreach ($order->getData()[0]['positions'] as $item) {
if ($item['menge'] > 0) {
$warehouses = $this->getWarehouseByGtin($item['gtin']);
$warehouse = $this->getWarehouseByGtin($item['gtin']);
if ($warehouses == false) {
if ($warehouse == false) {
continue;
}
foreach ($warehouses as $warehouse) {
$slack = new Client($this->slackWebhookUrl, [
'username' => 'CDS-Notify',
'channel' => '#online_verkäufe' . '_' . $warehouse,
@@ -92,7 +90,6 @@ class SlackNotifySubscriber implements EventSubscriberInterface
}
}
}
}
/**
* @param string $gtin
@@ -103,7 +100,7 @@ class SlackNotifySubscriber implements EventSubscriberInterface
if ($gtin == null) {
return false;
}
$warehouse = [];
$warehouse = "";
$product = $this->productRepository->findOneBy(['gtin' => $gtin]);
@@ -116,7 +113,7 @@ class SlackNotifySubscriber implements EventSubscriberInterface
$w = $this->warehouseRepository->findOneBy(['id' => $item->getWarehouse()->getId()]);
if ($w->getPrio() > $prio) {
$warehouse[] = $w->getName();
$warehouse = $w->getName();
$prio = $w->getPrio();
}
}