I have some very large photsets on Flickr and they don't seem to work with this module. Sets with around 20 photos work fine, but when I try to open a set with around 500-1000 photo's, the site just gives up trying after a while.

Any idea why this is so slow? Viewing the same set on my mobile phone using an Flickr app works fine, so I presume its not the Flickr API that is the bottleneck.

Comments

donnycarette’s picture

donnycarette commented
Assigned: Unassigned » donnycarette

Hi Baris,

I'm aware of the problem.
Module was developed for a site that had a Flickr Pro account where it didn't see to be a problem.
This issue is already added to my todolist, to look again for pulling the images.
Also in the next release it should be possible to enable caching for the images (see: http://drupal.org/node/1346976)
Only problem is to found some free time to take a look at this, hope to do this in a couple of weeks

BarisW’s picture

BarisW commented

Hi Donny,

I've tried implementing the cache, but that only caches the API call. The call itself is not the problem, but the theming of the results makes it so slow. I just found out that one line is the cause of the problem:

$image_meta = $f->photos_getInfo($photo['id']);

If I uncomment that line, all works fine. Is this line really needed?

donnycarette’s picture

donnycarette commented

Hi Baris,

I added that line to support extra information, if someone wants to extract more information about the image and override the template file.
If that line is causing the slow load, i'll look for other options.

I did a quick test, seems to load better. But should need to test with larger image set, which i don't have available at this time.
Also tested after clearing browser cache?

BarisW’s picture

BarisW commented

Yeah, did test after caching.

FYI, I testes with these sets: http://www.hetkamp.nl/flickr

If you want, you can test with my API key: http://www.flickr.com/services/apps/72157628295673647/key/

donnycarette’s picture

donnycarette commented

Looks to load pretty fast indeed, nice!
Your API key page is private, so i can't access it
Will do some more testing with the image_meta or collecting the extra information
Thanks for helping!

Ps The module should contain a dutch translation file, still need to look why that's is not imported while enabling the module

donnycarette’s picture

donnycarette commented
Assigned: donnycarette » Unassigned
Status: Active » Needs review

Fixed in new 7.x-2.0-beta2 release

donnycarette’s picture

donnycarette commented
Status: Needs review » Fixed

Changing status to fixed

Status: Fixed » Closed (fixed)

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