[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