[buddypress-trac] [BuddyPress Trac] #6503: Separate functions for creating a new nav link and registering a screen function.

[buddypress-trac]

#6503: Separate functions for creating a new nav link and registering a screen
function.
------------------------------+-----------------------
 Reporter:  dcavins           |       Owner:
     Type:  enhancement       |      Status:  reopened
 Priority:  normal            |   Milestone:  2.4
Component:  Component - Core  |     Version:  1.2
 Severity:  blocker           |  Resolution:
 Keywords:  has-patch         |
------------------------------+-----------------------

Comment (by dcavins):

 Yes, thanks for asking. This test feels like a bit of a hack, but `bp_nav`
 is a slippery little feller to test. I've attached a new patch that
 • adds a test that shows an example failure (I'd love advice on how to
 make this test better.)
 • ensures that `bp_core_new_nav_item()` and `bp_core_new_subnav_item()`
 return `false` in certain cases where the screen function registration
 fails. (Removing the early access check demonstrated this problem.)

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