Page MenuHomePhabricator

Special:Notifications broken when EventLogging extension is not enabled
Closed, ResolvedPublic

Description

When trying to open Special:Notifications on current master (8947629f527f9486ae3e8713de6b8ed98b4dbf95) I am getting an error:

PHP Fatal error:  Class 'EventLogging' not found in (...)/extensions/Echo/includes/EventLogging.php on line 16

I don't have EventLogging extension enabled on my wiki but previously it hasn't affected the way Echo worked.

Apparently there is a check missing before logging added in Icfe63fb8daa8e8fba029e22b4de3e4f65517c448 which leads to a crash.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 305005 had a related patch set uploaded (by WMDE-leszek):
Check if logging is enabled before logging Special:Notifcations page visit

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

Change 305005 merged by jenkins-bot:
Check if logging is enabled before logging EchoInteraction

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

Checked locally - no crashes.

jmatazzoni closed this task as Resolved.Aug 19 2016, 12:00 AM

Change 305603 had a related patch set uploaded (by Mattflaschen):
Make server-side logging always check whether schema is enabled

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

Change 305603 merged by jenkins-bot:
Make server-side logging always check whether schema is enabled

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