Page MenuHomePhabricator

Research the usage of NewsletterDeletionUpdate (and probably remove it)
Closed, ResolvedPublic

Description

Currently, it does not look like the NewsletterDeletionUpdate class does anything. It looks like it is an unnecessary class that doesn't add any new functionality. Please research and/or confirm this. If indeed it is useless, remove it from the extension.

Related Objects

StatusSubtypeAssignedTask
DuplicateQgil
ResolvedQgil
ResolvedQgil
DeclinedNone
ResolvedAddshore
DuplicateNone
Resolvedori
ResolvedBawolff
ResolvedGlaisher
ResolvedTinaj1234
DeclinedNone
ResolvedGlaisher
Resolved01tonythomas
ResolvedGlaisher
Resolved01tonythomas
OpenNone
ResolvedDaimona
ResolvedDaimona
Resolved01tonythomas
ResolvedQgil
ResolvedFilip
ResolvedxSavitar
ResolvedNone
Resolved01tonythomas
Resolved01tonythomas
Resolved01tonythomas
Resolved01tonythomas
ResolvedPppery

Event Timeline

Change 329804 had a related patch set uploaded (by Pppery):
Remove unnecessary class NewsletterDeletionUpdate

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

The class is indeed redundant; the functionality it provides it also implemented in Newsletter.hooks.php in OnArticleDelete.

Killed with fire :D Thank you!

Change 329804 merged by jenkins-bot:
Remove unnecessary class NewsletterDeletionUpdate

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