I'm trying to get Flickr Sync setup and it's giving me a bit of grief. I've entered the API info, setup the content type, authenticated etc.

When I run cron or manually try to import, I get these errors for each photo:

Cannot search for blank username
User doesn't exist, unable to save image 4365433155

I went as far as manually pushing in the admin (which has a flickr account in the profile) and then I get this:

Failed to retrieve jpeg file from Flickr.com, url =
Couldn't download file, unable to save image 4365433155

If I run it through cron, I also get some error messages as well, which are related to the returned info from the API I believe:

Undefined index: owner in flickrsync_node_save() (line 400)
Trying to get property of non-object in flickrsync_create_node() (line 346)

Any suggestions?

Comments

claytical’s picture

Status: Active » Needs review

It looks like the issue is the array itself. In flickrsync.module on line 391:

$flickr_photo = $f->photos_getInfo($photo_id);

returns an array inside of an array of ['photo']

I just added one line below it:

$flickr_photo = $flickr_photo['photo'];

...and everything imports now. There seems to be a vocab error that exists on import, although it doesn't seem to be affecting the actual process.

claytical’s picture

Assigned: Unassigned » claytical
Status: Needs review » Patch (to be ported)
StatusFileSize
new435 bytes

Here's the fix as a patch.

schtink’s picture

Had the same error on D7.14. Patch works perfectly. Thank you!

naringas’s picture

That patch worked for me too (#2, #1)... you should guys should commit it... thank you.

biswajitroy’s picture

Thanks a lots. This patch is working fine. I was also receiving the save error message "Cannot search for blank username
User doesn't exist, unable to save image"

biswajitroy’s picture

Status: Patch (to be ported) » Active

When I run cron then I have received another error message "Notice: Undefined offset: 1 in flickrsync_download_photo() (line 583 of /../../flickrsync/flickrsync.module).". Can any one help me for this issue ?

infines’s picture

Status: Active » Patch (to be ported)

Please open up another issue.

elly’s picture

Thanks for the patch - just FYI the patch works against 7.x-1.0-beta2+3-dev as well.

gclicon’s picture

Assigned: claytical » gclicon
Status: Patch (to be ported) » Postponed