HomePhabricator

Updated mediawiki/extensions Project: mediawiki/extensions/Flow…

Authored by jenkins-bot <jenkins-bot@gerrit.wikimedia.org>.

Description

Updated mediawiki/extensions Project: mediawiki/extensions/Flow 4f80b33c03c1e93f4e452834a78bf75bb6432588

Disallow revisions in deleted boards

Unless user has deletedhistory permissions

This could have a performance impact: now every check will
require workflow to be loaded.
However, on "regular" pages, the workflow should've already
been loaded anyway. In listings (contribs, RC, history, ...),
these should be batchloaded & buffered cache will take care
of this.

Even though the performance hit should be negligable, if any,
I originally wanted to do this differenty: I wanted to hook
into WikiPageDeletionUpdates & delete every topic (and post?)
in that board. However:

  • I don't want to create a bogus delete revision
  • If board gets restored, it could get weird to figure out if a topic was auto-deleted along with board (and should also be restored), or was previously genuinly deleted
  • Perhaps workflow could be deleted instead, but that's going to require some refactoring & we'd still need it to render the revisions in lists
  • Haven't even looked if there's a hook after restoring a board (though there probably is, or we could add one)

Either way, that idea didn't look promising, but I'm open to
other wild suggestions.

Bug: T90969
Change-Id: I60b34868871ab17455cb2e03f6a44ac42e1c896e

Details

Committed
Gerrit Code ReviewApr 7 2015, 8:55 PM
Parents
rMEXTea0089b654b8: Updated mediawiki/extensions Project: mediawiki/extensions/Gather…
Branches
Unknown
Tags
Unknown
ChangeId
I60b34868871ab17455cb2e03f6a44ac42e1c896e

Event Timeline

Gerrit Code Review <gerrit@wikimedia.org> committed rMEXT9588c88e048c: Updated mediawiki/extensions Project: mediawiki/extensions/Flow… (authored by jenkins-bot <jenkins-bot@gerrit.wikimedia.org>).Apr 7 2015, 8:55 PM