jenkins-bot has submitted this change and it was merged.
Change subject: Remove some unused code
......................................................................
Remove some unused code
Change-Id: I6c27a5c962fe79afcbd34f8f101d44f1382ff854
---
M SpecialSmiteSpam.php
M autoload.php
D includes/SmiteSpamDeleter.php
3 files changed, 0 insertions(+), 61 deletions(-)
Approvals:
Polybuildr: Looks good to me, approved
jenkins-bot: Verified
diff --git a/SpecialSmiteSpam.php b/SpecialSmiteSpam.php
index 5939225..3ffe1f7 100644
--- a/SpecialSmiteSpam.php
+++ b/SpecialSmiteSpam.php
@@ -19,30 +19,6 @@
$numPages = SiteStats::pages();
- $request = $this->getRequest();
-
- if ( $request->wasPosted() ) {
- $ssDeleter = new SmiteSpamDeleter();
- $pageIDs = $request->getArray( 'delete' );
- if ( $pageIDs ) {
- $messages = $ssDeleter->deletePages( $pageIDs );
- if ( isset( $messages['error'] ) ) {
- $out->addHTML( '<div class="errorbox">'
);
- foreach ( $messages['error'] as
$message ) {
- $out->addHTML( '<p>' . $message
. '</p>' );
- }
- $out->addHTML( '</div>' );
- }
- if ( isset( $messages['success'] ) ) {
- $out->addHTML( '<div
class="successbox">' );
- foreach ( $messages['success'] as
$message ) {
- $out->addHTML( '<p>' . $message
. '</p>' );
- }
- $out->addHTML( '</div>' );
- }
- }
- }
-
$out->addHTML(
Linker::link(
SpecialPage::getTitleFor(
'SmiteSpamTrustedUsers' ),
diff --git a/autoload.php b/autoload.php
index 395f6a3..70ae6e5 100644
--- a/autoload.php
+++ b/autoload.php
@@ -7,7 +7,6 @@
'SmiteSpamAnalyzer' => __DIR__ . '/includes/SmiteSpamAnalyzer.php',
'SmiteSpamApiQuery' => __DIR__ . '/api/SmiteSpamApiQuery.php',
'SmiteSpamApiTrustUser' => __DIR__ . '/api/SmiteSpamApiTrustUser.php',
- 'SmiteSpamDeleter' => __DIR__ . '/includes/SmiteSpamDeleter.php',
'SmiteSpamExternalLinksChecker' => __DIR__ .
'/includes/checkers/SmiteSpamExternalLinksChecker.php',
'SmiteSpamHooks' => __DIR__ . '/SmiteSpam.hooks.php',
'SmiteSpamRepeatedExternalLinksChecker' => __DIR__ .
'/includes/checkers/SmiteSpamRepeatedExternalLinksChecker.php',
diff --git a/includes/SmiteSpamDeleter.php b/includes/SmiteSpamDeleter.php
deleted file mode 100644
index 3c5a54f..0000000
--- a/includes/SmiteSpamDeleter.php
+++ /dev/null
@@ -1,36 +0,0 @@
-<?php
-/**
- * The class performing the actual deletion of selected pages.
- */
-class SmiteSpamDeleter {
- /**
- * Deletes a list of pages
- * @param array $pageIDs
- * @return array A list of messages to be displayed.
- */
- public function deletePages( $pageIDs ) {
- // @todo change to use Jobs?
- $messages = array();
- $successMessage = wfMessage(
'smitespam-delete-page-success-msg' )->text();
- $failureMessage = wfMessage(
'smitespam-delete-page-failure-msg' )->text();
- foreach ( $pageIDs as $pageID ) {
- if ( ctype_digit( $pageID ) || is_int( $pageID ) ) {
- $article = Article::newFromID( $pageID );
- if ( $article ) {
- $ok = $article->doDeleteArticle(
wfMessage( 'smitespam-deleted-reason' ) );
- $titleText =
Sanitizer::escapeHtmlAllowEntities( $article->getTitle()->getText() );
- if ( $ok ) {
- $messages['success'][] =
"$successMessage '$titleText'.";
- }
- else {
- $messages['error'][] =
"$failureMessage '$titleText'.";
- }
- continue;
- }
- }
- $pageID = Sanitizer::escapeHtmlAllowEntities( $pageID );
- $messages['error'][] = "$failureMessage '$pageID'.";
- }
- return $messages;
- }
-}
--
To view, visit https://gerrit.wikimedia.org/r/236430
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6c27a5c962fe79afcbd34f8f101d44f1382ff854
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/SmiteSpam
Gerrit-Branch: master
Gerrit-Owner: Polybuildr <[email protected]>
Gerrit-Reviewer: Polybuildr <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
