revert check_canedit nosubs thing

Abstraction violation. I now think the problem should be treated as a bug in httpauth.
saggim · Aug 30, 2010 · 2df0999 · 2df0999
1 parent 70db571
commit 2df0999
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/IkiWiki.pm b/IkiWiki.pm
@@ -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;
 			}

diff --git a/IkiWiki/Receive.pm b/IkiWiki/Receive.pm
@@ -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;
 			}
 		}