don't fail if %config is not set
[ikiwiki.git] / doc / plugins / filecheck / discussion.mdwn
1 First, thanks again for making this plugin.
2
3 I don't know if this is a problem for [[plugins/attachment]], but there seems to
4 be no way to detect text/plain using File::MimeInfo::Magic::magic().
5 There is a heuristic ::default that decides between text/plain and application/octet-stream.  
6
7 Would you be receptive to a patch that e.g. called File::MimeInfo::Magic::default()
8 if ::magic() returns undef?  --[[DavidBremner]]
9
10 > Yes, that looks to be ok. --[[Joey]]
11
12 >> OK, here is such a patch. One modification of previous behaviour is that 
13 >> that if default returns undef, this is returned. As far as I understand
14 >> the code/doc for File::MimeInfo, under is used only as an error return
15 >> for ::default
16 <pre>
17 From 84f40d7fa822c6cc09506cca2dbf38a8b19f8bb7 Mon Sep 17 00:00:00 2001
18 From: David Bremner <bremner@unb.ca>
19 Date: Fri, 29 Aug 2008 20:46:20 -0300
20 Subject: [PATCH] call File::MimeInfo::default if ::magic() returns undef
21
22 ---
23  IkiWiki/Plugin/filecheck.pm |    2 +-
24  1 files changed, 1 insertions(+), 1 deletions(-)
25
26 diff --git a/IkiWiki/Plugin/filecheck.pm b/IkiWiki/Plugin/filecheck.pm
27 index 9c7a8db..a94c02b 100644
28 --- a/IkiWiki/Plugin/filecheck.pm
29 +++ b/IkiWiki/Plugin/filecheck.pm
30 @@ -127,7 +127,7 @@ sub match_mimetype ($$;@) { #{{{
31         }
32         my $mimetype=File::MimeInfo::Magic::magic($file);
33         if (! defined $mimetype) {
34 -               $mimetype="unknown";
35 +               $mimetype=File::MimeInfo::Magic::default($file);
36         }
37
38         my $regexp=IkiWiki::glob2re($wanted);
39 --
40 1.5.6.3
41 </pre>