This script is wanted by Wikisource
http://git.wikimedia.org/blob/pywikibot%2Fcompat.git/master/patrol.py
Version: core-(2.0)
Severity: normal
URL: https://en.wikisource.org/w/index.php?title=Wikisource:Scriptorium&oldid=5021064
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=69980
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Port patrol.py to core | pywikibot/core | master | +529 -1 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Invalid | None | T72936 Important tasks to be solved (tracking) | |||
| Open | Feature | None | T57880 Functionality existing in compat but missing from core (tracking) | ||
| Resolved | droid | T74206 Port patrol.py to core | |||
| Resolved | Mpaa | T85727 Fetching the patrol token for earlier versions | |||
| Resolved | jayvdb | T85725 token methods should use the list of tokens in paraminfo |
IMO this script should be ported using mwlib, and then as a separate task (T71980) we investigate replacing mwlib with mwparserfromhell.
mwlib or patrol.py ?
If you mean mwlib, @XZise has already provided an assessment of its 'need' on T71980. In short, it isnt needed, but it also isnt very nice to require @droid to change the libraries as part of her OPW project - her goal is to remove reasons that users cant switch to core. Someone else can switch libraries before or after the port, if they believe it should be done. FWIW, I believe it should be done, and may be necessary for it to be done if tool-labs wont install mwlib soon, as Wikisource wants to use this on tool labs as soon as possible.
Well at least in core I'd be able to assist in doing both at once. The amount of mwlib in patrol.py is not that high that it isn't possible to convert the file and upload it to review including the mwlib usage. I could then provide assistance in removing it in her patch and we are good to go. Or someone with knowledge about compat first implements it there (or I provide a very basic patch which need to be converted from core-like to compat-compatible).
See the URL in the task description. The compat version was written by me for English Wikisource. It was also used on English Wikipedia, but is no longer needed there. @Billinghurst could provide more information about the current need for this on Wikisource, as I am not active on Wikisource atm.
@XZise, it would be simpler to just use the API , maybe with some textlib but I dont think that is necessary. https://en.wikisource.org/wiki/User:JVbot/patrol_whitelist is a pretty simple format, except for the links. mwlib was used because pywikipedia couldnt parse those links properly, and it wouldnt surprise me if we find some bugs still exist in pywikibot wrt the links used in that page. I also wouldnt be surprised if mwparserfromhell also had bugs wrt those links - mwparserfromhell is not very mature. Changing the parser to a new parser would need unit tests to ensure it correctly understands the whitelist page features, and doesnt have odd bugs (and in the case of mwparserfromhell, it could core dump which limits ability to report and resolve the problems).
Change 184118 had a related patch set uploaded (by Prianka):
Porting patrol.py to core.