urosm
/
ikiwiki
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

revert check_canedit nosubs thing 

Abstraction violation. I now think the problem should be treated as a bug
in httpauth.
master
Joey Hess 2010-08-30 18:31:56 -04:00
parent 70db57134a
commit 2df0999e40
2 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
IkiWiki.pm
View File

@ -1455,12 +1455,11 @@ sub filter ($$$) {
return $content;
}
sub check_canedit ($$$;$$) {
sub check_canedit ($$$;$) {
my $page=shift;
my $q=shift;
my $session=shift;
my $nonfatal=shift;
my $nosubs=shift;
my $canedit;
run_hooks(canedit => sub {
@ -1471,7 +1470,6 @@ sub check_canedit ($$$;$$) {
$canedit=1;
}
elsif (ref $ret eq 'CODE') {
error(sprintf(gettext("you are not allowed to change %s"), $page)) if $nosubs && ! $nonfatal;
$ret->() unless $nonfatal;
$canedit=0;
}

6
IkiWiki/Receive.pm
View File

@ -95,13 +95,13 @@ sub test () {
if ($change->{action} eq 'change' ||
$change->{action} eq 'add') {
if (defined $page) {
IkiWiki::check_canedit($page, $cgi, $session, 0, 1);
IkiWiki::check_canedit($page, $cgi, $session);
next;
}
else {
if (IkiWiki::Plugin::attachment->can("check_canattach")) {
IkiWiki::Plugin::attachment::check_canattach($session, $file, $change->{path});
IkiWiki::check_canedit($file, $cgi, $session, 0, 1);
IkiWiki::check_canedit($file, $cgi, $session);
next;
}
}
@ -118,7 +118,7 @@ sub test () {
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, 0, 1);
IkiWiki::check_canedit(defined $page ? $page : $file, $cgi, $session);
next;
}
}