jenkins-bot has submitted this change and it was merged.
Change subject: api: Fix "PHP Notice: Undefined variable: config"
......................................................................
api: Fix "PHP Notice: Undefined variable: config"
Follows-up 362c220aa1, which split the method up but didn't
re-create a local $config variable.
Change-Id: Ic32ac3c600d0b4d69e8129fa50ea47d9a6a54b20
---
M includes/api/ApiMain.php
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
BryanDavis: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/api/ApiMain.php b/includes/api/ApiMain.php
index 4783fb4..9e56819 100644
--- a/includes/api/ApiMain.php
+++ b/includes/api/ApiMain.php
@@ -861,6 +861,7 @@
// User entered incorrect parameters - generate error
response
$errMessage = $e->getMessageArray();
} else {
+ $config = $this->getConfig();
// Something is seriously wrong
if ( ( $e instanceof DBQueryError ) && !$config->get(
'ShowSQLErrors' ) ) {
$info = 'Database query error';
--
To view, visit https://gerrit.wikimedia.org/r/274500
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic32ac3c600d0b4d69e8129fa50ea47d9a6a54b20
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Krinkle <[email protected]>
Gerrit-Reviewer: Anomie <[email protected]>
Gerrit-Reviewer: BryanDavis <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits