Don't make a PNG thumbnail if the PDF creation failed.
authorSteinar H. Gunderson <sesse@samfundet.no>
Tue, 9 Aug 2005 14:33:25 +0000 (14:33 +0000)
committerSteinar H. Gunderson <sesse@samfundet.no>
Tue, 9 Aug 2005 14:33:25 +0000 (14:33 +0000)
createpdf.pl

index f33bbc5..0423330 100755 (executable)
@@ -153,24 +153,26 @@ EOF
        exit;
 }
 
-# Make a thumbnail from the finished PDF, for later reference. (Output to
-# stdout is so we make sure we get only the first page; it's the simplest
-# hack I can find offhand. :-) )
-system("convert -resize 192x192 output/$pdf_filename png:- > output/$pdf_filename.png");
-open DESC, ">output/$pdf_filename.desc";
+my $size = -s "output/$pdf_filename";
 
-if ($url =~ /^http/i) {
-       $url =~ tr/\n//d;
-       print DESC "$url\n";
-} else {
-       $filename =~ tr/\n//d;
-       print DESC "$filename\n";
-}
+if (defined($size) && $size > 0) {
+       # Make a thumbnail from the finished PDF, for later reference. (Output to
+       # stdout is so we make sure we get only the first page; it's the simplest
+       # hack I can find offhand. :-) )
+       system("convert -resize 192x192 output/$pdf_filename png:- > output/$pdf_filename.png");
+       open DESC, ">output/$pdf_filename.desc";
+
+       if ($url =~ /^http/i) {
+               $url =~ tr/\n//d;
+               print DESC "$url\n";
+       } else {
+               $filename =~ tr/\n//d;
+               print DESC "$filename\n";
+       }
 
-close DESC;
+       close DESC;
+}
        
-my $size = -s "output/$pdf_filename";
-
 (my $sanitized_outname = $outname) =~ tr/a-zA-Z0-9. -/_/c;
 
 print "Content-type: application/pdf\n";