Fix duplicate fcm notification issue #746

This commit is contained in:
Ramon Gutierrez 2023-07-19 16:14:49 +08:00
parent c2ff927f1f
commit 0eef2449e8

View file

@ -71,9 +71,9 @@ class FCMSender
foreach ($sessions as $sess) {
$device_id = $sess->getDevicePushID();
if (!empty($device_id)) {
if (!empty($device_id) && !isset($device_ids[$device_id])) {
// send fcm notification
$device_ids[] = $device_id;
$device_ids[$device_id] = true;
}
}
@ -87,7 +87,7 @@ class FCMSender
$data['jo_status'] = $job_order->getStatus();
// send fcm notification
$result = $this->send($device_ids, $this->translator->trans($title), $this->translator->trans($body), $data);
$result = $this->send(array_keys($device_ids), $this->translator->trans($title), $this->translator->trans($body), $data);
return $result;
}