I am not certain if the Flickr module already has support for original-size photos. As far as I can tell, every time I try to get the original of a photo using the Flickr module, it fails. I get the wrong URL and when trying to load the image, all I can see is an error image from Flickr saying the picture is currently unavailable.

I had to patch the flickr.inc file in order to allow the module to fetch the original size. When trying to obtain the URL of an original, it will use the "originalsecret" value rather than the usual "secret" value to generate the URL.

The patch is attached to this issue page for everyone's review.

CommentFileSizeAuthor
flickr.inc-originalsize.patch676 bytesremi

Comments

remi’s picture

remi commented

I just realised the code style in the patch file is all messed up because I was using tabs instead of spaces. My apologies. At least, it works!

lolandese’s picture

lolandese commented
Issue summary: View changes
Status: Active » Needs review

Changed status as there is a patch to review.

Thanks.

lolandese’s picture

lolandese commented
Version: 6.x-1.1 » 7.x-1.x-dev
Status: Needs review » Needs work

The patch needs to be rerolled for D7 if still relevant (it seems so).

For D6 see #2367755: D6 maintenance.

lolandese’s picture

lolandese commented
Status: Needs work » Closed (cannot reproduce)

Just tested the D7 Flickr Filter with:
[flickr-photo:id=4258169177, size=o, mintitle=999, minmetadata=999]

It gets the original size without using $photo['originalsecret']. In D7 this doesn't seem to be an issue.

I realize the patch was made for D6 for which it might have been relevant. Feel free to reopen if you think this issue is still relevant for D7. We only do critical bug fixes on D6 now (see #2367755: D6 maintenance)

Thanks.