Making WordPress.org

Opened 8 years ago

Closed 6 years ago

Last modified 2 years ago

#2110 closed defect (bug) (fixed)

Forums: Wrong page number in some reply links

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by:
Milestone: Priority: normal
Component: Support Forums Keywords:
Cc:

Description (last modified by SergeyBiryukov)

  1. Open this reply:
    https://wordpress.org/support/topic/after-update-to-46-keep-getting-curl-error-23/page/2/#post-7680728
    (That particular topic is already fixed.)
  2. The next reply on the same page suddenly has /page/3/ instead of /page/2/:
    https://wordpress.org/support/topic/after-update-to-46-keep-getting-curl-error-23/page/3/#post-8157376

Steps to reproduce:

  1. Create a topic and two pages of replies.
  2. Archive or mark as spam some replies on the first page.
  3. The first page now displays some replies from the second page, but their URLs still link to the second page.

Change History (8)

#1 @SergeyBiryukov
7 years ago

In 5693:

Support Forums: Update the necessary meta data when editing a reply created before 2017-07-17, as those replies can have potentially inaccurate data.

See #2110, #2481.

#2 @SergeyBiryukov
7 years ago

In 5695:

Support Forums: In Dropin::update_old_reply_meta(), don't recalculate position for archived replies.

Archived replies are not returned by bbp_get_all_child_ids(), so they end up with a zero position.

See #2110, #2481.

#3 @SergeyBiryukov
7 years ago

  • Description modified (diff)

#4 @SergeyBiryukov
7 years ago

  • Description modified (diff)

This ticket was mentioned in Slack in #forums by clorith. View the logs.


7 years ago

#6 @obenland
6 years ago

@SergeyBiryukov What's the status on this?

#7 @tellyworth
6 years ago

  • Resolution set to fixed
  • Status changed from new to closed

Closing as fixed, please re-open if that's not correct.

Note: See TracTickets for help on using tickets.