summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--IkiWiki/Plugin/attachment.pm33
-rw-r--r--doc/plugins/attachment.mdwn12
2 files changed, 33 insertions, 12 deletions
diff --git a/IkiWiki/Plugin/attachment.pm b/IkiWiki/Plugin/attachment.pm
index 48a1c58b4..186f3ea21 100644
--- a/IkiWiki/Plugin/attachment.pm
+++ b/IkiWiki/Plugin/attachment.pm
@@ -8,7 +8,7 @@ use CGI;
$CGI::DISABLE_UPLOADS=0;
# TODO move to admin prefs
-$config{valid_attachments}="(*.mp3 and maxsize(15mb)) or (* and maxsize(50kb))";
+$config{valid_attachments}="(*.mp3 and maxsize(15mb)) or (!ispage() and maxsize(50kb))";
sub import { #{{{
hook(type => "formbuilder_setup", id => "attachment", call => \&formbuilder_setup);
@@ -48,12 +48,12 @@ sub formbuilder (@) { #{{{
if (IkiWiki::file_pruned($filename, $config{srcdir})) {
error(gettext("bad attachment filename"));
}
-
+
# Use a pagespec to test that the attachment is valid.
if (exists $config{valid_attachments} &&
length $config{valid_attachments}) {
my $result=pagespec_match($filename, $config{valid_attachments},
- tempfile => $tempfile);
+ file => $tempfile);
if (! $result) {
error(gettext("attachment rejected")." ($result)");
}
@@ -101,15 +101,15 @@ sub match_maxsize ($$;@) { #{{{
}
my %params=@_;
- if (! exists $params{tempfile}) {
+ if (! exists $params{file}) {
return IkiWiki::FailReason->new("no tempfile specified");
}
- if (-s $params{tempfile} > $maxsize) {
- return IkiWiki::FailReason->new("attachment too large");
+ if (-s $params{file} > $maxsize) {
+ return IkiWiki::FailReason->new("file too large");
}
else {
- return IkiWiki::SuccessReason->new("attachment size ok");
+ return IkiWiki::SuccessReason->new("file not too large");
}
} #}}}
@@ -121,15 +121,26 @@ sub match_minsize ($$;@) { #{{{
}
my %params=@_;
- if (! exists $params{tempfile}) {
+ if (! exists $params{file}) {
return IkiWiki::FailReason->new("no tempfile specified");
}
- if (-s $params{tempfile} < $minsize) {
- return IkiWiki::FailReason->new("attachment too small");
+ if (-s $params{file} < $minsize) {
+ return IkiWiki::FailReason->new("file too small");
+ }
+ else {
+ return IkiWiki::SuccessReason->new("file not too small");
+ }
+} #}}}
+
+sub match_ispage ($$;@) { #{{{
+ my $filename=shift;
+
+ if (IkiWiki::pagetype($filename)) {
+ return IkiWiki::SuccessReason->new("file is a wiki page");
}
else {
- return IkiWiki::SuccessReason->new("attachment size ok");
+ return IkiWiki::FailReason->new("file is not a wiki page");
}
} #}}}
diff --git a/doc/plugins/attachment.mdwn b/doc/plugins/attachment.mdwn
index a03865987..019d1c9e4 100644
--- a/doc/plugins/attachment.mdwn
+++ b/doc/plugins/attachment.mdwn
@@ -28,7 +28,7 @@ expanded with additional tests.
For example, to limit arbitrary files to 50 kilobtes, but allow
larger mp3 files to be uploaded, a test like this could be used:
- (*.mp3 and maxsize(15mb)) or (* and maxsize(50kb))
+ (*.mp3 and maxsize(15mb)) or (!ispage() and maxsize(50kb))
The following additional tests are available:
@@ -41,3 +41,13 @@ The following additional tests are available:
* minsize(size)
Tests whether the attachment is no smaller than the specified size.
+
+* ispage()
+
+ Tests whether the attachment will be treated by ikiwiki as a wiki page.
+ (Ie, if it has an extension of ".mdwn", or of any other enabled page
+ format).
+
+ So, if you don't want to allow wiki pages to be uploaded as attachments,
+ use `!ispage()` ; if you only want to allow wiki pages to be uploaded
+ as attachments, use `ispage()`.