wikimedia/mediawiki-extensions-SpamBlacklist
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/SpamBlacklist synced 2024-11-24 07:04:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

* (bug 1505) Limit spam blacklist checks to new URLs to reduce disruption of existing pages being legitimately edited by legitimate people which happen to already have some spam on them.

Browse source 
Steals the load-existing-links function out of ConfirmEdit.
This commit is contained in:
Brion Vibber 2008-05-13 23:31:33 +00:00
parent 1dc3f3e263
commit 45a4c9f03b
1 changed files with 27 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
SpamBlacklist_body.php
View file

@ -222,7 +222,15 @@ class SpamBlacklist {
$text = $wgParser->preSaveTransform( $text, $title, $wgUser, $options ); $text = $wgParser->preSaveTransform( $text, $title, $wgUser, $options );
$out = $wgParser->parse( $text, $title, $options ); $out = $wgParser->parse( $text, $title, $options );
} }
$links = implode( "\n", array_keys( $out->getExternalLinks() ) ); $newLinks = array_keys( $out->getExternalLinks() );
$oldLinks = $this->getCurrentLinks( $title );
$addedLinks = array_diff( $newLinks, $oldLinks );
wfDebugLog( 'SpamBlacklist', "Old URLs: " . implode( ', ', $oldLinks ) );
wfDebugLog( 'SpamBlacklist', "New URLs: " . implode( ', ', $newLinks ) );
wfDebugLog( 'SpamBlacklist', "Added URLs: " . implode( ', ', $addedLinks ) );
$links = implode( "\n", $addedLinks );
# Strip whitelisted URLs from the match # Strip whitelisted URLs from the match
if( is_array( $whitelists ) ) { if( is_array( $whitelists ) ) {
@ -265,6 +273,24 @@ class SpamBlacklist {
wfProfileOut( $fname ); wfProfileOut( $fname );
return $retVal; return $retVal;
} }
/**
* Look up the links currently in the article, so we can
* ignore them on a second run.
*
* WARNING: I can add more *of the same link* with no problem here.
*/
function getCurrentLinks( $title ) {
$dbr =& wfGetDB( DB_SLAVE );
$id = $title->getArticleId(); // should be zero queries
$res = $dbr->select( 'externallinks', array( 'el_to' ),
array( 'el_from' => $id ), __METHOD__ );
$links = array();
while ( $row = $dbr->fetchObject( $res ) ) {
$links[] = $row->el_to;
}
return $links;
}
/** /**
* Fetch an article from this or another local MediaWiki database. * Fetch an article from this or another local MediaWiki database.