1
0
mirror of https://github.com/e107inc/e107.git synced 2025-01-17 12:48:24 +01:00

Bugtracker #4522 - can now upsize images. Make thumb.php consistent, maintain single copy in e107_images

This commit is contained in:
e107steved 2008-10-03 20:28:54 +00:00
parent e0261976d9
commit 466b70c50a
3 changed files with 115 additions and 151 deletions

View File

@ -1,45 +0,0 @@
<?php
/*
+ ----------------------------------------------------------------------------+
| e107 website system
|
| ©Steve Dunstan 2001-2002
| http://e107.org
| jalist@e107.org
|
| Released under the terms and conditions of the
| GNU General Public License (http://gnu.org).
|
| $Source: /cvs_backup/e107_0.8/e107_files/thumb.php,v $
| $Revision: 1.1.1.1 $
| $Date: 2006-12-02 04:33:37 $
| $Author: mcfly_e107 $
+----------------------------------------------------------------------------+
*/
/*
Usage: simply replace your <img src='filename.jpg'
with
<img src='".e_IMAGE."thumb.php?filename.jpg+size"' />
or
<img src='".e_IMAGE."thumb.php?<full path to file>/filename.jpg+size"' />
eg <img src='".e_IMAGE."thumb.php?home/images/myfilename.jpg+100)"' />
*/
require_once("../class2.php");
require_once(e_HANDLER."resize_handler.php");
if (e_QUERY){
$tmp = explode("+",rawurldecode(e_QUERY));
if(strpos($tmp[0], "/") === 0 || strpos($tmp[0], ":") >= 1){
$source = $tmp[0];
}else{
$source = "../".str_replace("../","",$tmp[0]);
}
$newsize = $tmp[1];
if(!resize_image($source, "stdout", $newsize)){
echo "Couldn't find: ".$source;
}
}
?>

View File

