Aaron Schulz has uploaded a new change for review.
https://gerrit.wikimedia.org/r/250304
Change subject: Make notifications use getMainStashInstance()
......................................................................
Make notifications use getMainStashInstance()
This makes sure resetNotificationCount() actually affects all DCs
Change-Id: If1af121b54d2ec50473a55308d8326f24a1b43b8
---
M includes/NotifUser.php
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo
refs/changes/04/250304/1
diff --git a/includes/NotifUser.php b/includes/NotifUser.php
index 7fcb0c0..911a382 100644
--- a/includes/NotifUser.php
+++ b/includes/NotifUser.php
@@ -68,10 +68,10 @@
if ( $user->isAnon() ) {
throw new MWException( 'User must be logged in to view
notification!' );
}
- global $wgMemc;
return new MWEchoNotifUser(
- $user, $wgMemc,
+ $user,
+ ObjectCache::getMainStashInstance(),
new EchoUserNotificationGateway( $user,
MWEchoDbFactory::newFromDefault() ),
new EchoNotificationMapper(),
new EchoTargetPageMapper()
--
To view, visit https://gerrit.wikimedia.org/r/250304
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If1af121b54d2ec50473a55308d8326f24a1b43b8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
