aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoey Hess <joey@kitenet.net>2010-06-08 21:13:37 -0400
committerJoey Hess <joey@kitenet.net>2010-06-08 21:13:46 -0400
commit0ccf21daafa76fd182df5e6bc156f22873fe062d (patch)
treeaca6709dc5f1f4f3c6aed8ab9f8bf02001c6948f
parenteff7fac985b41d038b071ab68cfe8bc0f428441b (diff)
downloadikiwiki-0ccf21daafa76fd182df5e6bc156f22873fe062d.tar
ikiwiki-0ccf21daafa76fd182df5e6bc156f22873fe062d.tar.gz
img: Fill in missing height or width when scaling image.
-rw-r--r--IkiWiki/Plugin/img.pm6
-rw-r--r--debian/changelog1
-rw-r--r--doc/bugs/img_plugin_and_missing_heigth_value.mdwn2
3 files changed, 6 insertions, 3 deletions
diff --git a/IkiWiki/Plugin/img.pm b/IkiWiki/Plugin/img.pm
index f06121578..63dfa9019 100644
--- a/IkiWiki/Plugin/img.pm
+++ b/IkiWiki/Plugin/img.pm
@@ -115,9 +115,6 @@ sub preprocess (@) {
$im = Image::Magick->new;
$r = $im->Read($outfile);
error sprintf(gettext("failed to read %s: %s"), $outfile, $r) if $r;
-
- $dwidth = $im->Get("width");
- $dheight = $im->Get("height");
}
else {
($dwidth, $dheight)=($w, $h);
@@ -133,6 +130,9 @@ sub preprocess (@) {
$imglink = $file;
}
}
+
+ $dwidth = $im->Get("width") unless defined $dwidth;
+ $dheight = $im->Get("height") unless defined $dheight;
}
}
else {
diff --git a/debian/changelog b/debian/changelog
index 0a7d17eb7..9a10332ea 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,7 @@ ikiwiki (3.20100518.3) UNRELEASED; urgency=low
* creation_day() etc use local time, not gmtime. To match calendars, which
use local time.
+ * img: Fill in missing height or width when scaling image.
-- Joey Hess <joeyh@debian.org> Mon, 31 May 2010 20:44:17 -0400
diff --git a/doc/bugs/img_plugin_and_missing_heigth_value.mdwn b/doc/bugs/img_plugin_and_missing_heigth_value.mdwn
index a82e2a7d7..bcb17c330 100644
--- a/doc/bugs/img_plugin_and_missing_heigth_value.mdwn
+++ b/doc/bugs/img_plugin_and_missing_heigth_value.mdwn
@@ -1,3 +1,5 @@
When I set up my picture page with `\[[!img defaults size=300x]]` then the html validator complains that the value for height is missing and the IE browsers won't show the pictures up at all; no problems with ff tho. If I set up my picture page with `\[[!img defaults size=300x300]]` then all the images are funny stretched. What am I doing wrong?
> This is a bug. --[[Joey]]
+
+> And .. [[fixed|done]] --[[Joey]]