* Using var_dump() with output buffering instead of print_r(), allows for

  distinguishing between an empty string and NULL
* Formatting
* Documentation: @copyright...

diff --git a/maintenance/parserTestsParserHook.php b/maintenance/parserTestsParserHook.php
index 56d1e6b..ae50ee7 100644 (file)
--- a/maintenance/parserTestsParserHook.php
+++ b/maintenance/parserTestsParserHook.php
@@ -1,5 +1,6 @@
 <?php
-if (!defined('MEDIAWIKI')) die();
+if ( ! defined( 'MEDIAWIKI' ) )
+       die();
 /**
  * A basic extension that's used by the parser tests to test whether input and
  * arguments are passed to extensions properly.
@@ -8,6 +9,7 @@ if (!defined('MEDIAWIKI')) die();
  * @subpackage Maintenance
  *
  * @author Ævar Arnfjörð Bjarmason <avarab@gmail.com>
+ * @copyright Copyright © 2005, 2006 Ævar Arnfjörð Bjarmason
  * @license http://www.gnu.org/copyleft/gpl.html GNU General Public License 2.0 or later
  */
 
@@ -18,8 +20,12 @@ function wfParserTestSetup( &$parser ) {
 }
 
 function wfParserTestHook( $in, $argv ) {
-       if ( count( $argv ) )
-               return "<pre>\n" . print_r( $argv, true ) . '</pre>';
-       else
-               return $in;
+       ob_start();
+       var_dump(
+               $in,
+               $argv
+       );
+       $ret = ob_get_clean();
+
+       return "<pre>\n$ret</pre>";
 }