Flickr provider

heynemann · heynemann · commit 5a2804a35070 · 2011-07-30T23:56:36.000-03:00
diff --git a/myimgat/providers/flickr.py b/myimgat/providers/flickr.py
@@ -117,7 +117,7 @@ def load_default_photos(self, album):
             'page': 1
         }
 
-        result = urlopen(url).read()
+        result = urlopen(page_url).read()
         result = loads(result)
 
         def parse_results(items):
@@ -139,7 +139,7 @@ def parse_results(items):
             for page in range(total_pages - 1):
                 page_url = url % {
                     'api_key': settings.FLICKR_API_KEY,
-                    'photoset_id': album.identifier,
+                    'user_id': self.user_id,
                     'page': page + 1
                 }
 
diff --git a/tests/test_flickr_provider.py b/tests/test_flickr_provider.py
@@ -30,14 +30,9 @@ def test_albums_can_load_their_photos():
     for album in albums:
         provider.load_photos(album=album)
 
-    for photo in album.photos:
-        assert photo.title
-        assert photo.url
-        assert photo.thumbnail
-        print photo.url
-        print photo.thumbnail
-
-
-
+        for photo in album.photos:
+            assert photo.title
+            assert photo.url
+            assert photo.thumbnail
 
 

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ def load_default_photos(self, album):`
`117`	`117`	`'page': 1`
`118`	`118`	`}`
`119`	`119`
`120`		`- result = urlopen(url).read()`
	`120`	`+ result = urlopen(page_url).read()`
`121`	`121`	`result = loads(result)`
`122`	`122`
`123`	`123`	`def parse_results(items):`
`@@ -139,7 +139,7 @@ def parse_results(items):`
`139`	`139`	`for page in range(total_pages - 1):`
`140`	`140`	`page_url = url % {`
`141`	`141`	`'api_key': settings.FLICKR_API_KEY,`
`142`		`- 'photoset_id': album.identifier,`
	`142`	`+ 'user_id': self.user_id,`
`143`	`143`	`'page': page + 1`
`144`	`144`	`}`
`145`	`145`