Page MenuHomePhabricator

Update watchlist handling for `anon` property
Closed, ResolvedPublic1 Estimated Story Points

Description

Follow-up to T239883: Watchlist api: Anon flag omitted if false in formatversion=2

Now set to false instead of being omitted entirely

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
DannyS712 set the point value for this task to 1.
DannyS712 moved this task from Unsorted to Next on the User-DannyS712 board.
DannyS712 moved this task from Backlog to Later on the MediaWiki-extensions-GlobalWatchlist board.

Since currently the extension only requires MW 1.35+, this seems like a pretty weak reason to bump the requirement to 1.36+. Moving to later with the expectation that if the version requirement gets bumped for a different reason this should be done, but until then its not a big deal

Change 651601 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/GlobalWatchlist@master] Remove legacy handling of undefined anon property

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

Change 651601 merged by jenkins-bot:
[mediawiki/extensions/GlobalWatchlist@master] Remove legacy handling of undefined anon property

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

DannyS712 claimed this task.
DannyS712 removed a project: Patch-For-Review.