Description
Description
Details
Details
| Subject | Repo | Branch | Lines /- | |
|---|---|---|---|---|
| Move counting of watchers to WatchedItemStore | mediawiki/core | master | 252 -30 | |
| Move countVisitingWatchers to WatchedItemStore | mediawiki/core | master | 85 -17 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | ifried | T8964 Watch pages for a few days only (add an expiry time) | |||
| Resolved | MusikAnimal | T100508 Watchlist expiry: Watch pages for a specified time frame (2013) | |||
| Open | None | T129478 Refactor watchlist related code to use WatchedItemStore | |||
| Resolved | Addshore | T129479 Move watchlist related code in InfoAction to WatchedItemStore |
Event Timeline
Comment Actions
Change 266572 had a related patch set uploaded (by Addshore):
Move counting of watchers to WatchedItemStore
Comment Actions
Change 268650 had a related patch set uploaded (by Addshore):
Move countVisitingWatchers to WatchedItemStore
Comment Actions
How about the copies of those queries that live in the API code? Is this an opportunity to reduce duplication?
Comment Actions
See T129482 and particularly my comment at T129482#2107083
The use in the API does things like $this->addTables $this->addColums etc and does not look quite as easy to just swap out, although it is still on the cards and tracked by the above ticket.