Incorrect links in tag comment notifications when there is more than one page of comments

Description

Originally reported on Google Code with ID 3773
What archive revision are you testing on? otwarchive v0.9.9.6

If appropriate, enter the URL of a page where the problem can be seen: http://archiveofourown.org/tags/Abrahamic%20Religions/comments

What steps will reproduce the problem?
1. Make a comment
2. Wait for the email with the notification.
3. Click one of the following links:

  • Reply to this comment

  • Go to the thread starting from this comment

  • Go to the thread to which this comment belongs.

What is the expected output? What do you see instead?

Link should take you to your comment within the wrangling interface. Instead it takes
you to the first page of the comments on the Abrahamic Religions tag.

Issue has been confirmed to exist on the Supernatural and Harry Potter tags as well,
which means that it extends to any tag that has more than 1 page of comment

Bug may be related to: http://code.google.com/p/otwarchive/issues/detail?id=3748 (404
on tags with multiple pages of comments) & http://code.google.com/p/otwarchive/issues/detail?id=2085
(borked notification emails)

(The Links in the attached email should lead to the following comment: http://archiveofourown.org/comments/4537807)
<hr>

Activity

Show:
lady.oscar.ao3
September 20, 2013, 11:03 AM
enigel lj
September 20, 2013, 9:29 PM
  • *Labels added*: Roadmap-TagWrangling

theoldbriarpipe
August 21, 2015, 9:32 AM
  • *Labels added*: Keep-TW

redsummernight
December 25, 2020, 4:26 AM
  1. Logged in as user A, a tag wrangler. Assigned self to a fandom. Left a comment on the tag.

  2. Logged in as user B, also a tag wrangler. Left a reply to the previous comment.

  3. User A received an email notification with the following links:

    1. Read all comments on Hololive

    2. Reply to this comment

    3. Go to the thread starting from this comment

    4. Go to the thread to which this comment belongs

  4. The text notification for user A had the same links:

Looks good.

james_
December 25, 2020, 7:49 AM

Tried on a normal tag and all looks well.

Then replied and got four links.

Looks good.

DeployedToBeta

Assignee

ticking instant

Reporter

morriganschild

Roadmap

Comments
Tags
Tag Wrangling

Priority

High

Affects versions

None

Fix versions

Components

BackEnd

Difficulty

Medium

Required Access Level

Tag Wrangler

Milestone

Internal 0.9

Google Code Issue ID

3773