[buddypress-trac] [BuddyPress Trac] #6606: Visibility issues with BP_Activity_List_Table::row_actions()

buddypress-trac noreply at wordpress.org
Tue Sep 1 19:40:25 UTC 2015


#6606: Visibility issues with BP_Activity_List_Table::row_actions()
--------------------------+------------------------------
 Reporter:  tw2113        |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  API           |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------------------

Comment (by johnjamesjacoby):

 A few things:

 * `WP_List_Table::row_actions()` is protected in 4.3, and as far back as
 [WP28493]
 * BuddyPress uses `protected` in Activity & Groups, but `public` in
 Members.
 * BuddyPress claims to support as far back to WordPress 3.8, but
 `row_actions()` was an assumed `public` method until WordPress 4.0.

 Ways to address this:
 * Make BuddyPress's `row_actions()` methods public, which should still
 function on top of `protected` methods in WordPress 4.3 without fatal
 errors, though this should be tested to confirm. Debug notices here would
 be acceptable IMO, considering the circumstances.
 * Bump the support version to WordPress 4.0 in the next BuddyPress release
 (2.4) so this becomes a non-issue, support wise.
 * The base `WP_List_Table` class was always extended at our own risk, so
 we could, in theory, invent our own `BP_List_Table` base class that is
 essentially a copy of `WP_List_Table` but specialized to the needs of
 BuddyPress. This is a bad idea, but it's an option that may eventually be
 necessary if `WP_List_Table` support is never officially supported.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6606#comment:2>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list