Issue 20 in flickrpy: Add getAllContexts method (patch included)

0 views
Skip to first unread message

flic...@googlecode.com

unread,
Dec 20, 2010, 9:03:54 AM12/20/10
to flickrp...@googlegroups.com
Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 20 by notfadea...@btopenworld.com: Add getAllContexts method
(patch included)
http://code.google.com/p/flickrpy/issues/detail?id=20

I needed to use the getAllContexts method as documented at
http://www.flickr.com/services/api/flickr.photos.getAllContexts.html to get
a count of pools/sets for a photo. I couldn't see this in the existing code
so have added it, patch file is attached.

leejo@ubuntu:~/bin/flickrpy-read-only$ svn info flickr.py
Path: flickr.py
Name: flickr.py
URL: http://flickrpy.googlecode.com/svn/trunk/flickr.py
Repository Root: http://flickrpy.googlecode.com/svn
Repository UUID: f27205ce-482c-0410-bd43-d700e4c7180c
Revision: 48
Node Kind: file
Schedule: normal
Last Changed Author: clarkeje
Last Changed Rev: 48
Last Changed Date: 2010-11-05 22:50:08 +0000 (Fri, 05 Nov 2010)
Text Last Updated: 2010-12-19 12:23:37 +0000 (Sun, 19 Dec 2010)
Checksum: 96de42728ec44e9337c86b36f0aefecb

Attachments:
flickr.py.getAllContexts.patch 1.5 KB

flic...@googlecode.com

unread,
Dec 23, 2010, 9:13:41 AM12/23/10
to flickrp...@googlegroups.com

Comment #1 on issue 20 by notfadea...@btopenworld.com: Add getAllContexts

Oops, there's a bug. Line 24 reads:

d["pools"].append({"id":pool.id,"title":pool.title})

Should read:

d["sets"].append({"id":set.id,"title":set.title})

flic...@googlecode.com

unread,
May 2, 2012, 12:24:01 PM5/2/12
to flickrp...@googlegroups.com
Updates:
Status: Accepted
Owner: jims...@gmail.com

Comment #2 on issue 20 by jims...@gmail.com: Add getAllContexts method
(patch included)
http://code.google.com/p/flickrpy/issues/detail?id=20

(No comment was entered for this change.)

flic...@googlecode.com

unread,
May 2, 2012, 12:36:28 PM5/2/12
to flickrp...@googlegroups.com
Updates:
Status: Fixed
Labels: -Type-Defect Type-Enhancement

Comment #3 on issue 20 by jims...@gmail.com: Add getAllContexts method
(patch included)
http://code.google.com/p/flickrpy/issues/detail?id=20

Patch applied, thanks!

Reply all
Reply to author
Forward
0 new messages