Legoktm has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/96677


Change subject: Only run onUserSaveSettings hook if not in update.php
......................................................................

Only run onUserSaveSettings hook if not in update.php

Bug: 57335
Change-Id: I4cf977ee4e0dfd5c564334481e9ae2f6bca85b61
---
M Hooks.php
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo 
refs/changes/77/96677/1

diff --git a/Hooks.php b/Hooks.php
index 758a1c5..add295f 100644
--- a/Hooks.php
+++ b/Hooks.php
@@ -816,7 +816,11 @@
                // Reset the notification count since it may have changed due 
to user
                // option changes. This covers both explicit changes in the 
preferences
                // and changes made through the options API (since both call 
this hook).
-               MWEchoNotifUser::newFromUser( $user )->resetNotificationCount();
+               if ( !defined( 'MW_UPDATER' ) ) {
+                       // Extensions like AbuseFilter might create an account, 
but
+                       // the tables we need might not exist. Bug 57335
+                       MWEchoNotifUser::newFromUser( $user 
)->resetNotificationCount();
+               }
                return true;
        }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/96677
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4cf977ee4e0dfd5c564334481e9ae2f6bca85b61
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Legoktm <legoktm.wikipe...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to