[wp-trac] [WordPress Trac] #19414: Filter 'kses_allowed_protocols' is only applied once in function wp_allowed_protocols() & function esc_url() returns empty string;

[WordPress Trac]

#19414: Filter 'kses_allowed_protocols' is only applied once in function
wp_allowed_protocols()  & function esc_url() returns empty string;
--------------------------+------------------------------
 Reporter:  Anatta        |       Owner:
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Security      |     Version:  3.3
 Severity:  major         |  Resolution:  wontfix
 Keywords:  close         |
--------------------------+------------------------------
Changes (by Anatta):

 * status:  new => closed
 * resolution:   => wontfix


Comment:

 Replying to [comment:6 nacin]:
 > There's a 'meta' argument for add_node() that allows for an 'onclick'
 argument.

 Thanks nacin; code is indeed poetry - there's just a lot of it to read!, I
 got caught up tracing why my filtered logout url was not being passed to
 the my-account admin bar menu and didn't consider the simpler solution of
 replacing the menu item.

 For anyone googling similar symptoms, this ticket should serve as a
 helpful resource.  I've successfully added Facebook javascript to the
 admin bar logout with the following:

 {{{
 add_action( 'wp_before_admin_bar_render', 'fb_admin_bar_logout');

 function fb_admin_bar_logout() {

         global $wp_admin_bar

         $wp_admin_bar->add_menu( array(
                         'parent' => 'my-account',
                         'id'     => 'logout',
                         'title'  => __( 'Logout' ),
                         'href'   => wp_logout_url(),
                         'meta' => array( 'onclick' => 'FB.logout();'
                 ) );
 }
 }}}

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19414#comment:7>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software