[wp-trac] [WordPress Trac] #21031: Orphaned comments emit debug notices in dashboard and comments lists (was: map_meta_cap emits notices if $post is empty)

WordPress Trac wp-trac at lists.automattic.com
Thu Jun 21 04:23:06 UTC 2012


#21031: Orphaned comments emit debug notices in dashboard and comments lists
------------------------------+------------------------------
 Reporter:  johnjamesjacoby   |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  Awaiting Review
Component:  Warnings/Notices  |     Version:
 Severity:  normal            |  Resolution:
 Keywords:  has-patch         |
------------------------------+------------------------------
Description changed by johnjamesjacoby:

Old description:

> map_meta_cap is not prepared to handle missing or invalid post ID's. To
> duplicate this, delete a post from the posts table without deleting its
> attached comments, then visit the admin dashboard with the recent
> comments widget visible.
>
> Update: this isn't something that happens naturally in WordPress core,
> but is something plugins are able to do directly to the database, which
> results in orphaned comments. The attached patches only suppress debug
> notices PHP spits out, and does not directly address the issue of what to
> do with orphaned comments, should a plugin choose to do this.

New description:

 map_meta_cap is not prepared to handle missing or invalid post ID's. To
 duplicate this, delete a post from the posts table without deleting its
 attached comments, then visit the admin dashboard with the recent comments
 widget visible.

 The comments list table exhibits a similar issue, where the $post global
 is assumed to be set and valid.

 This isn't something that happens naturally in WordPress core, but is
 something plugins are able to do directly to the database, which results
 in orphaned comments. The attached patches only suppress debug notices PHP
 spits out, and does not directly address the issue of what to do with
 orphaned comments, should a plugin choose to do this.

--

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21031#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list