Freegle / iznik-server / #2517

88%

Build Type

push

php-coveralls

Committed by edwh

Commit Message

Fix duplicate push notifications for unviewed chat messages.

The legacy notification path uses lastmsgseen (view tracking) instead of
lastmsgnotified (notification tracking). When triggered days later by cron
jobs or other events, messages that were already notified but not viewed
would get re-notified.

Fix: In chat-specific context (chatid provided), filter FCM notifications
from legacy path after notifyIndividualMessages() has already checked them
using lastmsgnotified.

Run Details

4 of 4 new or added lines in 1 file covered. (100.0%)

11 existing lines in 1 file now uncovered.

26570 of 29433 relevant lines covered (90.27%)

31.51 hits per line