Refactored solution to issue #239

collective · Aug 6, 2013 · 361e7a9 · 361e7a9
1 parent 3aeef44
commit 361e7a9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/collective/cover/tiles/collection.py b/src/collective/cover/tiles/collection.py
@@ -158,10 +158,15 @@ def get_configured_fields(self):
         return results
 
     def thumbnail(self, item):
+        tile_conf = self.get_tile_configuration()
+        image_conf = tile_conf.get('image', None)
         scales = item.restrictedTraverse('@@images')
         try:
-            scale = [i['scale'].split(' ')[0] for i in self.get_configured_fields() if i['id'] == 'image'][0]
-            return scales.scale('image', scale)
+            if image_conf:
+                scaleconf = image_conf['imgsize']
+                # scale string is something like: 'mini 200:200'
+                scale = scaleconf.split(' ')[0] # we need the name only: 'mini'
+                return scales.scale('image', scale)
         except:
             return None
 

diff --git a/src/collective/cover/tiles/templates/collection.pt b/src/collective/cover/tiles/templates/collection.pt
@@ -31,7 +31,8 @@
                 <img style="float: left; margin: 0 1em 0.5em 0;"
                      tal:attributes="src thumbnail/url;
                                      width thumbnail/width;
-                                     height thumbnail/height" />
+                                     height thumbnail/height;
+                                     alt obj/Description|obj/Title" />
               </a>
             </tal:image>