[buddypress-trac] [BuddyPress Trac] #7226: Update BP_buttons class to accept new arg param for $element_type

buddypress-trac noreply at wordpress.org
Fri Aug 26 01:32:25 UTC 2016


#7226: Update BP_buttons class to accept new arg param for $element_type
-------------------------+------------------
 Reporter:  hnla         |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  2.7
Component:  Core         |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------

Comment (by boonebgorges):

 Ha, I was in the middle of cleaning @hnla's patch when I got an email
 about @r-a-y's patch.

 I was going to suggest that we handle the change in syntax in a new
 ticket, but since the patch is written, I think it's cool to go with it. I
 agree that the logic and the syntax is much clearer.

 @r-a-y Did you write tests to test the backward compatibility stuff? Or
 did you just check to see if existing buttons were rendering properly?

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


More information about the buddypress-trac mailing list