Right now, you get a 404, this seems to be confusing for some users.

Something like "Nothing needs to be confirmed. Maybe the confirmation link has already been used?"

Also need to check if the /ok direct link has a wrong confirmation message (no mail displayed).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Assigned: Unassigned » corvus_ch
corvus_ch’s picture

Status: Active » Needs review
FileSize
3.04 KB
Berdir’s picture

Status: Needs review » Needs work
+++ b/includes/simplenews.subscription.incundefined
@@ -501,8 +501,15 @@ function simplenews_confirm_subscription() {
+        // Redirect and display message if no changes are availlable.

Should be "available", one l too much.

corvus_ch’s picture

Status: Needs work » Needs review
FileSize
3.64 KB

Status: Needs review » Needs work

The last submitted patch, simplenews-1462394-4.patch, failed testing.

corvus_ch’s picture

Status: Needs work » Needs review
corvus_ch’s picture

FileSize
3.98 KB
Berdir’s picture

Status: Needs review » Fixed

Thanks, commited.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.