[Email notification] multi page thread
floridave
Joined: 2003-12-22
Posts: 27300 |
Posted: Thu, 2007-03-08 22:25 |
When one receives an email notification of a update to a thread they are in this form: Quote:
New comment from: mindless Clicking the link takes you to the first page of a multi page thread. On single page threads it works fine. The behavior a week or 2 ago was, on multi page threads the link would go to the post regardless of the thread being multi page or not. Dave |
|
Posts: 8601
hm. the latest version of subscriptions.module has some long complicated code for figuring out the page number, so I figured it worked and removed my custom code. i'll take a look.
Posts: 8601
oof, the module doesn't even pass the right parameters to its own function.. fixing, thanks for the report!
Posts: 8601
Ok, should be fixed.. let me know if it doesn't work.
Posts: 27300
Thanks for that mindless, just letting you know that it does not work still:
Glad you are keeping the Drupal guys on their toes.
Dave
_____________________________________________
Blog & G2 || floridave - Gallery Team
Posts: 8601
oof, did that too fast.. one typo and one wrong param name. "should" be ok now
Posts: 27300
perhaps third time is the charm.
Dave
_____________________________________________
Blog & G2 || floridave - Gallery Team
Posts: 8601
Let's hope so. The code after my 2nd fix does work, but gets it wrong on just the 51st post (exact post that starts the 2nd page), which that one above just happened to be... now fixed.