[buddypress-trac] [BuddyPress Trac] #7335: Anonymize BP Core Dependency functions file.

buddypress-trac noreply at wordpress.org
Mon Nov 28 21:36:37 UTC 2016


#7335: Anonymize BP Core Dependency functions file.
-----------------------------------+------------------
 Reporter:  tw2113                 |       Owner:
     Type:  enhancement            |      Status:  new
 Priority:  normal                 |   Milestone:  2.8
Component:  Core                   |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |
-----------------------------------+------------------
Changes (by DJPaul):

 * keywords:  has-patch => has-patch 2nd-opinion


Comment:

 Looks good but it makes these files much longer and adds functions in a
 file that was originally designed to not really have (m)any functions in.
 The priorities are also a little harder to read this way. Would be
 interested in opinions from other people.

 Maybe assign anonymous functions to variables and use those to replace the
 add_action lines, keeping them short and tidy. Maybe this is just me over-
 reacting to a niggle.

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


More information about the buddypress-trac mailing list