[MediaWiki-l] deleteOldRevisions.php is an accident waiting to happen

Jean Valjean jeanvaljean2718 at gmail.com
Sat Jul 1 19:21:58 UTC 2017


Yeah, the reason I didn't submit it as a patch is that all I did was hack
the extension to operate based on namespace rather than page IDs. Some
people might look at that as a regression, if they wanted to continue using
page IDs. I'm not really sure what would be the best way to make it capable
of handling either page IDs or namespaces, at the option of the user,
unless we were going to have it read in the page IDs from a file.

I wonder, does anyone have any use cases where they would prefer to use
page IDs rather than page titles? I suspect the reason it was originally
written to use page IDs is that it was simpler to code, given that the
revision table has a rev_page field (for page ID) but no rev_page_namespace
or rev_page_title field.

Some other possibilities are that it could delete old revisions of pages in
a specified category or categories.

On Sat, Jul 1, 2017 at 2:12 PM, Greg Rundlett (freephile) <
greg at freephile.com> wrote:

> I don't know how easy it is to get pull requests merged, but if you want to
> get these improvements into core, you could click the 'edit' pen at
> https://github.com/wikimedia/mediawiki/blob/master/maintenance/
> deleteOldRevisions.php
> and submit a pull request.
>


More information about the MediaWiki-l mailing list