Closed (fixed)
Project:
Simplenews
Version:
4.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Nov 2019 at 10:15 UTC
Updated:
16 Aug 2023 at 17:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
adamps commentedI agree the existing situation is unhelpful and inconsistent. Most often you don't see the problem from the UI because somewhere along the way the module creates a temporary in-memory subscriber so avoids the error. It would be useful please to update the issue summary to describe the steps to see this error.
We need to think carefully about the best resolution.
This relates to #3035367: Track history of subscribe/unsubscribe and proof of consent and #2965662: [META] GDPR Compliance.
Comment #3
nofue commentedA client gets this message on a Vs. 9 site using simplenews 3.0.0-beta4
unsubscribing non-existent subscribers:
Exception: The subscriber does not exist. in Drupal\simplenews\Subscription\SubscriptionManager->unsubscribe() (Zeile 156 in …/web/modules/contrib/simplenews/src/Subscription/SubscriptionManager.php).The issue can be triggered by simply entering a non existent e-mail address to the "mass unsubscribe" field, checking one or any number of newsletters and letting go. Obviously something has been changed in the module, as the line is now line 156 instead of 169 as reported previously.
Comment #4
adamps commentedWe can just ignore the missing subscriber (return instead of throwing the exception)
Comment #5
adamps commentedComment #6
adamps commentedComment #8
adamps commentedComment #9
adamps commentedComment #11
adamps commentedComment #13
adamps commented