Page MenuHomePhabricator

Unused class ApiNewsletterManage.php in Newsletter extension ?
Closed, ResolvedPublic

Description

This class --> /var/www/core/core/extensions/Newsletter/includes/api/ApiNewsletterManage.php seems to be dead ?

Eitherway, it accepts a param 'publisher' which I dont think is used anymore.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 13 2017, 10:19 PM

Change 371762 had a related patch set uploaded (by 01tonythomas; owner: 01tonythomas):
[mediawiki/extensions/Newsletter@master] Remove unused API class ApiNewsletterManage

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

Change 371762 merged by jenkins-bot:
[mediawiki/extensions/Newsletter@master] Remove unused API class ApiNewsletterManage

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

Qgil added a subscriber: Qgil.

The patch has been merged. Is the problem fixed now?

01tonythomas closed this task as Resolved.Aug 30 2017, 6:28 AM

The patch has been merged. Is the problem fixed now?

Yes it is!