move wrapper building loop into Wrapper.pm

[git.ikiwiki.info.git] / IkiWiki / Plugin / blogspam.pm

diff --git a/IkiWiki/Plugin/blogspam.pm b/IkiWiki/Plugin/blogspam.pm
index d4b5b36d51e4417f497c234ced17f56482851091..8db3780e8888ee9c9188881e80ae26b6715436ff 100644 (file)
--- a/IkiWiki/Plugin/blogspam.pm
+++ b/IkiWiki/Plugin/blogspam.pm
@@ -9,6 +9,7 @@ my $defaulturl='http://test.blogspam.net:8888/';
 
 sub import {
        hook(type => "getsetup", id => "blogspam",  call => \&getsetup);
 
 sub import {
        hook(type => "getsetup", id => "blogspam",  call => \&getsetup);

+       hook(type => "checkconfig", id => "blogspam", call => \&checkconfig);

        hook(type => "checkcontent", id => "blogspam", call => \&checkcontent);
 }
 
        hook(type => "checkcontent", id => "blogspam", call => \&checkcontent);
 }
 


@@ -17,6 +18,7 @@ sub getsetup () {
                plugin => {
                        safe => 1,
                        rebuild => 0,
                plugin => {
                        safe => 1,
                        rebuild => 0,

+                       section => "auth",

                },
                blogspam_pagespec => {
                        type => 'pagespec',
                },
                blogspam_pagespec => {
                        type => 'pagespec',


@@ -43,17 +45,20 @@ sub getsetup () {
                },
 }
 
                },
 }
 

-sub checkcontent (@) {
-       my %params=@_;
-
+sub checkconfig () {
+       # This is done at checkconfig time because printing an error
+       # if the module is missing when a spam is posted would not
+       # let the admin know about the problem.

        eval q{
                use RPC::XML;
                use RPC::XML::Client;
        };
        eval q{
                use RPC::XML;
                use RPC::XML::Client;
        };

-       if ($@) {
-               warn($@);
-               return undef;
-       }
+       error $@ if $@;
+}
+
+sub checkcontent (@) {
+       my %params=@_;
+       my $session=$params{session};

        
        if (exists $config{blogspam_pagespec}) {
                return undef
        
        if (exists $config{blogspam_pagespec}) {
                return undef


@@ -62,11 +67,11 @@ sub checkcontent (@) {
        }
 
        my $url=$defaulturl;
        }
 
        my $url=$defaulturl;

-       $url = $params{blogspam_server} if exists $params{blogspam_server};
+       $url = $config{blogspam_server} if exists $config{blogspam_server};

        my $client = RPC::XML::Client->new($url);
 
        my $client = RPC::XML::Client->new($url);
 

-       my @options = split(",", $params{blogspam_options})
-               if exists $params{blogspam_options};
+       my @options = split(",", $config{blogspam_options})
+               if exists $config{blogspam_options};

 
        # Allow short comments and whitespace-only edits, unless the user
        # has overridden min-words themselves.
 
        # Allow short comments and whitespace-only edits, unless the user
        # has overridden min-words themselves.


@@ -83,8 +88,8 @@ sub checkcontent (@) {
        # and "buy".
        push @options, "exclude=stopwords";
 
        # and "buy".
        push @options, "exclude=stopwords";
 

-       my $res = $client->send_request('testComment', {
-               ip => $ENV{REMOTE_ADDR},
+       my %req=(
+               ip => $session->remote_addr(),

                comment => defined $params{diff} ? $params{diff} : $params{content},
                subject => defined $params{subject} ? $params{subject} : "",
                name => defined $params{author} ? $params{author} : "",
                comment => defined $params{diff} ? $params{diff} : $params{content},
                subject => defined $params{subject} ? $params{subject} : "",
                name => defined $params{author} ? $params{author} : "",


@@ -92,17 +97,20 @@ sub checkcontent (@) {
                options => join(",", @options),
                site => $config{url},
                version => "ikiwiki ".$IkiWiki::version,
                options => join(",", @options),
                site => $config{url},
                version => "ikiwiki ".$IkiWiki::version,

-       });
+       );
+       my $res = $client->send_request('testComment', \%req);

 
        if (! ref $res || ! defined $res->value) {
                debug("failed to get response from blogspam server ($url)");
                return undef;
        }
        elsif ($res->value =~ /^SPAM:(.*)/) {
 
        if (! ref $res || ! defined $res->value) {
                debug("failed to get response from blogspam server ($url)");
                return undef;
        }
        elsif ($res->value =~ /^SPAM:(.*)/) {

+               eval q{use Data::Dumper};
+               debug("blogspam server reports ".$res->value.": ".Dumper(\%req));

                return gettext("Sorry, but that looks like spam to <a href=\"http://blogspam.net/\">blogspam</a>: ").$1;
        }
        elsif ($res->value ne 'OK') {
                return gettext("Sorry, but that looks like spam to <a href=\"http://blogspam.net/\">blogspam</a>: ").$1;
        }
        elsif ($res->value ne 'OK') {

-               debug(gettext("blogspam server failure: ").$res->value);
+               debug("blogspam server failure: ".$res->value);

                return undef;
        }
        else {
                return undef;
        }
        else {