Make WordPress Core

Opened 3 years ago

Last modified 4 weeks ago

#43639 new defect (bug)

wp_terms_checklist post_id argument is null, should be 0

Reported by: tareiking Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Taxonomy Keywords: has-patch
Focuses: administration Cc:


In includes/class-wp-posts-list-table.php we call wp_terms_checklist with post_id set to null.

wp_terms_checklist post_id argument should be an int and 0 is used as the default elsewhere (ie: ajax-actions.php).

Reference: https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/class-wp-posts-list-table.php#L1543

Change History (5)

#1 @pento
2 years ago

  • Version trunk deleted

This ticket was mentioned in Slack in #core-site-health by hareesh-pillai. View the logs.

8 months ago

#3 @SergeyBiryukov
8 months ago

  • Component changed from General to Taxonomy
  • Keywords needs-patch added

This ticket was mentioned in PR #1173 on WordPress/wordpress-develop by donmhico.

4 weeks ago

  • Keywords has-patch added; needs-patch removed

Pass 0 instead of null on wp_terms_checklist() for consistency.

Trac ticket: https://core.trac.wordpress.org/ticket/43639

#5 @donmhico
4 weeks ago

Thanks for the ticket @tareiking. I've attached a PR that fixes this issue.

Note: See TracTickets for help on using tickets.