Revert live hack from r36493, because it has no effect whatsoever.

$params['action'] cannot be an invalid action, because the $params array is filled by ApiBase::extractRequestParams() which checks whether the parameters are valid and throws an error message (ironically implemented with a UsageException) if they're not.

diff --git a/includes/api/ApiMain.php b/includes/api/ApiMain.php
index 84fc81f..b0096d0 100644 (file)
--- a/includes/api/ApiMain.php
+++ b/includes/api/ApiMain.php
@@ -339,12 +339,7 @@ class ApiMain extends ApiBase {
 
                $this->mShowVersions = $params['version'];
                $this->mAction = $params['action'];
-               
-               # For debugging -- TS
-               if ( !isset( $this->mModules[$this->mAction] ) ) {
-                       throw new MWException( "Invalid action: {$this->mAction}" );
-               }
-               
+
                // Instantiate the module requested by the user
                $module = new $this->mModules[$this->mAction] ($this, $this->mAction);