Ticket #638 (reopened defect)

Opened 3 years ago

Last modified 2 days ago

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

Changed 3 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

Changed 10 months ago by ray

  • milestone set to 0.96

Changed 2 days ago by douglas

  • status changed from new to closed
  • resolution set to wontfix

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

Changed 2 days ago by ray

  • status changed from closed to reopened
  • resolution deleted

Changed 2 days ago by ray

Very likely this applies to EFM, too

Note: See TracTickets for help on using tickets.