When creating a Flickr CCK field, and using a photoset ID, the URL under the "Source: Flickr" link on the node is incorrect. It is of the individual photo form rather than the photoset form.

I found the location of the bug and fixed it locally. However, I'm on Windows and not running CVS at the moment, and I just don't have the time to set it all up to create a patch file, so if someone wants to do that it'd be great. I'll just attach the new flickrfield.module for now. Also, details are below.

flickrfield.module, Line 165
Change: $photo_url = flickr_photo_page_url($photo_data['owner'], $photo_data['id']);
To: $photo_url = flickr_photoset_page_url($photo_data['owner'], $photo_data['id']);

Comments

soxofaan’s picture

Version: 5.x-1.3 » 5.x-1.x-dev
Status: Active » Reviewed & tested by the community
StatusFileSize
new990 bytes

confirmed that the suggested change fixes the problem

patch in attachment

Anonymous’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev
Assigned: Unassigned »

This problem looks to be resolved in D6, can someone else confirm this ?

Thanks Paul

Anonymous’s picture

Status: Reviewed & tested by the community » Fixed

Thanks.

ill get this patch committed to the development branch later today

Status: Fixed » Closed (fixed)

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