[wp-trac] Re: [WordPress Trac] #10136: is_active_sidebar() should use wp_get_sidebars_widgets()

[WordPress Trac]

#10136: is_active_sidebar() should use wp_get_sidebars_widgets()
-------------------------------+--------------------------------------------
 Reporter:  greenshady         |        Owner:  azaozz  
     Type:  enhancement        |       Status:  reopened
 Priority:  normal             |    Milestone:  2.8.1   
Component:  Widgets            |      Version:  2.8     
 Severity:  normal             |   Resolution:          
 Keywords:  needs-patch close  |  
-------------------------------+--------------------------------------------
Changes (by Denis-de-Bernardy):

  * keywords:  => needs-patch close
  * type:  defect (bug) => enhancement


Comment:

 Sorry, read this a bit too quickly. There also are unregister_sidebar()
 and unregister_sidebar_widget(), btw. At any event:

  - wp_sidebars_widgets() is slower than is needed here
  - we may get side effects due to the built-in recent comments widget (or
 is that recent posts?), which calls it as part of the widget
  - there are several plugins that let you disable a widget on a per
 context basis, and these plugins can now access the sidebar (it's
 $args[id])

 Suggesting wontfix either way.

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