Opened 14 years ago

Closed 11 years ago

#638 closed defect (wontfix)

ImageManager sometimes fails to display thumbnails

Reported by: koto@… Owned by: yermol
Priority: normal Milestone: 0.96
Component: Plugin_ImageManager Version:
Severity: normal Keywords:
Cc:

Description

ImageManager? plugin does not display existing thumbnail files from images within subdirectories of $IMConfigimages_dir? if $IMConfigthumbnail_dir? is an empty string (i.e. if thumbnails are put in a same directory as image files).
The <img> tag produced by plugin interface (rendered by images.php) refers to non-existing file, completely ignoring the subdirectories and assuming the thumbnail resides in $IMConfigimages_dir?.

E.g.: instead of referring to thumbnail automatically created during upload (http://example.com/images/subdir/second/t_image.jpg ) it refers to http://example.com/images/t_image.jpg

Such behaviour occurs only when $IMConfigthumbnail_dir? is an empty string.

Proposed (working) fix:

plugins\ImageManager?\Classes\ImageManager?.php


Line 315 (getThumbUrl() function) should read:

Return Files::makeFile($this->getImagesURL() . $path_parts['dirname'], $thumbnail);

instead of:

Return Files::makeFile($this->getImagesURL(), $thumbnail);

Change History (6)

comment:1 Changed 13 years ago by anonymous

Hey, I had to the same problem,
the fastest and the easier way to fix it is set change in file
plugins/ImageManager/config.inc.php
on the line numer 161
$IMConfigthumbnail_dir? = '.';
this fix to cause a thumnail directory
is (become) the same directory as our image directory,
this change take of the problem with unvisible thumbnails

comment:2 Changed 12 years ago by ray

  • Milestone set to 0.96

comment:3 Changed 11 years ago by douglas

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

ImageManager? has been deprecated, it will no loner be developed...

comment:4 Changed 11 years ago by ray

  • Resolution wontfix deleted
  • Status changed from closed to reopened

comment:5 Changed 11 years ago by ray

Very likely this applies to EFM, too

comment:6 Changed 11 years ago by douglas

  • Resolution set to wontfix
  • Status changed from reopened to closed

This bug is fixed in PersistantStorage?, which is a reworking of ExtendedFileManager?

Note: See TracTickets for help on using tickets.