Merge "Re add wpScrolltop id in EditPage"

[lhc/web/wiklou.git] / maintenance / compareParsers.php

diff --git a/maintenance/compareParsers.php b/maintenance/compareParsers.php
index 723a6bf..f2540c7 100644 (file)
--- a/maintenance/compareParsers.php
+++ b/maintenance/compareParsers.php
@@ -43,15 +43,35 @@ class CompareParsers extends DumpIterator {
        public function __construct() {
                parent::__construct();
                $this->saveFailed = false;
-               $this->mDescription = "Run a file or dump with several parsers";
+               $this->addDescription( 'Run a file or dump with several parsers' );
                $this->addOption( 'parser1', 'The first parser to compare.', true, true );
                $this->addOption( 'parser2', 'The second parser to compare.', true, true );
                $this->addOption( 'tidy', 'Run tidy on the articles.', false, false );
-               $this->addOption( 'save-failed', 'Folder in which articles which differ will be stored.', false, true );
+               $this->addOption(
+                       'save-failed',
+                       'Folder in which articles which differ will be stored.',
+                       false,
+                       true
+               );
                $this->addOption( 'show-diff', 'Show a diff of the two renderings.', false, false );
-               $this->addOption( 'diff-bin', 'Binary to use for diffing (can also be provided by DIFF env var).', false, false );
-               $this->addOption( 'strip-parameters', 'Remove parameters of html tags to increase readability.', false, false );
-               $this->addOption( 'show-parsed-output', 'Show the parsed html if both Parsers give the same output.', false, false );
+               $this->addOption(
+                       'diff-bin',
+                       'Binary to use for diffing (can also be provided by DIFF env var).',
+                       false,
+                       false
+               );
+               $this->addOption(
+                       'strip-parameters',
+                       'Remove parameters of html tags to increase readability.',
+                       false,
+                       false
+               );
+               $this->addOption(
+                       'show-parsed-output',
+                       'Show the parsed html if both Parsers give the same output.',
+                       false,
+                       false
+               );
        }
 
        public function checkOptions() {
@@ -96,12 +116,13 @@ class CompareParsers extends DumpIterator {
                if ( !$this->stripParametersEnabled ) {
                        return $text;
                }
+
                return preg_replace( '/(<a) [^>]+>/', '$1>', $text );
        }
 
        /**
         * Callback function for each revision, parse with both parsers and compare
-        * @param $rev Revision
+        * @param Revision $rev
         */
        public function processRevision( $rev ) {
                $title = $rev->getTitle();
@@ -118,7 +139,9 @@ class CompareParsers extends DumpIterator {
                $content = $rev->getContent();
 
                if ( $content->getModel() !== CONTENT_MODEL_WIKITEXT ) {
-                       $this->error( "Page {$title->getPrefixedText()} does not contain wikitext but {$content->getModel()}\n" );
+                       $this->error( "Page {$title->getPrefixedText()} does not contain wikitext "
+                               . "but {$content->getModel()}\n" );
+
                        return;
                }
 
@@ -132,13 +155,21 @@ class CompareParsers extends DumpIterator {
                        $this->error( "Parsing for {$title->getPrefixedText()} differs\n" );
 
                        if ( $this->saveFailed ) {
-                               file_put_contents( $this->saveFailed . '/' . rawurlencode( $title->getPrefixedText() ) . ".txt", $text );
+                               file_put_contents(
+                                       $this->saveFailed . '/' . rawurlencode( $title->getPrefixedText() ) . ".txt",
+                                       $text
+                               );
                        }
                        if ( $this->showDiff ) {
-                               $this->output( wfDiff( $this->stripParameters( $output1->getText() ), $this->stripParameters( $output2->getText() ), '' ) );
+                               $this->output( wfDiff(
+                                       $this->stripParameters( $output1->getText() ),
+                                       $this->stripParameters( $output2->getText() ),
+                                       ''
+                               ) );
                        }
                } else {
                        $this->output( $title->getPrefixedText() . "\tOK\n" );
+
                        if ( $this->showParsedOutput ) {
                                $this->output( $this->stripParameters( $output1->getText() ) );
                        }
@@ -149,10 +180,9 @@ class CompareParsers extends DumpIterator {
                /* Look for the parser in a file appropiately named in the current folder */
                if ( !class_exists( $parserName ) && file_exists( "$parserName.php" ) ) {
                        global $wgAutoloadClasses;
-                       $wgAutoloadClasses[ $parserName ] = realpath( '.' ) . "/$parserName.php";
+                       $wgAutoloadClasses[$parserName] = realpath( '.' ) . "/$parserName.php";
                }
        }
-
 }
 
 $maintClass = "CompareParsers";