jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/377775 )
Change subject: Make CentralNotice work on sqlite
......................................................................
Make CentralNotice work on sqlite
group_concat_max_length is mysql specific.
Change-Id: I4685387b2d21723afa74ceddb957ab917de62c55
---
M includes/CNCampaignPager.php
1 file changed, 4 insertions(+), 1 deletion(-)
Approvals:
Addshore: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/CNCampaignPager.php b/includes/CNCampaignPager.php
index b615a19..a12de79 100644
--- a/includes/CNCampaignPager.php
+++ b/includes/CNCampaignPager.php
@@ -108,7 +108,10 @@
public function doQuery() {
// group_concat output is limited to 1024 characters by
default, increase
// the limit temporarily so the list of all languages can be
rendered.
- $this->getDatabase()->query( 'SET SESSION group_concat_max_len
= 10000' );
+ $db = $this->getDatabase();
+ if ( $db->getType() === 'mysql' ) {
+ $db->query( 'SET SESSION group_concat_max_len = 10000',
__METHOD__ );
+ }
parent::doQuery();
}
--
To view, visit https://gerrit.wikimedia.org/r/377775
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4685387b2d21723afa74ceddb957ab917de62c55
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/CentralNotice
Gerrit-Branch: master
Gerrit-Owner: Brian Wolff <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: AndyRussG <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>
Gerrit-Reviewer: XenoRyet <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
