Skip to:
Content

BuddyPress.org

Opened 20 months ago

Last modified 7 weeks ago

#7999 new enhancement

Support 'user_ids' argument in 'bp_has_members()`

Reported by: Mamaduka Owned by:
Milestone: 7.0.0 Priority: normal
Severity: normal Version: 1.7
Component: Members Keywords:
Cc:

Description

This task requires BP_Core_Members_Template::construct() refactoring to use array-style arguments. Patch is provided in #3797.

Changes will also affect following classes/functions:

  • BP_Core_Members_Template
  • bp_core_get_users()

Attachments (1)

7999.diff (4.8 KB) - added by Mamaduka 20 months ago.

Download all attachments as: .zip

Change History (6)

@Mamaduka
20 months ago

#1 @boonebgorges
20 months ago

  • Milestone changed from Awaiting Review to 4.0

This looks like a good and easy improvement.

#2 follow-up: @boonebgorges
20 months ago

  • Milestone changed from 4.0 to Up Next

Oh, I see now that we have to change the function signature of BP_Core_Members_Template first. This is more work that should be supported by tests. Let's do it next.

#3 in reply to: ↑ 2 @Mamaduka
20 months ago

Replying to boonebgorges:

Oh, I see now that we have to change the function signature of BP_Core_Members_Template first. This is more work that should be supported by tests. Let's do it next.

Thanks for Boone.

Should I add unit test for user_ids argument?

#4 @boonebgorges
20 months ago

Thanks, George! I'm more concerned about having tests for the changes in https://buddypress.trac.wordpress.org/attachment/ticket/3797/3797.core-members-template.2.diff. I see that you changed the way that the functions are used in the tests, which is a good start, but I'll want to have a closer look to make sure that we're properly converting deprecated arguments. This might mean one or two tests that pass the deprecated arguments (with @expectedDeprecated) that then check things like pagination, etc on the results.

#5 @imath
7 weeks ago

  • Milestone changed from Up Next to 7.0.0
Note: See TracTickets for help on using tickets.