the is_integer check is unnecessary / invalid / breaking the functionality of flickr_set_load, since filckr album ids exceed php's maximum integer representation. This may be an issue w/ my php version, but it definitely fails for, e.g. album id # 72157615809178832

this patch addresses this issue by changing is_integer to is_numeric.

This will not guarantee that the parameter is a valid set id, but is sufficient to prevent malicious usage.
Flickr's error handling will take care of any numeric-but-invalid ids.

CommentFileSizeAuthor
flickr_sets.module.patch474 bytesaaronbauman

Comments

rjanson’s picture

rjanson commented

I've applied this patch and verified that my flickr sets are loading properly again.

Anonymous’s picture

Anonymous (not verified) commented
Assigned: aaronbauman »
Status: Needs review » Reviewed & tested by the community

Thanks , ill get this committed shortly.

Best, Paul

Anonymous’s picture

Anonymous (not verified) commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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