When unsubscribing from a newsletter with a mail address unknown to the system, the unsubscribe mail contains a link without a hash. This link leads to a 404 page.

Problem with this is that we don't generate a whole subscriber object inside of simplenews_unsubscribe_user(). On the other side simplenews_subscribe_user() generates a whole object which causes a correct link with a hash appended.

CommentFileSizeAuthor
#2 simplenews_create-subscriber-on-unsubscribe_2645086_2.patch1.79 KBlucastockmann
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lucastockmann created an issue. See original summary.

lucastockmann’s picture

lucastockmann CreditAttribution: lucastockmann at undpaul commented
Status: Active » Needs review
FileSize
simplenews_create-subscriber-on-unsubscribe_2645086_2.patch1.79 KB

This patch creates a user when unsubscribing with a mail address unknown to the system.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented

I don't really understand the problem description and how the patch helps to fix this?

In case a subscriber isn't subscribed, then we don't send an unsubscribe link, we send a different mail that explains exactly that? See simplenews_build_unsubscribe_mail()

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Postponed (maintainer needs more info)
TR’s picture

TR CreditAttribution: TR commented
Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

No further information provided. If you continue to have this problem then feel free to reopen this issue and address the maintainer's questions in #3.