There are some variables in the simplenews_confirm_subscription function that are not named consistently. Breaks the confirmation page and causes a 404.

CommentFileSizeAuthor
#1 simplenews-921172.patch2.63 KBJackinloadup

Comments

Jackinloadup’s picture

Jackinloadup commented
Status: Active » Needs review
StatusFileSize
new simplenews-921172.patch2.63 KB
miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented

Code looks fine to me. Coming back to apply later. Are there other reviews/opinions?

simon georges’s picture

simon georges commented
Status: Needs review » Reviewed & tested by the community

Code looks fine, patch applies cleanly.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

committed.

Status: Fixed » Closed (fixed)

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