Make WordPress Core

Opened 15 years ago

Closed 11 years ago

#19872 closed enhancement (maybelater)

Add Hook/Filter to image_resize_dimensions in media.php

Reported by: webbtj Owned by:
Priority: normal Milestone:
Component: Media Version: 3.3.1
Severity: normal Keywords:
Cc: Focuses:

Description

It would be great if a hook, a filter, a global, an option, or something somewhere were created to allow the image_resize_dimensions() function in media.php to resize images to be larger than the original. In a typical scenario I can understand why you would not want to allow this, however there are rare circumstances where this would be helpful (generating a background image). Currently lines 349 and 350 of media.php read:

$new_w = min($dest_w, $orig_w);
$new_h = min($dest_h, $orig_h);

and lines 378 to 380 read:

// if the resulting image would be the same size or larger we don't want to resize it
if ( $new_w >= $orig_w && $new_h >= $orig_h )
	return false;

It would be great if we were provided a boolean to alter these two blocks, for example:

if($allow_resize){
	$new_w = $dest_w;
	$new_h = $dest_h;
}else{
	$new_w = min($dest_w, $orig_w);
	$new_h = min($dest_h, $orig_h);
}

...

// if the resulting image would be the same size or larger we don't want to resize it
if ( $new_w >= $orig_w && $new_h >= $orig_h && !$allow_resize)
	return false;

Change History (5)

#1 @scribu
15 years ago

  • Milestone Awaiting Review
  • Resolution → wontfix
  • Status new → closed

There already are two hooks in the wp_generate_attachment_metadata() function: 'intermediate_image_sizes_advanced' and 'wp_generate_attachment_metadata'.

They should be enough.

#2 @scribu
15 years ago

  • Component General → Media
  • Keywords 2nd-opinion added
  • Milestone → Awaiting Review
  • Resolution wontfix
  • Status closed → reopened
  • Type feature request → enhancement

Sorry, you were obviously talking about something else.

#3 @ericlewis
12 years ago

Related / dupe: #21294

Version 0, edited 12 years ago by ericlewis (next)

#4 @wonderboymusic
12 years ago

#21294 was marked as a duplicate.

#5 @chriscct7
11 years ago

  • Keywords 2nd-opinion removed
  • Milestone Awaiting Review
  • Resolution → maybelater
  • Status reopened → closed

There doesn't seem to be much interest in adding these filters. Closing as maybelater.

Note: See TracTickets for help on using tickets.