Page MenuHomePhabricator

Introduce WatchedItemStore
Closed, ResolvedPublic13 Estimated Story Points

Event Timeline

Change 267259 had a related patch set uploaded (by Addshore):
Move WatchedItem logic to WatchedItemStore

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

Tobi_WMDE_SW set the point value for this task to 13.Mar 2 2016, 1:37 PM

Change 267259 merged by jenkins-bot:
Move WatchedItem logic to WatchedItemStore

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

Change 266572 had a related patch set uploaded (by Addshore):
Move counting of watchers to WatchedItemStore

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

Change 268650 had a related patch set uploaded (by Addshore):
Move countVisitingWatchers to WatchedItemStore

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

Once the following 2 patches are merged InfoAction will be done:

This will also probably be the most untangled ApiQueryInfo can be at this stage..

Change 276441 had a related patch set uploaded (by Addshore):
Add countUnreadNotifications to WatchedItemStore

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

Again, looking at:

  • ApiQueryWatchlist
  • ApiQueryWatchlistRaw

These look far to tangled to try and factor the watchlist table logic out, so they will probably not be tackled.

Refactoring other parts of the code to make use of WatchedItemStore is not tracked in T129478.

Refactoring other parts of the code to make use of WatchedItemStore is not tracked in T129478.

*now

Marking as resolved as this has indeed been introduced!

thx @Ricordisamoa, we'll update commit messages

Yup, Will do!