[buddypress-trac] [BuddyPress] #3312: Allow replacement of groups object in group component loading

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Fri Jul 8 14:50:13 UTC 2011


#3312: Allow replacement of groups object in group component loading
--------------------------+-----------------
  Reporter:  ddean        |      Owner:
      Type:  enhancement  |     Status:  new
  Priority:  normal       |  Milestone:  1.3
 Component:  Groups       |    Version:  1.3
Resolution:               |   Keywords:
--------------------------+-----------------

Comment (by ddean):

 There are pros and cons to each way. This looks like the state of the
 discussion:

 ==== Filter instantiated object: ====
 Pros
  - more consistent with other BP and WP filters
  - allows plugin authors to modify the group object without subclassing or
 creating a replacement `BP_Groups_Group` class
 Cons
  - uses more system resources

 ==== Filter class name: ====
 Pros
  - uses fewer system resources
  - a better match for this case
 Cons
  - requires plugin authors to subclass or replace the `BP_Groups_Group`
 class in order to use the filter
  - breaks with convention for filters in BP and WP

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3312#comment:8>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list