Glaisher has uploaded a new change for review.
https://gerrit.wikimedia.org/r/248339
Change subject: Remove duplicate method from NewsletterDb
......................................................................
Remove duplicate method from NewsletterDb
Change-Id: I40a5f681527e278f351f179f79755c766e165ea9
---
M includes/Echo/EchoNewsletterUserLocator.php
M includes/NewsletterDb.php
2 files changed, 3 insertions(+), 25 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Newsletter
refs/changes/39/248339/1
diff --git a/includes/Echo/EchoNewsletterUserLocator.php
b/includes/Echo/EchoNewsletterUserLocator.php
index b8cb733..ea477f3 100644
--- a/includes/Echo/EchoNewsletterUserLocator.php
+++ b/includes/Echo/EchoNewsletterUserLocator.php
@@ -10,7 +10,7 @@
public static function locateNewsletterSubscribedUsers( EchoEvent
$event ) {
$extra = $event->getExtra();
$ids = NewsletterDb::newFromGlobalState()
- ->getUserIdsSubscribedToNewsletter(
$extra['newsletterId'] );
+ ->getSubscribersFromID( $extra['newsletterId'] );
return UserArray::newFromIDs( $ids );
diff --git a/includes/NewsletterDb.php b/includes/NewsletterDb.php
index 4c0c26b..15061fe 100644
--- a/includes/NewsletterDb.php
+++ b/includes/NewsletterDb.php
@@ -52,28 +52,6 @@
}
/**
- * @param int $newsletterId
- *
- * @return int[]
- */
- public function getUserIdsSubscribedToNewsletter( $newsletterId ) {
- // @todo use selectFieldValues() here
- $res = $this->readDb->select(
- 'nl_subscriptions',
- array( 'nls_subscriber_id' ),
- array( 'nls_newsletter_id' => $newsletterId ),
- __METHOD__,
- array()
- );
-
- $subscriberIds = array();
- foreach ( $res as $row ) {
- $subscriberIds[] = $row->nls_subscriber_id;
- }
- return $subscriberIds;
- }
-
- /**
* @param int $userId
* @param int $newsletterId
*
@@ -166,7 +144,7 @@
/**
* @param int $id
*
- * @return array
+ * @return string[]
*/
public function getPublishersFromID( $id ) {
return $this->readDb->selectFieldValues(
@@ -180,7 +158,7 @@
/**
* @param int $id
*
- * @return array
+ * @return string[]
*/
public function getSubscribersFromID( $id ) {
return $this->readDb->selectFieldValues(
--
To view, visit https://gerrit.wikimedia.org/r/248339
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I40a5f681527e278f351f179f79755c766e165ea9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Newsletter
Gerrit-Branch: master
Gerrit-Owner: Glaisher <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
