I disagree, I wrote lots of codes that deletion is a part of them (e.g. this code for deleting items in Wikidata https://www.wikidata.org/wiki/User:Ladsgroup/del.py). Maybe there is one usage of them in core but it has been widely used in customized codes.
Best
On Mon, Jun 16, 2014 at 8:17 PM, John Mark Vandenberg jayvdb@gmail.com wrote:
On 16 June 2014 16:33, John phoenixoverride@gmail.com wrote:
Avoiding the headache of re-inventing the wheel across dozens of scripts that use that code?
There is only one use of Page.protect() , even in compat, so IMO that code should mostly be moved from Page to scripts/protect.py.
There is also only one use of Page.delete(mark=True), so IMO that code should mostly be moved from Page to scripts/category.py.
There are many more uses of Page.delete() prompt=True, especially in compat, so I can see the need for re-use here.
On Mon, Jun 16, 2014 at 10:11 PM, Merlijn van Deen valhallasw@arctus.nl wrote:
Then it should be in the default 'bot' class, not in the Page class.
I like that approach. I'll work on it after my other Bot changeset is checked in.
https://gerrit.wikimedia.org/r/#/c/139309/
-- John Vandenberg
Pywikipedia-l mailing list Pywikipedia-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/pywikipedia-l