Skip to:
Content

bbPress.org

Opened 15 years ago

Closed 15 years ago

#1589 closed defect (bug) (fixed)

Topics tag pagination fails

Reported by: inexpert's profile inexpert Owned by:
Milestone: 2.0 Priority: normal
Severity: major Version: 2.0
Component: Component - Topic Tags Keywords: has-patch
Cc:

Description

Using the bbPress Plugin from the repository.

When I click on a tag I get the right list of topics, but the link to the second page of the list is not correct.

I would like to help but I don't know the wordpress core structure sorry, and I did not find any kind of documentation about it. It would take too much time for me to understand it by my self.
If there is any documentation please refer me to it.

Thank you.

Attachments (1)

bbp-topic-template.php.patch​ (495 bytes) - added by inexpert 15 years ago.
Patch for Topic Tag Pagination

Download all attachments as: .zip

Change History (5)

#1 @inexpert
15 years ago

  • Severity changed from normal to major

#2 @cnorris23
15 years ago

  • Milestone changed from Awaiting Review to 2.0
  • Summary changed from Topics list searching by tag to Topics tag pagination fails

Confirmed.

@inexpert
15 years ago

Patch for Topic Tag Pagination

#3 @inexpert
15 years ago

  • Keywords has-patch added

First of all congratulations, you confirmed this bug in a really short time.
Secondly, I feel kind of comfortable with the code.

I figured it out... the bug was just that "Topic Tag" is not considered when rewriting the URL. So here is a patch for those who want to fix it, although is probably not as pretty as it should.

I am probably not using existing functions or not applying filters or whatever. I apologise but I am new to the code.

Last edited 15 years ago by inexpert (previous) (diff)

#4 @johnjamesjacoby
15 years ago

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

(In [3386]) Fix issue where pagination on topic tag page was not working. This uncovered underlying issues with how posts and globals were reset as part of theme compatibility. Fixes #1589.

Note: See TracTickets for help on using tickets.