Skip to:
Content

bbPress.org

Opened 13 years ago

Closed 13 years ago

#1827 closed defect (bug) (fixed)

Methods that extend BP_Component and BP_Groups_Component can't be marked private

Reported by: boonebgorges's profile boonebgorges Owned by:
Milestone: Priority: normal
Severity: major Version:
Component: Extend - BuddyPress Keywords:
Cc:

Description

In r3904, you marked a bunch of methods in BBP_Forums_Group_Extension and BBP_Forums_Component private or protected. This causes fatal errors, because access for the parent methods is assumed to be public in BP_Group_Extension and BP_Component.

Moreover, marking them private means that it won't be possible to unhook and then rehook these methods from outside the class, using remove_action() and add_action() (since do_action() calls call_user_func_array() outside the scope of the class) . I don't see a reason for disallowing this kind of hook/unhook - we should default to public unless there is a good reason to mark private/protected.

Attachments (2)

1827.patch (2.1 KB) - added by boonebgorges 13 years ago.
1827.2.patch (2.3 KB) - added by boonebgorges 13 years ago.
Missed one

Download all attachments as: .zip

Change History (3)

@boonebgorges
13 years ago

@boonebgorges
13 years ago

Missed one

#1 @johnjamesjacoby
13 years ago

  • Milestone 2.1 deleted
  • Resolution set to fixed
  • Status changed from new to closed

Thanks. Fixed these before I saw this ticket.

Fixed in r3908 and r3909.

Note: See TracTickets for help on using tickets.