img: Fix adding of dependency from page to the image.
authorJoey Hess <joey@gnu.kitenet.net>
Mon, 27 Jul 2009 20:22:26 +0000 (22:22 +0200)
committerJoey Hess <joey@gnu.kitenet.net>
Mon, 27 Jul 2009 20:22:26 +0000 (22:22 +0200)
This was impressively broken. add_depends was being called with params
backwards, and on parameter was set to the name of the generated
file, which isn't in the source.

Now updates to images will update the page that contains them, thus
updating them. This is unncessary for fullsize images, so skipped.

IkiWiki/Plugin/img.pm
debian/changelog

index 44d67bd8375057e77d5853ee93cdd7943108ed14..68b001671653000b4907ababf1d63782d834381c 100644 (file)
@@ -44,6 +44,7 @@ sub preprocess (@) {
        }
 
        add_link($params{page}, $image);
+
        # optimisation: detect scan mode, and avoid generating the image
        if (! defined wantarray) {
                return;
@@ -65,6 +66,8 @@ sub preprocess (@) {
        my $r;
 
        if ($params{size} ne 'full') {
+               add_depends($params{page}, $image);
+
                my ($w, $h) = ($params{size} =~ /^(\d*)x(\d*)$/);
                error sprintf(gettext('wrong size format "%s" (should be WxH)'), $params{size})
                        unless (defined $w && defined $h &&
@@ -102,8 +105,6 @@ sub preprocess (@) {
                $imglink = $file;
        }
 
-       add_depends($imglink, $params{page});
-
        my ($fileurl, $imgurl);
        if (! $params{preview}) {
                $fileurl=urlto($file, $params{destpage});
index 565f19c7c73bd6ce26e1dbeca16ab15bdf39cd0e..aa14939600b7bd0a3674b63197ad72ad8685467b 100644 (file)
@@ -8,6 +8,7 @@ ikiwiki (3.15) UNRELEASED; urgency=low
   * Add further build machinery to generate translated underlays from
     the po file, for use by wikis whose primary language is not English.
   * Add Danish basewiki translation by Jonas Smedegaard.
+  * img: Fix adding of dependency from page to the image.
 
  -- Joey Hess <joeyh@debian.org>  Tue, 02 Jun 2009 17:03:41 -0400