Error in unsubscribe function

Created on 4 July 2024, 9 months ago

Problem/Motivation

In the user_mailman_register_update_remote function of user_mailman_register.module, there is an error in the condition for updating subscriptions to lists that are synced with roles. The result is that unsubscribe do not work for lists that are not synced with roles

Wrong:

    if (($list['role_sync'] == USER_MAILMAN_REGISTER_SYNC_SUB_AND_UNSUB ||
    $list['role_sync'] == USER_MAILMAN_REGISTER_SYNC_UNSUB_ONLY) &&
    !$email_changed && !$roles_changed) {

Right:

    if ($list['role_sync'] == USER_MAILMAN_REGISTER_SYNC_SUB_AND_UNSUB &&
    !$email_changed && !$roles_changed) {

🐛 Bug report

Status

Active

Version

1.0

Component

Code

Created by

🇺🇸United States rclemings

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @rclemings
Issue was unassigned.
Status changed to Needs review 9 months ago9:38pm 4 July 2024
Comment 9 months ago →
🇺🇸United States rclemings
Patch attached.
Status changed to Fixed 4 months ago6:08pm 20 December 2024
Comment 4 months ago →
🇺🇸United States rclemings
Included in 1.0.0-beta3.
Comment 3 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024