![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 14 months ago
Closed 14 months ago
#63525 closed defect (bug) (reported-upstream)
Doc comment code improvement.
| Reported by: | upadalavipul | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Editor | Version: | |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: | docs |
Description
I have reviewed the code in the file below and noticed that we are using the global $wp_widget_factory variable with the data type declared as int, which I believe is incorrect. This variable is actually an object of the WP_Widget_Factory class. Therefore, I suggest updating the function comment to reflect the correct data type.
File: wp-includes/blocks/legacy-widget.php
Attachments (1)
Change History (3)
#1
@
14 months ago
- Keywords close added
Hello and thanks for the patch,
However, this file is not managed in WordPress Core but on Gutenberg side. Please rather open a ticket on Gutenberg repo: https://github.com/WordPress/gutenberg/issues
Marking this as close as it needs to be reported upstream.
#2
@
14 months ago
- Component General → Editor
- Keywords close removed
- Milestone Awaiting Review
- Resolution → reported-upstream
- Status new → closed
Thanks! No need to open another issue; it was already fixed in https://github.com/WordPress/gutenberg/pull/69775
patch Added.