mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/AbuseFilter.git
synced 2024-11-24 06:03:49 +00:00
Remove comments literally repeating what the code says
I find it obvious that a file called "AbuseFilterTokenizerTest" is a "test for the AbuseFilterTokenizer class". A comment that is just repeating this information is typicalls not helpful, but distracting and a potential source of mistakes, e.g. when stuff is copy-pasted, but the comment not adjusted. Change-Id: I1d4cc06e9e5631955ff73bf675090cf9c33c9390
This commit is contained in:
parent
9cb58a4737
commit
fa3ce90851
|
@ -2608,7 +2608,6 @@ class AbuseFilter {
|
|||
$new_id = $dbw->insertId();
|
||||
}
|
||||
|
||||
// Actions
|
||||
$availableActions = $page->getConfig()->get( 'AbuseFilterActions' );
|
||||
$actionsRows = [];
|
||||
foreach ( array_filter( $availableActions ) as $action => $_ ) {
|
||||
|
@ -2640,7 +2639,6 @@ class AbuseFilter {
|
|||
$afh_row[$afh_col] = $newRow[$af_col];
|
||||
}
|
||||
|
||||
// Actions
|
||||
$displayActions = [];
|
||||
foreach ( $actions as $action ) {
|
||||
$displayActions[$action['action']] = $action['parameters'];
|
||||
|
@ -2649,7 +2647,6 @@ class AbuseFilter {
|
|||
|
||||
$afh_row['afh_changed_fields'] = implode( ',', $differences );
|
||||
|
||||
// Flags
|
||||
$flags = [];
|
||||
if ( $newRow['af_hidden'] ) {
|
||||
$flags[] = 'hidden';
|
||||
|
|
|
@ -339,7 +339,6 @@ class AbuseFilterViewDiff extends AbuseFilterView {
|
|||
$body .= $infoHeader . $info;
|
||||
}
|
||||
|
||||
// Pattern
|
||||
$patternHeader = $this->getHeaderRow( 'abusefilter-diff-pattern' );
|
||||
$pattern = '';
|
||||
$pattern .= $this->getDiffRow(
|
||||
|
@ -352,7 +351,6 @@ class AbuseFilterViewDiff extends AbuseFilterView {
|
|||
$body .= $patternHeader . $pattern;
|
||||
}
|
||||
|
||||
// Actions
|
||||
$actionsHeader = $this->getHeaderRow( 'abusefilter-edit-consequences' );
|
||||
$actions = '';
|
||||
|
||||
|
|
|
@ -1189,7 +1189,6 @@ class AbuseFilterViewEdit extends AbuseFilterView {
|
|||
$row->af_global = $request->getCheck( 'wpFilterGlobal' )
|
||||
&& $this->getConfig()->get( 'AbuseFilterIsCentral' );
|
||||
|
||||
// Actions
|
||||
$actions = [];
|
||||
foreach ( array_filter( $this->getConfig()->get( 'AbuseFilterActions' ) ) as $action => $_ ) {
|
||||
// Check if it's set
|
||||
|
|
|
@ -1,7 +1,5 @@
|
|||
<?php
|
||||
/**
|
||||
* Tests for the AFPData class
|
||||
*
|
||||
* This program is free software; you can redistribute it and/or modify
|
||||
* it under the terms of the GNU General Public License as published by
|
||||
* the Free Software Foundation; either version 2 of the License, or
|
||||
|
|
|
@ -1,7 +1,5 @@
|
|||
<?php
|
||||
/**
|
||||
* Tests for the AbuseFilter parser
|
||||
*
|
||||
* This program is free software; you can redistribute it and/or modify
|
||||
* it under the terms of the GNU General Public License as published by
|
||||
* the Free Software Foundation; either version 2 of the License, or
|
||||
|
|
|
@ -1,7 +1,5 @@
|
|||
<?php
|
||||
/**
|
||||
* Tests for the AFPData class
|
||||
*
|
||||
* This program is free software; you can redistribute it and/or modify
|
||||
* it under the terms of the GNU General Public License as published by
|
||||
* the Free Software Foundation; either version 2 of the License, or
|
||||
|
|
Loading…
Reference in a new issue