HomePhabricator

Cleanup debugging code for isTalkpageManagerUser

Description

Cleanup debugging code for isTalkpageManagerUser

  • Remove ArticleEditUpdateNewTalk debug code. It's unreachable since the hook listener itself has a type-hint on user.
  • Tweak BeforeEchoEventInsert. It is normal for it to be null for some events, so handle that explicitly and don't treat it as a special case.
  • Remove the debug code for AbortEmailNotification, but keep the return true. This should not really happen, but it doesn't hurt to handle it.

Bug: T109063
Change-Id: Ibfbd59914ea2026e1283d94dda40903d7c6de7fa

Details

Committed
Mattflaschen-WMFAug 19 2015, 7:44 PM
Parents
rEFLW99c81fb31cfc: Merge "Turn spaces into tabs"
Branches
Unknown
Tags
Unknown
References
refs/changes/76/232576/1
ChangeId
Ibfbd59914ea2026e1283d94dda40903d7c6de7fa