Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
After clicking on the link in the confirmation mail I get an "Access denied"
The subscriber doesn´t get confirmed.
Comments
Comment #1
ParisLiakos CreditAttribution: ParisLiakos commentedDo anonymous users have
subscribe newsletters
permission?Comment #2
anni CreditAttribution: anni commentedYes.
Comment #3
ParisLiakos CreditAttribution: ParisLiakos commentedwhats the URL like?
Comment #4
anni CreditAttribution: anni commentedHi,
it´s "...newsletter/confirm?hash=k60TQyNwstUXYtSCn84QjbGlN670GrQMTd6efSZMOzU".
Comment #5
ParisLiakos CreditAttribution: ParisLiakos commentedhmm i have no clue as to what could cause this behavior:/
any entries in reports recent logs (dblog) ?
Comment #6
anni CreditAttribution: anni commentedYes.
Type: access denied
Date: Thursday, February 23, 2012 - 12:51pm
User: Anonymous (not verified)
Location: .../newsletter/confirm?hash=k60TQyNwstUXYtSCn84QjbGlN670GrQMTd6efSZMOzU
Referrer:
Message: newsletter/confirm
Severity: warning
Hostname: ...
Operations:
Comment #7
ParisLiakos CreditAttribution: ParisLiakos commentedsorry,no idea whats causing this.maybe another module you are using?
Comment #8
anni CreditAttribution: anni commentedMaybe simplenews?
I´ll try a clean installation and will report back.
Comment #9
ParisLiakos CreditAttribution: ParisLiakos commentedYes simplenews could be the cause, since it has a menu entry
newsletter/confirm
just like newsletter module does.Simplenews should use
simplenews/confirm
instead.Thus i am sending this to simplenews issue queue
Comment #10
miro_dietikerWe prefer to state "Simplenews conflicts Newsletter module".
First simplenews was first to "allocate" the /newsletter/ module namespace.
Then, simplenews provides concepts like "Newsletter" and not a "Simplenews". We decided during name cleanups to confirm this previous name also.
Also if we once will implement an entity for the Newsletter, the entity will have a name like "Newsletter" and not "Simplenews"...
Even if you think this might be wrong and a namespace in the URL should be prefixed with the module name, we won't be able to do that. Beta level guarantees API stability and the callback URL is an API relevant thing. We won't break millions of already sent links.
I still vote for a merge of those modules and i also don't see a reason to use both at the same time.
Comment #11
ParisLiakos CreditAttribution: ParisLiakos commentedAgreed on most points but
Module's name and URL here in d.o was and is the namespace.It is the best practice to use your module's namespace to avoid incompatibilities like these.
It was the wrong thing to do and still is.Just because it happened doesnt make it right:)
That said, i agree, no point to break existing links and API (especially when its highly unlikely for one to use these 2 modules together), but it should be noted as a thing to fix in the future.I would say "postponed" is the right status here
Comment #12
BerdirAgreed with postponed, we might look into this later. I am not sure if the namespacing rules apply to menu paths like they do to function names but I also agree that newsletter doesn't belong to us just because we were here first.
And yes, would be wrong to change it now and break all existing links and integrations.
I will add a point to the known problems section.
Comment #13
anni CreditAttribution: anni commented@rootatwc
So, at the time there is no chance to get a working confirm link (in newsletter module) if newsletter and simplenews is installed both?
Comment #14
BerdirNo, there isn't.
Why would you want to use both modules at the same time anyway, that makes no sense?
Comment #15
anni CreditAttribution: anni commentedBecause I need functions from newsletter module (include Views, send newsletters after x days, etc.), but encountered bugs.
So I had to find a solution for standard newsletters and thats the good old simplenews.
Comment #16
BerdirSee http://drupal.org/project/simplenews_scheduler for recurring newsletters and inserting views can for example be done in the templates.
Comment #17
anni CreditAttribution: anni commentedHmmmmm, why not. I´ll give it a try.
Thank you!
Comment #18
ParisLiakos CreditAttribution: ParisLiakos commentedgreat, we can come back to this and fix it once the 2.x-dev branch opens:)
Comment #19
Abelito CreditAttribution: Abelito commentedIm not sure how different the module is for Drupal 6, but I submitted a patch for the D6 version here: http://drupal.org/node/1801300
- I have tested it, and it seems to be working just fine.
Comment #20
miro_dietikerReopening in 7.x-2.x.
We can discuss now about a new naming scheme.
Comment #21
j.johnson.bbt CreditAttribution: j.johnson.bbt commentedSuppose someone installed both Simplenews and Newsletter and then they uninstall both.
How do they go about deleting this menu entry "newsletter/confirm" so they can start over with a fresh install? Thanks.
Comment #22
AdamPS CreditAttribution: AdamPS at AlbanyWeb for think modular - digital solutions GmbH commentedFixed in 4.x by #3392981: Final tidy up of subscription routes