Download & Extend

Errorcheck in line 87 should have $settree['rsp']['err'] instead of $top_level_tree...

Project:Flickr Module
Version:master
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

The check for error returned in line 87 of flickr.module is wrong, I believe.

It is currently:
    if (!empty($top_level_tree['rsp']['err']) || empty($settree))

but it should be
    if (!empty($settree['rsp']['err']) || empty($settree))

This causes an error instead of the expected response if the API key is invalid.

Comments

#1

Status:active» closed (fixed)

This is ancient history and the current module is in great shape.