Page MenuHomePhabricator

Notifications from no-longer-enabled extensions which use custom formatters cause errors
Closed, ResolvedPublic

Description

For example, the PageTriage extension creates 'reviewed' notices and specifies their format. If you turn off PageTriage, all of its notices are still listed in your Notifications flyout, but they are all broken due to the missing formatter.

Perhaps notifications that are missing formatters should just not be shown at all.


Version: master
Severity: major

Details

Reference
bz41866

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 12:48 AM
bzimport added a project: Notifications.
bzimport set Reference to bz41866.

Also, the error that we do have:

  • Overlaps with text, like notifications did with each other in bug 41968.
  • Is actually a missing message.

(In reply to comment #1)

  • Is actually a missing message.

Oops. This was fixed in Gerrit change 15959. Though it could probably use $1, the event type.