At Pantheon, we use extensive caching around image_get_info() to avoid network file system traffic. Calling getimagesize() bypasses this cache with bad consequences for users browsing many images through IMCE.

For quick reference, this is the call we'd like it to use:
http://api.drupal.org/api/drupal/includes%21image.inc/function/image_get...

Making excessive calls to getimagesize() and related file system functions on network mounts is a known problem that even has its own module for certain cases: http://drupal.org/project/imageinfo_cache

Pantheon will award a bounty of $100 for getting this simple fix into stable releases of IMCE for Drupal 6 and 7 by November 8, 2012.

The included patch is for Drupal 7.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

David Strauss’s picture

Issue summary: View changes

phrasing

David Strauss’s picture

Issue summary: View changes

phrasing

David Strauss’s picture

Issue summary: View changes

deadline

David Strauss’s picture

Issue summary: View changes

D6

ufku’s picture

Version: master » 7.x-1.x-dev
Category: bug » task
Priority: Major » Normal
FileSize
1.68 KB

For the majority of drupal installations imce_image_info is faster than image_get_info. I'm not willing to change how it works though I can provide a way to switch to image_get_info.

The patch allows to switch between two versions of the function by checking a variable.

David Strauss’s picture

That would be perfect. We can set that variable for our users to make use of the cache. Happy to pay the bounty for the proposed patch getting committed and made available in a stable release.

Thanks!

ufku’s picture

Status: Active » Fixed

Committed to dev branches. No need for the bounty. Thanks.

David Strauss’s picture

Awesome! Do you know when we can expect this to make it into a release? I mostly posted the bounty to encourage an update to the stable release, which is currently just over a year old.

ufku’s picture

I'll review a few more issues in the following days, after which I plan to make a new release.

Look to its coming, at first light, on the fifth day.

David Strauss’s picture

Look to its coming, at first light, on the fifth day.

:-)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Date