[wp-trac] [WordPress Trac] #19779: _edit_attachments_query_helper does not properly preface table name when querying unattached media

WordPress Trac wp-trac at lists.automattic.com
Sun Jan 8 20:12:43 UTC 2012


#19779: _edit_attachments_query_helper does not properly preface table name when
querying unattached media
--------------------------+-----------------------------------
 Reporter:  benbalter     |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  General       |    Version:  3.3.1
 Severity:  minor         |   Keywords:  has-patch 2nd-opinion
--------------------------+-----------------------------------
 If a plugin modifies the media list on `upload.php` by joining `wp_posts`
 on itself (for example, to filter attachments which are children of a
 certain post type), MySQL throws an error when querying for  unattached
 files because `post_parent < 1` in `_edit_attachments_query_helper` is
 ambiguous without prefacing the table name (because of the join). All
 other columns in the query are explicitly named.

 Steps to Reproduce:

 1. Hook into `posts_join_paged` on `upload.php` and join wp_posts on
 itself ( e.g., `JOIN wp_posts wp_posts2 ON wp_posts2.post_parent =
 wp_posts.ID`)
 2. Navigate to media upload and query for unattached media
 3. Receive an error that `post_parent` is ambiguous

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19779>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list