Opened 2 years ago
Last modified 22 months ago
#42791 assigned enhancement
wp_add_dashboard_widget() should accept location and priority parameters
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | good-first-bug has-patch has-unit-tests |
Focuses: | Cc: | ||
PR Number: |
Description
The wp_add_dashboard_widget()
function should accept $location
and $priority
parameters to bring the function inline with add_meta_box()
, which it ultimately calls.
Attachments (5)
Change History (10)
#3
@
22 months ago
- Keywords needs-unit-tests added
- Owner set to davidjlaietta
- Status changed from new to assigned
Hi @davidjlaietta, thanks for the patch! I've assigned the ticket to you to mark this good-first-bug
as "claimed". Sorry for the delay in response.
I did a first pass on the patch, and the big standouts that are still missing are changelog entries for the new parameters in the DocBlock, and we could probably also benefit from unit tests here as well.
For info on changelog entries, check out this article in the core handbook.
Note: See
TracTickets for help on using
tickets.
Attached diff includes $location and $priority arguments, and maintains similar logic for wp_add_dashboard_widget.
If location is not set, it'll be set to normal unless it's a side widget. If priority is not set it will be set to core unless dashboard_browser_nag is set, then it'll be set to high.
New usage: