mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/SpamBlacklist
synced 2024-12-01 02:06:14 +00:00
2745442aec
For Wikidata, this causes users to be unable to create items. (bug 59797)
In case of Special:NewItem, $context->getWikiPage() does
$context->getTitle() to get the special page and then WikiPage::factory
with the special page $title. special page does not make a valid
WikiPage and throws an exception.
Previously, $title = special page and was used in getParserOutput.
$title is invalid there but Wikibase never used that variable.
Would be good to make sure $title is actually related to the content.
This reverts commit 508a3706d6
.
Change-Id: Ib24515be6e76d3f29e2f9048fbb81e5a25b5857a
223 lines
5.8 KiB
PHP
223 lines
5.8 KiB
PHP
<?php
|
|
|
|
/**
|
|
* Hooks for the spam blacklist extension
|
|
*/
|
|
class SpamBlacklistHooks {
|
|
/**
|
|
* Hook function for EditFilterMergedContent
|
|
*
|
|
* @param IContextSource $context
|
|
* @param Content $content
|
|
* @param Status $status
|
|
* @param string $summary
|
|
* @param User $user
|
|
* @param bool $minoredit
|
|
*
|
|
* @return bool
|
|
*/
|
|
static function filterMergedContent( IContextSource $context, Content $content, Status $status, $summary, User $user, $minoredit ) {
|
|
$title = $context->getTitle();
|
|
|
|
if ( isset( $title->spamBlackListFiltered ) && $title->spamBlackListFiltered ) {
|
|
// already filtered
|
|
return true;
|
|
}
|
|
|
|
// get the link from the not-yet-saved page content.
|
|
$pout = $content->getParserOutput( $title );
|
|
$links = array_keys( $pout->getExternalLinks() );
|
|
|
|
// HACK: treat the edit summary as a link
|
|
if ( $summary !== '' ) {
|
|
$links[] = $summary;
|
|
}
|
|
|
|
$spamObj = BaseBlacklist::getInstance( 'spam' );
|
|
$matches = $spamObj->filter( $links, $title );
|
|
|
|
if ( $matches !== false ) {
|
|
$status->fatal( 'spamprotectiontext' );
|
|
|
|
foreach ( $matches as $match ) {
|
|
$status->fatal( 'spamprotectionmatch', $match );
|
|
}
|
|
}
|
|
|
|
// Always return true, EditPage will look at $status->isOk().
|
|
return true;
|
|
}
|
|
|
|
/**
|
|
* Hook function for APIEditBeforeSave.
|
|
* This allows blacklist matches to be reported directly in the result structure
|
|
* of the API call.
|
|
*
|
|
* @param $editPage EditPage
|
|
* @param $text string
|
|
* @param $resultArr array
|
|
* @return bool
|
|
*/
|
|
static function filterAPIEditBeforeSave( $editPage, $text, &$resultArr ) {
|
|
$title = $editPage->mArticle->getTitle();
|
|
|
|
// get the links from the not-yet-saved page content.
|
|
$content = ContentHandler::makeContent(
|
|
$text,
|
|
$editPage->getTitle(),
|
|
$editPage->contentModel,
|
|
$editPage->contentFormat
|
|
);
|
|
$editInfo = $editPage->mArticle->prepareContentForEdit( $content, null, null, $editPage->contentFormat );
|
|
$pout = $editInfo->output;
|
|
$links = array_keys( $pout->getExternalLinks() );
|
|
|
|
// HACK: treat the edit summary as a link
|
|
$summary = $editPage->summary;
|
|
if ( $summary !== '' ) {
|
|
$links[] = $summary;
|
|
}
|
|
|
|
$spamObj = BaseBlacklist::getInstance( 'spam' );
|
|
$matches = $spamObj->filter( $links, $title );
|
|
|
|
if ( $matches !== false ) {
|
|
$resultArr['spamblacklist'] = implode( '|', $matches );
|
|
}
|
|
|
|
// mark the title, so filterMergedContent can skip it.
|
|
$title->spamBlackListFiltered = true;
|
|
|
|
// return convention for hooks is the inverse of $wgFilterCallback
|
|
return ( $matches === false );
|
|
}
|
|
|
|
/**
|
|
* Verify that the user can send emails
|
|
*
|
|
* @param $user User
|
|
* @param $hookErr array
|
|
* @return bool
|
|
*/
|
|
public static function userCanSendEmail( &$user, &$hookErr ) {
|
|
/** @var $blacklist EmailBlacklist */
|
|
$blacklist = BaseBlacklist::getInstance( 'email' );
|
|
if ( $blacklist->checkUser( $user ) ) {
|
|
return true;
|
|
}
|
|
|
|
$hookErr = array( 'spam-blacklisted-email', 'spam-blacklisted-email-text', null );
|
|
|
|
return false;
|
|
}
|
|
|
|
/**
|
|
* Processes new accounts for valid email addresses
|
|
*
|
|
* @param $user User
|
|
* @param $abortError
|
|
* @return bool
|
|
*/
|
|
public static function abortNewAccount( $user, &$abortError ) {
|
|
/** @var $blacklist EmailBlacklist */
|
|
$blacklist = BaseBlacklist::getInstance( 'email' );
|
|
if ( $blacklist->checkUser( $user ) ) {
|
|
return true;
|
|
}
|
|
|
|
$abortError = wfMessage( 'spam-blacklisted-email-signup' )->escaped();
|
|
return false;
|
|
}
|
|
|
|
/**
|
|
* Hook function for EditFilter
|
|
* Confirm that a local blacklist page being saved is valid,
|
|
* and toss back a warning to the user if it isn't.
|
|
*
|
|
* @param $editPage EditPage
|
|
* @param $text string
|
|
* @param $section string
|
|
* @param $hookError string
|
|
* @return bool
|
|
*/
|
|
static function validate( $editPage, $text, $section, &$hookError ) {
|
|
$thisPageName = $editPage->mTitle->getPrefixedDBkey();
|
|
|
|
if( !BaseBlacklist::isLocalSource( $editPage->mTitle ) ) {
|
|
wfDebugLog( 'SpamBlacklist', "Spam blacklist validator: [[$thisPageName]] not a local blacklist\n" );
|
|
return true;
|
|
}
|
|
|
|
$type = BaseBlacklist::getTypeFromTitle( $editPage->mTitle );
|
|
if ( $type === false ) {
|
|
return true;
|
|
}
|
|
|
|
$lines = explode( "\n", $text );
|
|
|
|
$badLines = SpamRegexBatch::getBadLines( $lines, BaseBlacklist::getInstance( $type ) );
|
|
if( $badLines ) {
|
|
wfDebugLog( 'SpamBlacklist', "Spam blacklist validator: [[$thisPageName]] given invalid input lines: " .
|
|
implode( ', ', $badLines ) . "\n" );
|
|
|
|
$badList = "*<code>" .
|
|
implode( "</code>\n*<code>",
|
|
array_map( 'wfEscapeWikiText', $badLines ) ) .
|
|
"</code>\n";
|
|
$hookError =
|
|
"<div class='errorbox'>" .
|
|
wfMessage( 'spam-invalid-lines' )->numParams( $badLines )->text() . "<br />" .
|
|
$badList .
|
|
"</div>\n" .
|
|
"<br clear='all' />\n";
|
|
} else {
|
|
wfDebugLog( 'SpamBlacklist', "Spam blacklist validator: [[$thisPageName]] ok or empty blacklist\n" );
|
|
}
|
|
|
|
return true;
|
|
}
|
|
|
|
/**
|
|
* Hook function for PageContentSaveComplete
|
|
* Clear local spam blacklist caches on page save.
|
|
*
|
|
* @param Page $wikiPage
|
|
* @param User $user
|
|
* @param Content $content
|
|
* @param string $summary
|
|
* @param bool $isMinor
|
|
* @param bool $isWatch
|
|
* @param string $section
|
|
* @param int $flags
|
|
* @param int $revision
|
|
* @param Status $status
|
|
* @param int $baseRevId
|
|
*
|
|
* @return bool
|
|
*/
|
|
static function pageSaveContent(
|
|
Page $wikiPage,
|
|
User $user,
|
|
Content $content,
|
|
$summary,
|
|
$isMinor,
|
|
$isWatch,
|
|
$section,
|
|
$flags,
|
|
$revision,
|
|
Status $status,
|
|
$baseRevId
|
|
) {
|
|
if( !BaseBlacklist::isLocalSource( $wikiPage->getTitle() ) ) {
|
|
return true;
|
|
}
|
|
global $wgMemc, $wgDBname;
|
|
|
|
// This sucks because every Blacklist needs to be cleared
|
|
foreach ( BaseBlacklist::getBlacklistTypes() as $type => $class ) {
|
|
$wgMemc->delete( "$wgDBname:{$type}_blacklist_regexes" );
|
|
}
|
|
return true;
|
|
}
|
|
}
|