Fixed preferencesSaveProcess ignores push notification permissions

Xon

Well-known member
Affected version
2.2.6 Patch 2
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.
 
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 Bottom