Skip to:
Content

Opened 2 years ago

Closed 2 years ago

#4023 closed defect (bug) (fixed)

groups_check_group_exists() - documentation/codex states parameter is group_id, whereas is actually group_slug

Reported by: rcain Owned by:
Milestone: 1.6 Priority: normal
Severity: trivial Version: 1.5.3
Component: Groups Keywords:
Cc:

Description

function groups_check_group_exists() - documentation/codex states parameter is group_id, whereas BP_Groups_Group::group_exists() (which it calls) actually requires group_slug

note: further issue - since this function actually (and usefully!) returns group_id given group_slug it should logically be renamed to eg: groups_get_id($group_slug) (ie. the converse of existing function groups_get_slug(group_id).

note: this refers to BP V 1.5.4 - not available yet on 'version' drop down on this ticket system.

Change History (2)

comment:1 boonebgorges2 years ago

  • Milestone changed from Awaiting Review to 1.6

This is a good suggestion.

The codex can be edited by anyone. If you see anything that should be changed, please correct it.

comment:2 boonebgorges2 years ago

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

(In [5808]) Deprecates groups_check_group_exists() in favor of more aptly named groups_get_id(). Fixes #4023

Note: See TracTickets for help on using tickets.