Instead, pass the page id when using methods for a page. The change
avoids constructing Database a dummy page id when those methods aren't
going to be used.
getFromId doesn't seem like it needs a page id, since the linter id is
the primary key.
Also, a namespace id should no longer optional to setForPage. The
LinterWriteNamespaceColumnStage option already gates whether to include
it in the row.
Follows-Up: I9fd6e7724dcf33be0b1feb19ec8eb448738cab09
Change-Id: Ib3d3622144b670ebe1a4ce04e6db6811584d42c8
And addresses some other cleanup from review comments.
Follows-Up: I9fd6e7724dcf33be0b1feb19ec8eb448738cab09
Change-Id: If87b0bf91930f0f8d89ed046d18aadb8f346f9aa
* Having separate config variables to enable the maintenance
migrateNamespace and migrateTagTemplate scripts is duplicitous
and should be shared with the write enable config variables.
Bug: T329342
Change-Id: I4cb453fc0678b065cb42a2ca59863da1ab9cdbe4
* It should be safe to automatically run during wiki update on
3rd party wikis, and requires manual invocation on enwiki.
* The tag and template migrate for enwiki will be run by an admin
during a backport window so the process can be monitored and
may run for a day potentially.
Bug: T175177
Change-Id: I6ce30767300cc907639d72b34f7756f3d9563ca7
* It should be safe to automatically run during wiki update on
3rd party wikis, and requires manual invocation on enwiki
* The namespace migrate for enwiki will be run by an admin during
a backport window so the process can be monitored and may run
for a day potentially.
Bug: T299612
Change-Id: I19091a6f177e8b67d80c0b8f46782929d2e951d4
