Fixed preferencesSaveProcess ignores push notification permissions

[Xon]

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.

 

[XF Bug Bot]

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.