From: http://kerravonsen.dreamwidth.org/ Date: Wed, 18 Dec 2013 05:04:17 +0000 (-0400) Subject: fix for thumbnail size bug X-Git-Url: https://sipb.mit.edu/gitweb.cgi/ikiwiki.git/commitdiff_plain/6f2420e9b8122e2e95385694a71a0ff53cdd050e?hp=978688977ef3d6eef14cde992e86b0752aef09d4 fix for thumbnail size bug --- diff --git a/doc/plugins/contrib/album/discussion.mdwn b/doc/plugins/contrib/album/discussion.mdwn index dad781808..3ad1d9541 100644 --- a/doc/plugins/contrib/album/discussion.mdwn +++ b/doc/plugins/contrib/album/discussion.mdwn @@ -476,3 +476,42 @@ My wishlist for the plugin would include: ---- +I've tracked down the "always showing the 96x96 thumbnails" bug! + +The problem is in the pagetemplate function, which calls "thumbnail" to determine the name of the thumbnail image to use. As you know, the "img" method of generating thumbnails includes the size of the thumbnail as part of its name (to ensure that resizing thumbnails will create a new file of the correct size). The problem is... that in the pagetemplate function, the thumbnailsize is NOT passed in to the call to "thumbnail", so it always returns the default size, 96x96. Hence nothing that anyone can do will change the thumbnails to anything else. Oh, the different-sized thumbnail images ARE created, but they're never linked to. + +Here's a context-diff of my fix: + +
+*** /home/kat/files/repos/ikiwiki_smcv/IkiWiki/Plugin/album.pm	2013-12-18 14:50:06.861623226 +1100
+--- album.pm	2013-12-18 15:51:09.393582879 +1100
+***************
+*** 484,489 ****
+--- 484,490 ----
+  		my $viewer = $params{page};
+  		my $album = $pagestate{$viewer}{album}{album};
+  		my $image = $pagestate{$viewer}{album}{image};
++                 my $thumbnailsize = $pagestate{$album}{album}{thumbnailsize};
+  
+  		return unless defined $album;
+  		return unless defined $image;
+***************
+*** 495,501 ****
+  
+  		if ($template->query(name => 'thumbnail')) {
+  			$template->param(thumbnail =>
+! 				thumbnail($viewer, $params{destpage}));
+  		}
+  		if (IkiWiki::isinlinableimage($image)
+  			&& ($template->query(name => 'imagewidth') ||
+--- 496,502 ----
+  
+  		if ($template->query(name => 'thumbnail')) {
+  			$template->param(thumbnail =>
+! 				thumbnail($viewer, $params{destpage}, $thumbnailsize));
+  		}
+  		if (IkiWiki::isinlinableimage($image)
+  			&& ($template->query(name => 'imagewidth') ||
+
+ +-- [[KathrynAndersen]]