Skip to:
Content

BuddyPress.org

Opened 5 years ago

Closed 5 years ago

#6394 closed defect (bug) (fixed)

Clean up page number parameter for AJAX pagination in bp_filter_request()

Reported by: dcavins Owned by: dcavins
Milestone: 2.3 Priority: normal
Severity: normal Version: 2.2.3
Component: Appearance - Template Pack Keywords: has-patch
Cc:

Description

The pagination click handler in buddypress.js sends kind of an ugly pagination parameter because of the way it parses the page number from the pagination link. (It sends something like $_POST['page']: 2&num ). This can cause problems for BP Group Hierarchy in certain cases, so I suspect it might cause trouble for other plugins, too.

My patch just changes the parsing a bit so that the $_POST['page'] parameter sent is only the page number.

Attachments (1)

6394.01.patch (973 bytes) - added by dcavins 5 years ago.
Clean up page number sent in AJAX pagination requests

Download all attachments as: .zip

Change History (2)

@dcavins
5 years ago

Clean up page number sent in AJAX pagination requests

#1 @dcavins
5 years ago

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

In 9809:

Clean up page param in AJAX pagination request.

When including the page parameter in AJAX pagination requests, strip
out trailing arguments. E.g.: instead of sending '2&num,’ send '2'.

Fixes #6394.

Note: See TracTickets for help on using tickets.