r11176 - people/evgeni/thumbnails-from-screenshots

[Evgeni Golov]

Author: evgeni
Date: 2010-07-29 21:10:55 +0000 (Thu, 29 Jul 2010)
New Revision: 11176

Modified:
   people/evgeni/thumbnails-from-screenshots/fetch-thumbnails
Log:
jump the big dict only once

Modified: people/evgeni/thumbnails-from-screenshots/fetch-thumbnails
===================================================================
--- people/evgeni/thumbnails-from-screenshots/fetch-thumbnails	2010-07-29 21:02:37 UTC (rev 11175)
+++ people/evgeni/thumbnails-from-screenshots/fetch-thumbnails	2010-07-29 21:10:55 UTC (rev 11176)
@@ -51,8 +51,11 @@
 if not path.exists(__THUMB_DIR):
     mkdir(__THUMB_DIR)
 
-screenshots = json.load(urllib2.urlopen(__JSON_URL))
-screenshots = screenshots['screenshots']
+screenshots_json = json.load(urllib2.urlopen(__JSON_URL))
+screenshots_json = screenshots_json['screenshots']
+screenshots = {}
+for s in screenshots_json:
+    screenshots[s['name']] = s['large_image_url']
 
 db = debtags.DB()
 db.read(open(__DEBTAGS_FILE, "r"))
@@ -64,10 +67,10 @@
     p = {}
     if 'role::program' in db.tags_of_package(g):
         p['name'] = g
-        p['screenshot'] = None
-        for s in screenshots:
-            if s['name'] == g:
-                p['screenshot'] = s['large_image_url']
+        if p['name'] in screenshots.keys():
+            p['screenshot'] = screenshots[p['name']]
+        else:
+            p['screenshot'] = None
         work.append(p)
 
 for p in work: