In some development code I noticed that when handling File objects without
filename values, there would be problems calling getPath and such.
The width and height value testing will be validated later anyway, and by
removing such a narrow test we can use events to generate thumbnails of
media formats supported by recently added plugins on demand.
*/
public function getThumbnail($width=null, $height=null, $crop=false)
{
*/
public function getThumbnail($width=null, $height=null, $crop=false)
{
- if (intval($this->width) < 1 || intval($this->height) < 1) {
- // Old files may have 0 until migrated with scripts/upgrade.php
- // For any legitimately unrepresentable ones, we could generate our
- // own image (like a square with MIME type in text)
- throw new UnsupportedMediaException('No image geometry available.');
- }
-
if ($width === null) {
$width = common_config('thumbnail', 'width');
$height = common_config('thumbnail', 'height');
if ($width === null) {
$width = common_config('thumbnail', 'width');
$height = common_config('thumbnail', 'height');
}
// throws exception on failure to generate thumbnail
}
// throws exception on failure to generate thumbnail
- $outname = "thumb-{$width}x{$height}-" . $this->filename;
+ $outname = "thumb-{$width}x{$height}-" . $image->filename;
$outpath = self::path($outname);
// The boundary box for our resizing
$outpath = self::path($outname);
// The boundary box for our resizing
// Doublecheck that parameters are sane and integers.
if ($box['width'] < 1 || $box['width'] > common_config('thumbnail', 'maxsize')
|| $box['height'] < 1 || $box['height'] > common_config('thumbnail', 'maxsize')
// Doublecheck that parameters are sane and integers.
if ($box['width'] < 1 || $box['width'] > common_config('thumbnail', 'maxsize')
|| $box['height'] < 1 || $box['height'] > common_config('thumbnail', 'maxsize')
- || $box['w'] < 1 || $box['x'] >= $this->width
- || $box['h'] < 1 || $box['y'] >= $this->height) {
+ || $box['w'] < 1 || $box['x'] >= $image->width
+ || $box['h'] < 1 || $box['y'] >= $image->height) {
// Fail on bad width parameter. If this occurs, it's due to algorithm in ImageFile->scaleToFit
// Fail on bad width parameter. If this occurs, it's due to algorithm in ImageFile->scaleToFit
- common_debug("Boundary box parameters for resize of {$this->filepath} : ".var_export($box,true));
+ common_debug("Boundary box parameters for resize of {$image->filepath} : ".var_export($box,true));
throw new ServerException('Bad thumbnail size parameters.');
}
throw new ServerException('Bad thumbnail size parameters.');
}
+ common_debug(sprintf('Generating a thumbnail of File id==%u of size %ux%u', $this->id, $width, $height));
// Perform resize and store into file
$image->resizeTo($outpath, $box);
// Avoid deleting the original
// Perform resize and store into file
$image->resizeTo($outpath, $box);
// Avoid deleting the original
- if ($image->getPath() != $this->getPath()) {
+ if ($image->getPath() != self::path($image->filename)) {
$image->unlink();
}
return File_thumbnail::saveThumbnail($this->id,
$image->unlink();
}
return File_thumbnail::saveThumbnail($this->id,