Opened 2 years ago
Closed 2 years ago
#53394 closed defect (bug) (fixed)
@global $wp_widget_factory is used but not define in documentation of this function sanitize_widget_instance( $value, $id_base = null )
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.8 | Priority: | normal |
Severity: | normal | Version: | 5.7 |
Component: | Widgets | Keywords: | has-patch |
Focuses: | docs | Cc: |
Description
@global $wp_widget_factory is used but not define in documentation of this function sanitize_widget_instance( $value, $id_base = null )
Attachments (12)
Change History (19)
#1
follow-up:
↓ 2
@
2 years ago
- Component changed from Media to Widgets
- Keywords needs-refresh added
#2
in reply to:
↑ 1
@
2 years ago
- Keywords needs-refresh removed
Hi @audrasjb,
Thanks for sharing this thing and I have uploaded a new patch with the all the above changes
#3
follow-up:
↓ 4
@
2 years ago
Thank you. However, for WP coding standards compliance, we need a newline before and after the @global declaration. To clarify, here is what we need:
* @since 3.9.0 * * @global WP_Widget_Factory $wp_widget_factory * * @param array $value Widget instance to sanitize.
(feel free to delete the previous diff files when your patch is somehow uploaded multiple times ;)
Note: See
TracTickets for help on using
tickets.
Hello @jontyravi, thank you for the patch(s),
It looks like there is a small indentation issue on the line you added. Also, there should be a new empty line between @since and @global declarations.