The redirect to to front after subscription is confirmed / subscription removal is confirmed does not work.

Comments

simon georges’s picture

czigor’s picture

#1074784: Uniformisation of behaviour during subscription removal is only for the case when no confirmation page is set. I do not have redirection even if there is a confirmation page set at admin/config/simplenews/subscription.

2xe’s picture

Same problem here. Confirmation page is set, but when clicking the confirmation link in the e-mail, the front page is used instead.

--
edit:

Checked out the code, and on line 128 in simplenews.subscription.inc there is a test for empty($arguments). The subscription callback does never pass on any arguments, so the confirmation removal and add forms will always display.

I changed the test to (!empty($arguments)) which will work for me (short term) and it then produces the expected behavior, redirecting to the correct page.

mordonez’s picture

I attach a patch that fixes both redirects to frontpage or custom page

mordonez’s picture

I attach a patch that fixes both redirects to frontpage or custom page

mordonez’s picture

StatusFileSize
new2.29 KB

I fix the path name

simon georges’s picture

Status: Active » Needs review

Changing status, so I think about reviewing it.

miro_dietiker’s picture

pushing this a little to my radar.

Anonymous’s picture

Status: Needs review » Reviewed & tested by the community

Let's see what the maintainer thinks. Thanks mordonez!

luluthegeek’s picture

Patch #6 worked for me. Thanks !

turco’s picture

Issue tags: +redirect, +confirmation, +subscription

Patch #6 worked for me too. Thank you

simon georges’s picture

Assigned: Unassigned » simon georges
Issue tags: -redirect, -confirmation, -subscription

Removing tags. And affecting the issue to me so that I think of it during my next sprint.

simon georges’s picture

Assigned: simon georges » Unassigned
Status: Reviewed & tested by the community » Fixed

Committed, attributed to mordonez.

Thanks, everybody !

Status: Fixed » Closed (fixed)

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

ArtActivator.com’s picture

Version: 7.x-1.x-dev » 7.x-1.0-rc1
Status: Closed (fixed) » Active

Subscription from block does not redirect to page set in config.
I saw that there no such code in function block_submit

drummondf’s picture

Issue summary: View changes

Also experiencing this issue with block subscription, although path is provided it is not redirecting