call check_canedit

[ikiwiki.git] / IkiWiki / Plugin / attachment.pm

diff --git a/IkiWiki/Plugin/attachment.pm b/IkiWiki/Plugin/attachment.pm
index 7af399fc068125475ae8563216d3105be0fe289f..8babe2be287067865f6253fb39fab1be590f4566 100644 (file)
--- a/IkiWiki/Plugin/attachment.pm
+++ b/IkiWiki/Plugin/attachment.pm
@@ -6,9 +6,9 @@ use strict;
 use IkiWiki 2.00;
 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,23 +48,45 @@ sub formbuilder (@) { #{{{
                if (IkiWiki::file_pruned($filename, $config{srcdir})) {
                        error(gettext("bad attachment filename"));
                }
-
-               # Check that the attachment matches the configured
-               # pagespec.
-               my $result=pagespec_match($filename, $config{valid_attachments},
-                       tempfile => $tempfile);
-               if (! $result) {
-                       error(gettext("attachment rejected")." ($result)");
-               }
                
-               my $fh=$q->upload('attachment');
-               if (! defined $fh || ! ref $fh) {
-                       error("failed to get filehandle");
+               # XXX Put the attachment in a subdir corresponding to the
+               # page being edited.
+               # The editpage code has already checked that
+               # $form->field('page') is valid.
+               $filename="XXX/$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},
+                               file => $tempfile);
+                       if (! $result) {
+                               error(gettext("attachment rejected")." ($result)");
+                       }
                }
-               binmode($fh);
-               while (<$fh>) {
-                       print STDERR $_."\n";
+
+               # Also check that the user is allowed to edit it by other
+               # policies.
+               IkiWiki::check_canedit($filename, $q, $params{session}, 1);
+
+               # Move the attachment into place.
+               # Try to use a fast rename; fall back to copying.
+               prep_writefile($filename, $config{srcdir});
+               unlink($config{srcdir}."/".$filename);
+               if (! rename($tempfile, $config{srcdir}."/".$filename)) {
+                       my $fh=$q->upload('attachment');
+                       if (! defined $fh || ! ref $fh) {
+                               error("failed to get filehandle");
+                       }
+                       binmode($fh);
+                       writefile($filename, $config{srcdir}, undef, 1, sub {
+                               IkiWiki::fast_file_copy($tempfile, $filename, $fh, @_);
+                       });
                }
+
+               # TODO add to vcs
+               
+               # TODO trigger a wiki build if there's no vcs
        }
 } # }}}
 
@@ -75,20 +97,20 @@ sub parsesize { #{{{
        no warnings;
        my $base=$size+0; # force to number
        use warnings;
-       my $exponent=1;
+       my $multiple=1;
        if ($size=~/kb?$/i) {
-               $exponent=10;
+               $multiple=2**10;
        }
        elsif ($size=~/mb?$/i) {
-               $exponent=20;
+               $multiple=2**20;
        }
        elsif ($size=~/gb?$/i) {
-               $exponent=30;
+               $multiple=2**30;
        }
        elsif ($size=~/tb?$/i) {
-               $exponent=40;
+               $multiple=2**40;
        }
-       return $base * 2**$exponent;
+       return $base * $multiple;
 } #}}}
 
 sub match_maxsize ($$;@) { #{{{
@@ -99,15 +121,15 @@ sub match_maxsize ($$;@) { #{{{
        }
 
        my %params=@_;
-       if (! exists $params{tempfile}) {
-               return IkiWiki::FailReason->new("no tempfile specified");
+       if (! exists $params{file}) {
+               return IkiWiki::FailReason->new("no file 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");
        }
 } #}}}
 
@@ -119,15 +141,26 @@ sub match_minsize ($$;@) { #{{{
        }
 
        my %params=@_;
-       if (! exists $params{tempfile}) {
-               return IkiWiki::FailReason->new("no tempfile specified");
+       if (! exists $params{file}) {
+               return IkiWiki::FailReason->new("no file 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 (defined 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");
        }
 } #}}}