Page MenuHomePhabricator

Newsletter cleanup
Closed, ResolvedPublic

Description

EchoNewsletterUserLocator:

  • return User[]|array mismatch with return UserArray::newFromIDs( $newsletter->getSubscribers() );

NewsletterDeletionUpdate

  • $store->deleteNewsletter( $this->newsletter ); missing reason parameter
  • getSubscribersFromID return type mismatch

NewsletterTablePager

  • IDatabase hint used for $mDB, but no tableName method on iDatabase

SpecialNewsletters

  • $filterForm->setSubmitId case mismatch on function

NewsletterEditPage

  • createNew should be explicitly defined as a member variable

NewsletterStore

  • getDefaultInstance should have return docs
  • getSubscribersFromID return type mismatch with docs

Event Timeline

FYI @Devirk You don't have to create tasks for each individual one

You can put multiple commits against this bug if you want

Filip updated the task description. (Show Details)

Change 329680 had a related patch set uploaded (by Filip):
Newsletter cleanup

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

Impoted Newsletter cleanup 1/2 [NewsletterEditPage, NewsletterStore] as https://codein.withgoogle.com/dashboard/tasks/5454349838843904/ waiting for admins to approve!

Change 329680 merged by jenkins-bot:
Newsletter cleanup

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

@MtDu please mark the ones you have completed in the task description as well, and resolve this thing!

MtDu removed a project: Patch-For-Review.
MtDu updated the task description. (Show Details)

All errors have been fixed. Hence closing as resolved!
Thanks!