jenkins-bot has submitted this change and it was merged.

Change subject: Fix retrieval of query modules
......................................................................


Fix retrieval of query modules

Bug: 69872
Change-Id: I79bd2901c92c2afd3b294aa3c52f4cc35a6cf4d8
(cherry picked from commit 0a238ce1a06ff49c50d62e12ec252221a0d688d6)
---
M SpecialApiSandbox.php
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  MaxSem: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/SpecialApiSandbox.php b/SpecialApiSandbox.php
index d25d3f8..f091f32 100644
--- a/SpecialApiSandbox.php
+++ b/SpecialApiSandbox.php
@@ -129,9 +129,9 @@
         */
        private function getQueryModules( $type ) {
                $options = array();
-               $params = $this->apiQuery->getAllowedParams();
-               sort( $params[$type][ApiBase::PARAM_TYPE] );
-               foreach ( $params[$type][ApiBase::PARAM_TYPE] as $module ) {
+               $modules = $this->apiQuery->getModuleManager()->getNames( $type 
);
+               sort( $modules );
+               foreach ( $modules as $module ) {
                        $options["$type=$module"] = "$type=$module";
                }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/156502
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I79bd2901c92c2afd3b294aa3c52f4cc35a6cf4d8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ApiSandbox
Gerrit-Branch: wmf/1.24wmf18
Gerrit-Owner: MaxSem <maxsem.w...@gmail.com>
Gerrit-Reviewer: MaxSem <maxsem.w...@gmail.com>
Gerrit-Reviewer: Wctaiwan <wctai...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to