Page MenuHomePhabricator

Hard deprecate Title::isWatchable()
Closed, ResolvedPublic

Description

Replace usages of Title::isWatchable() with WatchlistManager::isWatchable() and hard deprecate Title::isWatchable().

Wikimedia production extensions/skins:

  • Flow
  • MinervaNeue

Other extensions/skins:

  • BlueSpiceSocialWatch
  • WatchAnalytics (filed an issue, but not a patch since the extension does not currently support MW 1.36+)

Core:

  • Remove usages in core
  • Hide deprecations in Title tests
  • Hard deprecate Title::isWatchable()

Event Timeline

Change 683688 had a related patch set uploaded (by Cicalese; author: Cicalese):

[mediawiki/extensions/Flow@master] Replace Title::isWatchable() with WatchlistManager::isWatchable()

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

Change 683689 had a related patch set uploaded (by Cicalese; author: Cicalese):

[mediawiki/extensions/BlueSpiceSocialWatch@master] Replace Title::isWatchable() with WatchlistManager::isWatchable()

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

Change 683690 had a related patch set uploaded (by Cicalese; author: Cicalese):

[mediawiki/skins/MinervaNeue@master] Replace Title::isWatchable() with WatchlistManager::isWatchable()

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

Change 683693 had a related patch set uploaded (by Cicalese; author: Cicalese):

[mediawiki/core@master] Replace Title::isWatchable() with WatchlistManager::isWatchable()

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

Change 683697 had a related patch set uploaded (by Cicalese; author: Cicalese):

[mediawiki/core@master] Hard deprecate Title::isWatchable()

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

Change 683690 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Replace Title::isWatchable() with WatchlistManager::isWatchable()

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

Change 683689 merged by jenkins-bot:

[mediawiki/extensions/BlueSpiceSocialWatch@master] Replace Title::isWatchable() with WatchlistManager::isWatchable()

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

Change 683693 merged by jenkins-bot:

[mediawiki/core@master] Replace Title::isWatchable() with WatchlistManager::isWatchable()

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

Change 683688 merged by jenkins-bot:

[mediawiki/extensions/Flow@master] Replace Title::isWatchable() with WatchlistManager::isWatchable()

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

Change 683697 merged by jenkins-bot:

[mediawiki/core@master] Hard deprecate Title::isWatchable()

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