Not sure why I originally used array_key_exists(). Switch for isset()
authorSam Reed <reedy@users.mediawiki.org>
Wed, 12 Jan 2011 01:17:31 +0000 (01:17 +0000)
committerSam Reed <reedy@users.mediawiki.org>
Wed, 12 Jan 2011 01:17:31 +0000 (01:17 +0000)
includes/api/ApiQuery.php

index 0f1d63d..89689d8 100644 (file)
@@ -190,15 +190,15 @@ class ApiQuery extends ApiBase {
         * @return mixed string or null
         */
        function getModuleType( $moduleName ) {
-               if ( array_key_exists ( $moduleName, $this->mQueryPropModules ) ) {
+               if ( isset( $this->mQueryPropModules[$moduleName] ) ) {
                        return 'prop';
                }
 
-               if ( array_key_exists ( $moduleName, $this->mQueryListModules ) ) {
+               if ( isset( $this->mQueryListModules[$moduleName] ) ) {
                        return 'list';
                }
 
-               if ( array_key_exists ( $moduleName, $this->mQueryMetaModules ) ) {
+               if ( isset( $this->mQueryMetaModules[$moduleName] ) ) {
                        return 'meta';
                }