jenkins-bot has submitted this change and it was merged.
Change subject: ApiQueryExtracts: Call setWarning() on $this, not $result
......................................................................
ApiQueryExtracts: Call setWarning() on $this, not $result
Calling it on $result results in the warning looking like it's coming
from the 'result' module rather than 'textextracts'.
Change-Id: I396409c5e1d09385d9b564ca8cf854c899bb1ba4
---
M ApiQueryExtracts.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Legoktm: Looks good to me, approved
jenkins-bot: Verified
diff --git a/ApiQueryExtracts.php b/ApiQueryExtracts.php
index 0b5c37c..627dc0c 100644
--- a/ApiQueryExtracts.php
+++ b/ApiQueryExtracts.php
@@ -47,7 +47,7 @@
$limit = intval( $params['limit'] );
if ( $limit > 1 && !$params['intro'] ) {
$limit = 1;
- $result->setWarning( "exlimit was too large for a whole
article extracts request, lowered to $limit" );
+ $this->setWarning( "exlimit was too large for a whole
article extracts request, lowered to $limit" );
}
if ( isset( $params['continue'] ) ) {
$continue = intval( $params['continue'] );
--
To view, visit https://gerrit.wikimedia.org/r/171890
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I396409c5e1d09385d9b564ca8cf854c899bb1ba4
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TextExtracts
Gerrit-Branch: master
Gerrit-Owner: Anomie <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Waldir <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
