diff --git a/extension.json b/extension.json index 7a68f12ee..1f6dbea89 100644 --- a/extension.json +++ b/extension.json @@ -140,52 +140,8 @@ "AbuseFilterAliases": "AbuseFilter.alias.php" }, "AutoloadNamespaces": { - "MediaWiki\\Extension\\AbuseFilter\\Maintenance\\": "maintenance/", - "MediaWiki\\Extension\\AbuseFilter\\Api\\": "includes/Api/", - "MediaWiki\\Extension\\AbuseFilter\\VariableGenerator\\": "includes/VariableGenerator/", - "MediaWiki\\Extension\\AbuseFilter\\Hooks\\": "includes/Hooks/", - "MediaWiki\\Extension\\AbuseFilter\\LogFormatter\\": "includes/LogFormatter/", - "MediaWiki\\Extension\\AbuseFilter\\Parser\\": "includes/Parser/", - "MediaWiki\\Extension\\AbuseFilter\\Filter\\": "includes/Filter/", - "MediaWiki\\Extension\\AbuseFilter\\Pager\\": "includes/Pager/", - "MediaWiki\\Extension\\AbuseFilter\\Special\\": "includes/Special/", - "MediaWiki\\Extension\\AbuseFilter\\View\\": "includes/View/", - "MediaWiki\\Extension\\AbuseFilter\\Watcher\\": "includes/Watcher/", - "MediaWiki\\Extension\\AbuseFilter\\Consequences\\": "includes/Consequences/", - "MediaWiki\\Extension\\AbuseFilter\\ChangeTags\\": "includes/ChangeTags/", - "MediaWiki\\Extension\\AbuseFilter\\Variables\\": "includes/Variables/" - }, - "AutoloadClasses": { - "AbuseFilter": "includes/AbuseFilter.php", - "AbuseFilterHooks": "includes/AbuseFilterHooks.php", - "AbuseFilterPreAuthenticationProvider": "includes/AbuseFilterPreAuthenticationProvider.php", - "AbuseFilterChangesList": "includes/AbuseFilterChangesList.php", - "TableDiffFormatterFullContext": "includes/TableDiffFormatterFullContext.php", - "MediaWiki\\Extension\\AbuseFilter\\GlobalNameUtils": "includes/GlobalNameUtils.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterLookup": "includes/FilterLookup.php", - "MediaWiki\\Extension\\AbuseFilter\\KeywordsManager": "includes/KeywordsManager.php", - "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterPermissionManager": "includes/AbuseFilterPermissionManager.php", - "MediaWiki\\Extension\\AbuseFilter\\EchoNotifier": "includes/EchoNotifier.php", - "MediaWiki\\Extension\\AbuseFilter\\ThrottleFilterPresentationModel": "includes/ThrottleFilterPresentationModel.php", - "MediaWiki\\Extension\\AbuseFilter\\SpecsFormatter": "includes/SpecsFormatter.php", - "MediaWiki\\Extension\\AbuseFilter\\TextExtractor": "includes/TextExtractor.php", - "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterServices": "includes/AbuseFilterServices.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterProfiler": "includes/FilterProfiler.php", - "MediaWiki\\Extension\\AbuseFilter\\BlockAutopromoteStore": "includes/BlockAutopromoteStore.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterUser": "includes/FilterUser.php", - "MediaWiki\\Extension\\AbuseFilter\\CentralDBManager": "includes/CentralDBManager.php", - "MediaWiki\\Extension\\AbuseFilter\\CentralDBNotAvailableException": "includes/CentralDBNotAvailableException.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterValidator": "includes/FilterValidator.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterCompare": "includes/FilterCompare.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterImporter": "includes/FilterImporter.php", - "MediaWiki\\Extension\\AbuseFilter\\EditBoxBuilder": "includes/EditBoxBuilder.php", - "MediaWiki\\Extension\\AbuseFilter\\EditBoxBuilderFactory": "includes/EditBoxBuilderFactory.php", - "MediaWiki\\Extension\\AbuseFilter\\InvalidImportDataException": "includes/InvalidImportDataException.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterStore": "includes/FilterStore.php", - "MediaWiki\\Extension\\AbuseFilter\\AbuseLogger": "includes/AbuseLogger.php", - "MediaWiki\\Extension\\AbuseFilter\\AbuseLoggerFactory": "includes/AbuseLoggerFactory.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterRunner": "includes/FilterRunner.php", - "MediaWiki\\Extension\\AbuseFilter\\FilterRunnerFactory": "includes/FilterRunnerFactory.php" + "MediaWiki\\Extension\\AbuseFilter\\": "includes/", + "MediaWiki\\Extension\\AbuseFilter\\Maintenance\\": "maintenance/" }, "TestAutoloadClasses": { "AbuseFilterConsequencesTest": "tests/phpunit/AbuseFilterConsequencesTest.php", @@ -271,7 +227,7 @@ "localBasePath": "modules", "remoteExtPath": "AbuseFilter/modules" }, - "callback": "AbuseFilterHooks::onRegistration", + "callback": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onRegistration", "HookHandlers": { "ToolLinks": { "class": "MediaWiki\\Extension\\AbuseFilter\\Hooks\\Handlers\\ToolLinksHandler", @@ -294,24 +250,24 @@ } }, "Hooks": { - "EditFilterMergedContent": "AbuseFilterHooks::onEditFilterMergedContent", + "EditFilterMergedContent": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onEditFilterMergedContent", "GetAutoPromoteGroups": "AutoPromoteGroups", - "TitleMove": "AbuseFilterHooks::onTitleMove", - "ArticleDelete": "AbuseFilterHooks::onArticleDelete", - "RecentChange_save": "AbuseFilterHooks::onRecentChangeSave", + "TitleMove": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onTitleMove", + "ArticleDelete": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onArticleDelete", + "RecentChange_save": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onRecentChangeSave", "ListDefinedTags": "ChangeTags", "ChangeTagsListActive": "ChangeTags", "LoadExtensionSchemaUpdates": "SchemaChanges", "ContributionsToolLinks": "ToolLinks", "HistoryPageToolLinks": "ToolLinks", "UndeletePageToolLinks": "ToolLinks", - "UploadVerifyUpload": "AbuseFilterHooks::onUploadVerifyUpload", - "UploadStashFile": "AbuseFilterHooks::onUploadStashFile", - "PageSaveComplete": "AbuseFilterHooks::onPageSaveComplete", - "RenameUserSQL": "AbuseFilterHooks::onRenameUserSQL", - "UserMergeAccountFields": "AbuseFilterHooks::onUserMergeAccountFields", - "BeforeCreateEchoEvent": "AbuseFilterHooks::onBeforeCreateEchoEvent", - "ParserOutputStashForEdit": "AbuseFilterHooks::onParserOutputStashForEdit", + "UploadVerifyUpload": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onUploadVerifyUpload", + "UploadStashFile": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onUploadStashFile", + "PageSaveComplete": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onPageSaveComplete", + "RenameUserSQL": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onRenameUserSQL", + "UserMergeAccountFields": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onUserMergeAccountFields", + "BeforeCreateEchoEvent": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onBeforeCreateEchoEvent", + "ParserOutputStashForEdit": "MediaWiki\\Extension\\AbuseFilter\\AbuseFilterHooks::onParserOutputStashForEdit", "UnitTestsAfterDatabaseSetup": "Tests", "UnitTestsBeforeDatabaseTeardown": "Tests" }, diff --git a/includes/AbuseFilter.php b/includes/AbuseFilter.php index 43b1c9b64..4fb0c789d 100644 --- a/includes/AbuseFilter.php +++ b/includes/AbuseFilter.php @@ -1,8 +1,13 @@ msg( 'abusefilter-action-' . $filterAction )->text(); }, $throttledActions diff --git a/includes/View/AbuseFilterViewExamine.php b/includes/View/AbuseFilterViewExamine.php index 93666b79a..f1e34498e 100644 --- a/includes/View/AbuseFilterViewExamine.php +++ b/includes/View/AbuseFilterViewExamine.php @@ -2,11 +2,11 @@ namespace MediaWiki\Extension\AbuseFilter\View; -use AbuseFilter; -use AbuseFilterChangesList; use ChangesList; use HTMLForm; use IContextSource; +use MediaWiki\Extension\AbuseFilter\AbuseFilter; +use MediaWiki\Extension\AbuseFilter\AbuseFilterChangesList; use MediaWiki\Extension\AbuseFilter\AbuseFilterPermissionManager; use MediaWiki\Extension\AbuseFilter\EditBoxBuilderFactory; use MediaWiki\Extension\AbuseFilter\FilterLookup; diff --git a/includes/View/AbuseFilterViewTestBatch.php b/includes/View/AbuseFilterViewTestBatch.php index eca8009f1..591fe3b0f 100644 --- a/includes/View/AbuseFilterViewTestBatch.php +++ b/includes/View/AbuseFilterViewTestBatch.php @@ -2,10 +2,10 @@ namespace MediaWiki\Extension\AbuseFilter\View; -use AbuseFilterChangesList; use ActorMigration; use HTMLForm; use IContextSource; +use MediaWiki\Extension\AbuseFilter\AbuseFilterChangesList; use MediaWiki\Extension\AbuseFilter\AbuseFilterPermissionManager; use MediaWiki\Extension\AbuseFilter\EditBoxBuilderFactory; use MediaWiki\Extension\AbuseFilter\Parser\ParserFactory as AfParserFactory; diff --git a/maintenance/AddMissingLoggingEntries.php b/maintenance/AddMissingLoggingEntries.php index 47902e638..fe742170f 100644 --- a/maintenance/AddMissingLoggingEntries.php +++ b/maintenance/AddMissingLoggingEntries.php @@ -122,7 +122,7 @@ class AddMissingLoggingEntries extends LoggedUpdateMaintenance { continue; } - // This copies the code in AbuseFilter::doSaveFilter + // This copies the code in FilterStore $logEntry = new ManualLogEntry( 'abusefilter', 'modify' ); $logEntry->setPerformer( $user ); $logEntry->setTarget( SpecialAbuseFilter::getTitleForSubpage( $row->afh_filter ) ); diff --git a/maintenance/UpdateVarDumps.php b/maintenance/UpdateVarDumps.php index cfd2005ed..66a6eb22e 100644 --- a/maintenance/UpdateVarDumps.php +++ b/maintenance/UpdateVarDumps.php @@ -525,7 +525,7 @@ class UpdateVarDumps extends LoggedUpdateMaintenance { private function doUpdateText( IResultWrapper $res, ExternalStoreAccess $esAccess ) { $orphaned = []; foreach ( $res as $row ) { - // This is copied from AbuseFilter::loadVarDump + // This is copied from the old AbuseFilter::loadVarDump $oldFlags = explode( ',', $row->old_flags ); $text = $row->old_text; if ( in_array( 'external', $oldFlags ) ) { @@ -565,7 +565,7 @@ class UpdateVarDumps extends LoggedUpdateMaintenance { $varArray = $this->updateVariables( $varArray ); // Recreating flags will also ensure that we don't add 'nativeDataArray' $newFlags = [ 'utf-8' ]; - // This is copied from AbuseFilter::storeVarDump + // This is copied from the old AbuseFilter::storeVarDump $toStore = FormatJson::encode( $varArray ); if ( in_array( 'gzip', $oldFlags ) && function_exists( 'gzdeflate' ) ) { $toStore = gzdeflate( $toStore ); diff --git a/tests/phpunit/AbuseFilterConsequencesTest.php b/tests/phpunit/AbuseFilterConsequencesTest.php index 16789a649..4d846cb59 100644 --- a/tests/phpunit/AbuseFilterConsequencesTest.php +++ b/tests/phpunit/AbuseFilterConsequencesTest.php @@ -1,6 +1,7 @@