Make WordPress Core

Opened 2 years ago

Last modified 2 years ago

#56607 new defect (bug)

Improve doc for WP_Screen class

Reported by: chouby's profile Chouby Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.3
Component: Administration Keywords: has-patch
Focuses: docs Cc:

Description

WooCommerce 6.9 introduced a conflict with several plugins by accessing to the property WP_Screen::id and expecting it to be a string, while conflicting plugins just set the current screen with a simple call to set_current_screen() without parameter. In this case, get_current_screen() returns a WP_Screen object with the property id set to null.

This pushed me to review the doc of the class and notice that several properties are not correctly documented for this case.

Attachments (1)

56607.patch (1.4 KB) - added by Chouby 2 years ago.

Download all attachments as: .zip

Change History (2)

@Chouby
2 years ago

#1 @Chouby
2 years ago

  • Keywords has-patch added

Just for reference, the issue opened in WooCommerce: https://github.com/woocommerce/woocommerce/issues/34725

Note: See TracTickets for help on using tickets.