Merge branch 'master' into revert

[git.ikiwiki.info.git] / IkiWiki / Receive.pm

diff --git a/IkiWiki/Receive.pm b/IkiWiki/Receive.pm
index ae1bd8bef44c8b6918f2d2ced55c2074d8bbca13..88fb4972571e7013480f943483a73a11e5b9a93d 100644 (file)
--- a/IkiWiki/Receive.pm
+++ b/IkiWiki/Receive.pm
@@ -48,16 +48,15 @@ EOF
 
 sub test () {
        exit 0 if trusted();
 
 sub test () {
        exit 0 if trusted();

-       
+

        IkiWiki::lockwiki();
        IkiWiki::loadindex();
        IkiWiki::lockwiki();
        IkiWiki::loadindex();

-       
+

        # Dummy up a cgi environment to use when calling check_canedit
        # and friends.
        eval q{use CGI};
        error($@) if $@;
        my $cgi=CGI->new;
        # Dummy up a cgi environment to use when calling check_canedit
        # and friends.
        eval q{use CGI};
        error($@) if $@;
        my $cgi=CGI->new;

-       $ENV{REMOTE_ADDR}='unknown' unless exists $ENV{REMOTE_ADDR};

 
        # And dummy up a session object.
        require IkiWiki::CGI;
 
        # And dummy up a session object.
        require IkiWiki::CGI;


@@ -73,10 +72,22 @@ sub test () {
                        regdate => time,
                }) || error("failed adding user");
        }
                        regdate => time,
                }) || error("failed adding user");
        }

-       
-       my %newfiles;

 
 

-       foreach my $change (IkiWiki::rcs_receive()) {
+       test_changes(cgi => $cgi,
+               session => $session,
+               changes => [IkiWiki::rcs_receive()]
+       );
+       exit 0;
+}
+
+sub test_changes {
+       my %params = @_;
+       my $cgi = $params{cgi};
+       my $session = $params{session};
+       my @changes = @{$params{changes}};
+
+       my %newfiles;
+       foreach my $change (@changes) {

                # This untaint is safe because we check file_pruned and
                # wiki_file_regexp.
                my ($file)=$change->{file}=~/$config{wiki_file_regexp}/;
                # This untaint is safe because we check file_pruned and
                # wiki_file_regexp.
                my ($file)=$change->{file}=~/$config{wiki_file_regexp}/;


@@ -88,7 +99,7 @@ sub test () {
 
                my $type=pagetype($file);
                my $page=pagename($file) if defined $type;
 
                my $type=pagetype($file);
                my $page=pagename($file) if defined $type;

-               
+

                if ($change->{action} eq 'add') {
                        $newfiles{$file}=1;
                }
                if ($change->{action} eq 'add') {
                        $newfiles{$file}=1;
                }


@@ -96,21 +107,20 @@ sub test () {
                if ($change->{action} eq 'change' ||
                    $change->{action} eq 'add') {
                        if (defined $page) {
                if ($change->{action} eq 'change' ||
                    $change->{action} eq 'add') {
                        if (defined $page) {

-                               if (IkiWiki->can("check_canedit")) {
-                                       IkiWiki::check_canedit($page, $cgi, $session);
-                                       next;
-                               }
+                               IkiWiki::check_canedit($page, $cgi, $session);
+                               next;

                        }
                        else {
                                if (IkiWiki::Plugin::attachment->can("check_canattach")) {
                                        IkiWiki::Plugin::attachment::check_canattach($session, $file, $change->{path});
                        }
                        else {
                                if (IkiWiki::Plugin::attachment->can("check_canattach")) {
                                        IkiWiki::Plugin::attachment::check_canattach($session, $file, $change->{path});

+                                       IkiWiki::check_canedit($file, $cgi, $session);

                                        next;
                                }
                        }
                }
                elsif ($change->{action} eq 'remove') {
                        # check_canremove tests to see if the file is present
                                        next;
                                }
                        }
                }
                elsif ($change->{action} eq 'remove') {
                        # check_canremove tests to see if the file is present

-                       # on disk. This will fail is a single commit adds a
+                       # on disk. This will fail when a single commit adds a

                        # file and then removes it again. Avoid the problem
                        # by not testing the removal in such pairs of changes.
                        # (The add is still tested, just to make sure that
                        # file and then removes it again. Avoid the problem
                        # by not testing the removal in such pairs of changes.
                        # (The add is still tested, just to make sure that


@@ -120,17 +130,16 @@ sub test () {
 
                        if (IkiWiki::Plugin::remove->can("check_canremove")) {
                                IkiWiki::Plugin::remove::check_canremove(defined $page ? $page : $file, $cgi, $session);
 
                        if (IkiWiki::Plugin::remove->can("check_canremove")) {
                                IkiWiki::Plugin::remove::check_canremove(defined $page ? $page : $file, $cgi, $session);

+                               IkiWiki::check_canedit(defined $page ? $page : $file, $cgi, $session);

                                next;
                        }
                }
                else {
                        error "unknown action ".$change->{action};
                }
                                next;
                        }
                }
                else {
                        error "unknown action ".$change->{action};
                }

-               
+

                error sprintf(gettext("you are not allowed to change %s"), $file);
        }
                error sprintf(gettext("you are not allowed to change %s"), $file);
        }

-
-       exit 0;

 }
 
 1
 }
 
 1