Aude has uploaded a new change for review. https://gerrit.wikimedia.org/r/188203
Change subject: Avoid empty list input for DatabaseBase::makeList in subscriptions ...................................................................... Avoid empty list input for DatabaseBase::makeList in subscriptions Bug: T88329 Change-Id: I8d5231288003e21015e126b56af010ad61b93659 --- M client/includes/Usage/Sql/SqlSubscriptionManager.php 1 file changed, 23 insertions(+), 18 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase refs/changes/03/188203/1 diff --git a/client/includes/Usage/Sql/SqlSubscriptionManager.php b/client/includes/Usage/Sql/SqlSubscriptionManager.php index 97a9ca0..2d3bb1a 100644 --- a/client/includes/Usage/Sql/SqlSubscriptionManager.php +++ b/client/includes/Usage/Sql/SqlSubscriptionManager.php @@ -125,17 +125,20 @@ * @return string[] Entity ID strings from $subscriptions which $subscriber is already subscribed to. */ private function querySubscriptions( DatabaseBase $db, $subscriber, array $subscriptions ) { - $rows = $db->select( - 'wb_changes_subscription', - 'cs_entity_id', - array( - 'cs_subscriber_id' => $subscriber, - 'cs_entity_id' => $subscriptions, - ), - __METHOD__ - ); + if ( $subscriptions ) { + $rows = $db->select( + 'wb_changes_subscription', + 'cs_entity_id', + array( + 'cs_subscriber_id' => $subscriber, + 'cs_entity_id' => $subscriptions, + ), + __METHOD__ + ); - $subscriptions = $this->extractField( $rows, 'cs_entity_id' ); + $subscriptions = $this->extractField( $rows, 'cs_entity_id' ); + } + return $subscriptions; } @@ -165,14 +168,16 @@ * @param string[] $subscriptions */ private function deleteSubscriptions( DatabaseBase $db, $subscriber, array $subscriptions ) { - $db->delete( - 'wb_changes_subscription', - array( - 'cs_subscriber_id' => $subscriber, - 'cs_entity_id' => $subscriptions, - ), - __METHOD__ - ); + if ( $subscriptions ) { + $db->delete( + 'wb_changes_subscription', + array( + 'cs_subscriber_id' => $subscriber, + 'cs_entity_id' => $subscriptions, + ), + __METHOD__ + ); + } } /** -- To view, visit https://gerrit.wikimedia.org/r/188203 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I8d5231288003e21015e126b56af010ad61b93659 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: master Gerrit-Owner: Aude <aude.w...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits