Fixed preferencesSaveProcess ignores push notification permissions

Affected version
2.2.6 Patch 2

Xon

Well-known member
When preferencesSaveProcess is being called, it doesn't check the $visitor->canUsePushNotifications() before fetching and processing push notification opt-outs

This can result in the user's opt-outs being lost in the following case;
  • User makes push notification opt-outs
  • User is put into a group which does not permit push notifications
  • User makes changes to alert opt-outs/preferences
  • User is put into a group which does permit push notifications
  • Original opt-outs for push notifications have been lost.
 

XF Bug Bot

XenForo bug fixer bot
Staff member
Thank you for reporting this issue, it has now been resolved. We are aiming to include any changes that have been made in a future XF release (2.2.7).

Change log:
Only overwrite push notification opt-out preferences if the visitor has permission to use push notifications.
There may be a delay before changes are rolled out to the XenForo Community.
 
Top