Freegle / iznik-server / #2511

88%

push

php-coveralls

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.

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

26570 of 29433 relevant lines covered (90.27%)

31.51 hits per line