@ -11,11 +11,10 @@
| GNU General Public License (http://gnu.org).
|
| $Source: /cvs_backup/e107_0.8/e107_handlers/resize_handler.php,v $
| $Revision: 1.8 $
| $Date: 2008-01-12 16:51:43 $
| $Revision: 1.9 $
| $Date: 2008-10-03 20:28:54 $
| $Author: e107steved $
|
| Mod to give correct return code if source image already smaller than max size
|
+----------------------------------------------------------------------------+
*/
@ -25,11 +24,14 @@ function resize_image($source_file, $destination_file, $type = "upload", $model
{
// $destination_file - 'stdout' sends direct to browser. Otherwise treated as file name
// - if its a file, given '644' permissions
// $type - "upload"
// - numeric - sets new width of image
// - anything else - default preference for image width used, or failing that, 120 pixels
// 'avatar' is used
// $model - if "copy", creates a new file from $destination_file with the prefix 'thumb_'.
// $type - numeric - sets new width of image
// - "upload" - uses preference 'im_width', or 400px if not defined
// - anything else - default preference for image width & heightused, or failing that, 120 px x 100 px
// 'avatar' may be used to invoke the default (see usersettings.php)
// $model - "copy" - creates a new file for the destination, by prefixing $destination_file with 'thumb_'. Return error for small images.
// - 'upsize' - small images are enlarged
// - 'noscale' - small images are transferred at their original size
// - 'nocopy' - used in content manager plugin
// Otherwise overwrites any existing $destination_file
// Returns: TRUE - essentially, if $destination_file (or a file with a modified name) is valid:
@ -56,8 +58,9 @@ function resize_image($source_file, $destination_file, $type = "upload", $model
$new_height = varset($pref['im_height'], 100);
}
// $im_quality = varset($pref['im_quality'], 99);
$im_quality = 99;
$im_quality = varset($pref['im_quality'], 99);
$image_stats = getimagesize($source_file);
if ($image_stats == null)
@ -83,7 +86,39 @@ function resize_image($source_file, $destination_file, $type = "upload", $model
{ // Nothing to do if image width already smaller than the maximum
// If we were basically ensuring an existing file was within limits, return TRUE
// If we had to create a new file, return FALSE since it wasn't done.
return (($source_file == $destination_file) && ($model != 'copy'));
switch ($model)
{
case 'copy' : // Not sure what to do here!
return FALSE; // This is what it used to do
break;
case 'upsize' : // Scale source up to required size
break; // Just fall through to do that.
case 'noscale' : // No scaling of small images- just want destination to be the same as source
if ($destination_file == 'stdout')
{
$fileExt = strtolower(substr(strrchr($source_file, "."), 1));
$mimeTypes = array(
'jpg' => 'jpeg',
'gif' => 'gif',
'png' => 'png',
'jpeg' => 'jpeg',
'pjpeg' => 'jpeg',
'bmp' => 'bmp'
);
if (!isset($mimeTypes[$fileExt])) { return FALSE; } // only allow image files }
header("Content-type: image/".$mimeTypes[$fileExt]);
if (@readfile($source_file) === FALSE) { return FALSE; }
}
else
{
return copy($source_file,$destination_file);
}
return TRUE;
break;
default :
return ($source_file == $destination_file);
}
// return (($source_file == $destination_file) && ($model != 'copy'));
}
$ratio = ($imagewidth / $new_size);
@ -96,30 +131,30 @@ function resize_image($source_file, $destination_file, $type = "upload", $model
}
if (($destination_file != 'stdout') && ($model == 'copy'))
{
$destination_file = dirname($destination_file).'/thumb_'.basename($destination_file);
}
$returnError = 0; // Return value from some of the commands
switch ($mode)
{
case "ImageMagick" :
if ($destination_file == "stdout")
{ /* if destination is stdout, output directly to the browser */
{ // if destination is stdout, output directly to the browser
// $destination_file = "jpg:-";
header("Content-type: image/jpeg");
// Use double quotes instead of single to keep Bill happy
passthru ($pref['im_path']."convert -quality ".$im_quality." -antialias -geometry ".$new_size."x".$new_imageheight." ".escapeshellarg($source_file)." \"jpg:-\"");
passthru ($pref['im_path']."convert -quality ".$im_quality." -antialias -geometry ".$new_size."x".$new_imageheight." ".escapeshellarg($source_file)." \"jpg:-\"", $returnError);
}
else
{ /* otherwise output to file */
if ($model == "copy")
{
$name = substr($destination_file, (strrpos($destination_file, "/")+1));
$name2 = "thumb_".$name;
$destination_file = str_replace($name, $name2, $destination_file);
}
{ // otherwise output to file
// Use double quotes instead of single to keep Bill happy
// exec ($pref['im_path']."convert -quality ".$im_quality." -antialias -geometry ".$new_size."x".$new_imageheight." ".escapeshellarg($source_file)." '".$destination_file."'");
exec ($pref['im_path']."convert -quality ".$im_quality." -antialias -geometry ".$new_size."x".$new_imageheight." ".escapeshellarg($source_file)." \"".$destination_file."\"");
exec ($pref['im_path']."convert -quality ".$im_quality." -antialias -geometry ".$new_size."x".$new_imageheight." ".escapeshellarg($source_file)." \"".$destination_file."\"", $dummy, $returnError);
}
if ($returnError) echo "ImageMagick resize/output error: {$returnError}<br />";
break;
case "gd1" :
case 'gd1' :
case 'gd2' :
switch ($image_stats[2])
{
case IMAGETYPE_PNG : // 3 - PNG
@ -139,71 +174,43 @@ function resize_image($source_file, $destination_file, $type = "upload", $model
{
return FALSE;
}
$dst_img = imagecreate($new_size, $new_imageheight); // Create blank image of correct size as target
// Only next line is different between gd1 and gd2
imagecopyresized($dst_img, $src_img, 0, 0, 0, 0, $new_size, $new_imageheight, $imagewidth, $imageheight);
if ($model == "copy")
if ($mode == 'gd1')
{
$name = substr($destination_file, (strrpos($destination_file, "/")+1));
$name2 = "thumb_".$name;
$destination_file = str_replace($name, $name2, $destination_file);
}
if ($destination_file == "stdout")
{
header("Content-type: image/jpeg");
imagejpeg($dst_img, '', $im_quality);
$dst_img = imagecreate($new_size, $new_imageheight); // Create blank image of correct size as target
if (!imagecopyresized($dst_img, $src_img, 0, 0, 0, 0, $new_size, $new_imageheight, $imagewidth, $imageheight)) { $returnError = -4; }
}
else
{
imagejpeg($dst_img, $destination_file, $im_quality);
imagedestroy($src_img);
imagedestroy($dst_img);
$dst_img = imagecreatetruecolor($new_size, $new_imageheight);
if (!imagecopyresampled($dst_img, $src_img, 0, 0, 0, 0, $new_size, $new_imageheight, $imagewidth, $imageheight)) { $returnError = -5; }
}
break;
case "gd2" :
switch ($image_stats[2])
{
case IMAGETYPE_PNG : // 3 - PNG
$src_img = @imagecreatefrompng($source_file);
break;
case IMAGETYPE_GIF : // 1 - GIF
if (!function_exists('imagecreatefromgif')) return FALSE; // Some versions of GD library don't support GIF
$src_img = @imagecreatefromgif($source_file);
break;
case IMAGETYPE_JPEG : // 2 - Jpeg
$src_img = @imagecreatefromjpeg($source_file);
break;
default :
return FALSE; // Unsupported image type
}
if (!$src_img)
if ($returnError)
{
echo "Resizing error: {$returnError}<br />";
return FALSE;
}
$dst_img = imagecreatetruecolor($new_size, $new_imageheight);
// Only next line is different between gd1 and gd2
imagecopyresampled($dst_img, $src_img, 0, 0, 0, 0, $new_size, $new_imageheight, $imagewidth, $imageheight);
if ($model == "copy")
{
$name = substr($destination_file, (strrpos($destination_file, "/")+1));
$name2 = "thumb_".$name;
$destination_file = str_replace($name, $name2, $destination_file);
}
if ($destination_file == "stdout")
{
header("Content-type: image/jpeg");
imagejpeg($dst_img, '', $im_quality);
if (!imagejpeg($dst_img, '', $im_quality)) $returnError = -1;
}
else
{
imagejpeg($dst_img, $destination_file, $im_quality);
imagedestroy($src_img);
imagedestroy($dst_img);
if (!imagejpeg($dst_img, $destination_file, $im_quality)) { $returnError = -1; }
}
if (!imagedestroy($src_img)) { $returnError = -2; }
if (!imagedestroy($dst_img)) { $returnError = -3; }
if ($returnError)
{
echo "Resizing error: {$returnError}<br />";
return FALSE;
}
break;
default :
echo "Invalid resize function: {$mode}<br />";
return FALSE;
} // End switch($mode)
if ($destination_file == "stdout") return TRUE; // Can't do anything more if file sent to stdout - assume success

View File

@ -11,53 +11,55 @@
| GNU General Public License (http://gnu.org).
|
| $Source: /cvs_backup/e107_0.8/e107_images/thumb.php,v $
| $Revision: 1.1.1.1 $
| $Date: 2006-12-02 04:34:25 $
| $Author: mcfly_e107 $
| $Revision: 1.2 $
| $Date: 2008-10-03 20:28:54 $
| $Author: e107steved $
+----------------------------------------------------------------------------+
*/
/*
Usage: simply replace your <img src='filename.jpg'
with
<img src='".e_IMAGE."thumb.php?filename.jpg+size"' />
<img src='".e_IMAGE_ABS."thumb.php?filename.jpg+size"' />
or
<img src='".e_IMAGE."thumb.php?<full path to file>/filename.jpg+size"' />
eg <img src='".e_IMAGE."thumb.php?home/images/myfilename.jpg+100)"' />
<img src='".e_IMAGE_ABS."thumb.php?<full path to file>/filename.jpg+size"' />
eg <img src='".e_IMAGE_ABS."thumb.php?home/images/myfilename.jpg+100)"' />
By default a small image is upsized. To render the image unchanged, append '+noscale', thus:
eg <img src='".e_IMAGE_ABS."thumb.php?home/images/myfilename.jpg+100+noscale)"' />
*/
require_once("../class2.php");
require_once(e_HANDLER."resize_handler.php");
// var 3.
// 1= newspost images/preview
// 2=
if (e_QUERY){
$tmp = explode("+",rawurldecode(e_QUERY));
if(strpos($tmp[0], "/") === 0 || strpos($tmp[0], ":") >= 1){
$source = $tmp[0];
}else{
$source = "../".str_replace("../","",$tmp[0]);
}
$newsize = $tmp[1];
if(!file_exists(e_IMAGE."newspost_images/preview/preview_".basename($source))){
if(!resize_image($source, e_IMAGE."newspost_images/preview/preview_".basename($source), $newsize)){
echo "Couldn't find: ".$source;
}
}
header("Content-type: image/jpg");
$imagedata = file_get_contents(e_IMAGE."newspost_images/preview/preview_".basename($source));
echo $imagedata;
exit;
if (e_QUERY)
{
$tmp = explode('+',rawurldecode(e_QUERY));
if(strpos($tmp[0], '/') === 0 || strpos($tmp[0], ":") >= 1)
{
$source = $tmp[0]; // Full path to image specified
}
else
{
$source = "../".str_replace('../','',$tmp[0]);
}
if (!$source)
{
echo "No image name.<br />";
exit;
}
$newsize = intval($tmp[1]);
if (($newsize < 5) || ($newsize > 4000)) // Pretty generous limits
{
echo "Bad image size: {$newsize}<br />";
exit;
}
$opts = varset($tmp[2],'upsize');
if(!resize_image($source, 'stdout', $newsize, $opts))
{
echo "Couldn't find: {$source}<br />";
}
}
?>