Yaron Koren has submitted this change and it was merged. Change subject: Added "having" param for #cargo_compound_query ......................................................................
Added "having" param for #cargo_compound_query Change-Id: I6a28d09c8f2a117991d42c33f4ecb0480f0b837d --- M parserfunctions/CargoCompoundQuery.php 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Yaron Koren: Checked; Looks good to me, approved diff --git a/parserfunctions/CargoCompoundQuery.php b/parserfunctions/CargoCompoundQuery.php index bf26335..8551775 100644 --- a/parserfunctions/CargoCompoundQuery.php +++ b/parserfunctions/CargoCompoundQuery.php @@ -48,6 +48,7 @@ $whereStr = null; $joinOnStr = null; $groupByStr = null; + $havingStr = null; $orderByStr = null; $limitStr = null; @@ -70,6 +71,8 @@ $joinOnStr = $value; } elseif ( $key == 'group by' ) { $groupByStr = $value; + } elseif ( $key == 'having' ) { + $havingStr = $value; } elseif ( $key == 'order by' ) { $orderByStr = $value; } elseif ( $key == 'limit' ) { @@ -79,7 +82,7 @@ } } $sqlQueries[] = CargoSQLQuery::newFromValues( $tablesStr, $fieldsStr, $whereStr, $joinOnStr, - $groupByStr, $orderByStr, $limitStr ); + $havingStr, $groupByStr, $orderByStr, $limitStr ); $querySpecificParams[] = $displayParamsForThisQuery; } -- To view, visit https://gerrit.wikimedia.org/r/212298 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6a28d09c8f2a117991d42c33f4ecb0480f0b837d Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Cargo Gerrit-Branch: master Gerrit-Owner: Yaron Koren <yaro...@gmail.com> Gerrit-Reviewer: Yaron Koren <yaro...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits