[wp-trac] [WordPress Trac] #37526: Introduce the possibility to register new administration panels

WordPress Trac noreply at wordpress.org
Thu Oct 6 22:22:55 UTC 2016


#37526: Introduce the possibility to register new administration panels
-------------------------------------------------+-------------------------
 Reporter:  flixos90                             |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
Component:  Administration                       |  Review
 Severity:  normal                               |     Version:
 Keywords:  2nd-opinion has-patch needs-unit-    |  Resolution:
  tests                                          |     Focuses:  multisite
-------------------------------------------------+-------------------------

Comment (by flixos90):

 [attachment:37526.5.diff] improves the
 `get_current_administration_panel()` method so that it no longer needs to
 support a parameter. The `is_*_admin()` functions already take care of
 checking whether to use the current screen or the constant for detection.

 Also the `constant_name` property is not used anymore. However I think it
 should remain part of the class since every administration panel should
 have its own constant to signalize that it's active - just to make this
 more clear to the user, I would leave it in. But I'm open for discussion.

 Anyway, does anyone have feedback on this whole thing? :)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/37526#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list