Follow up to rEFLW39ea2b9964f4: Use ArticleRevisionUndeleted instead of ArticleUndelete / T122262: Improve Flow deletion/undeletion resilience:
It now does a transaction per undeleted Flow revision. It would be better to do it per overall undeletion operation. However, the page ID is passed to ArticleRevisionUndeleted, and it can vary by archived revision.
| Subject | Author | Repo | Branch | Lines +/- | |
|---|---|---|---|---|---|
| Refactor BoardMover so there is one transaction for all undeleted revs | Mattflaschen | mediawiki/extensions/Flow | master | +52 -15 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | • Mattflaschen-WMF | T122262 Improve Flow deletion/undeletion resilience | |||
| Resolved | • Mattflaschen-WMF | T135883 Better transactionality for Flow undeletion |
Change 290360 had a related patch set uploaded (by Mattflaschen):
Refactor BoardMover so there is one transaction for all undeleted revs
Change 290360 merged by jenkins-bot:
Refactor BoardMover so there is one transaction for all undeleted revs
Checked in betalabs - no new deletion issues have been found.
Clicking 'Delete' on Flow board page redirects to action=delete page that states that there are certain number of revision for this page, e.g. "You are about to delete Talk:ET1 along with all of its history." where "history" is a link to the page history. However, Flow page history does not clearly list those revisions (and, of course, there is no way to select a specific revision from the history page).
Wikitext pages have more transparent workflow for deletion - their history page is titled as e.g. "Revision history of Mavetuna5813" and allows selection of a certain version for deletion.