WordPress.org

Make WordPress Core

Opened 22 months ago

Closed 21 months ago

#18644 closed defect (bug) (fixed)

Media list table doesn't check for comment support

Reported by: johnbillion Owned by: nacin
Priority: normal Milestone: 3.3
Component: Media Version: 3.3
Severity: normal Keywords: has-patch
Cc:

Description

If you remove support for comments from attachments*, the Comment column is still displayed on the Media screen. This is inconsistent with posts and pages, where the column gets removed.

*

remove_post_type_support( 'attachment', 'comments' );

Attachments (2)

18644.patch​ (881 bytes) - added by johnbillion 22 months ago.
18644.2.patch​ (897 bytes) - added by SergeyBiryukov 21 months ago.

Download all attachments as: .zip

Change History (8)

johnbillion — 22 months ago

comment:1 johnbillion — 22 months ago

  • Keywords has-patch added

Patch

comment:2 nacin — 22 months ago

  • Milestone changed from Awaiting Review to 3.3
  • Owner set to nacin
  • Status changed from new to accepted

comment:3 jane — 21 months ago

Yes, please fix this.

SergeyBiryukov — 21 months ago

comment:4 SergeyBiryukov — 21 months ago

18644.2.patch​ also fixes missing i18n in alt text.

comment:5 nacin — 21 months ago

This should be esc_attr__ in the future.

comment:6 nacin — 21 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In [18931]:

Check post_type_supports before adding comments column to attachments. Translate a string. props johnbillion, SergeyBiryukov. fixes #18644.

Note: See TracTickets for help on using tickets.