WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#9245 closed enhancement (fixed)

Inconsistent use of imagejpeg() calls - missing apply_filter

Reported by: DJPaul Owned by:
Milestone: 2.8 Priority: low
Severity: trivial Version:
Component: General Keywords: has-patch imagejpeg
Focuses: Cc:

Description

Hi

In /wp-admin/includes/image.php line 71, could you add a call to apply_filters as a third argument so we can control the quality of the image via plugin? This is already done elsewhere in Wordpress - see /wp-includes/media.php line 381. Thanks.

Attachments (1)

imagejpeg_filter.9245.diff (1.0 KB) - added by filosofo 5 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 DJPaul5 years ago

  • Cc DJPaul added

comment:2 filosofo5 years ago

  • Keywords has-patch imagejpeg added

My patch adds a filter there, using the default quality of 90 used at the other imagejpeg call, and I included a second argument of context for both.

comment:3 FFEMTcJ5 years ago

  • Milestone changed from Unassigned to 2.8
  • Type changed from defect (bug) to enhancement

comment:4 westi5 years ago

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

In [10682] - Allow a plugin to change the jpeg image quality. Fixes #9245 props filosofo.

Note: See TracTickets for help on using tickets.