Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Reduce the length of 6 long lines of code

Browse source 
This shortens some lines below 120 characters when a tab is
counted as 4 characters (not that line lengths are currently
counted like that).

Bug: T243598
Change-Id: I828cd540268810bd56589885e38ad03f8bafc6f9
This commit is contained in:
Sam Wilson 2020-03-13 08:55:30 +08:00
parent 3fb7e265eb
commit bada529138
4 changed files with 17 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
includes/api/ApiBase.php
View file

@ -735,7 +735,11 @@ abstract class ApiBase extends ContextSource {
$newName = str_replace( $placeholder, $value, $name );
if ( !$targets ) {
try {
$results[$newName] = $this->getParameterFromSettings( $newName, $settings, $parseLimit );
$results[$newName] = $this->getParameterFromSettings(
$newName,
$settings,
$parseLimit
);
} catch ( ApiUsageException $ex ) {
$results[$newName] = $ex;
}

6
includes/api/ApiComparePages.php
View file

@ -134,7 +134,8 @@ class ApiComparePages extends ApiBase {
if ( !$toRev ) {
$title = Title::newFromLinkTarget( $title );
$this->dieWithError(
[ 'apierror-missingrev-title', wfEscapeWikiText( $title->getPrefixedText() ) ], 'nosuchrevid'
[ 'apierror-missingrev-title', wfEscapeWikiText( $title->getPrefixedText() ) ],
'nosuchrevid'
);
}
$toRelRev = $toRev;
@ -406,7 +407,8 @@ class ApiComparePages extends ApiBase {
if ( !$suppliedContent ) {
if ( $title->exists() ) {
$this->dieWithError(
[ 'apierror-missingrev-title', wfEscapeWikiText( $title->getPrefixedText() ) ], 'nosuchrevid'
[ 'apierror-missingrev-title', wfEscapeWikiText( $title->getPrefixedText() ) ],
'nosuchrevid'
);
} else {
$this->dieWithError(

3
includes/api/ApiPageSet.php
View file

@ -1229,7 +1229,8 @@ class ApiPageSet extends ApiBase {
$special = $spFactory->getPage( $dbkey );
if ( $special instanceof RedirectSpecialArticle ) {
// Only RedirectSpecialArticle is intended to redirect to an article, other kinds of
// RedirectSpecialPage are probably applying weird URL parameters we don't want to handle.
// RedirectSpecialPage are probably applying weird URL parameters we don't want to
// handle.
$context = new DerivativeContext( $this );
$context->setTitle( $titleObj );
$context->setRequest( new FauxRequest );

8
includes/logging/LogFormatter.php
View file

@ -302,7 +302,9 @@ class LogFormatter {
switch ( $entry->getSubtype() ) {
case 'protect':
$text = wfMessage( 'protectedarticle' )
->rawParams( $target . ' ' . $parameters['4::description'] )->inContentLanguage()->escaped();
->rawParams( $target . ' ' . $parameters['4::description'] )
->inContentLanguage()
->escaped();
break;
case 'unprotect':
$text = wfMessage( 'unprotectedarticle' )
@ -310,7 +312,9 @@ class LogFormatter {
break;
case 'modify':
$text = wfMessage( 'modifiedarticleprotection' )
->rawParams( $target . ' ' . $parameters['4::description'] )->inContentLanguage()->escaped();
->rawParams( $target . ' ' . $parameters['4::description'] )
->inContentLanguage()
->escaped();
break;
case 'move_prot':
$text = wfMessage( 'movedarticleprotection' )