Wikipedia:Bots/Requests for approval/EarwigBot 20
- The following discussion is an archived debate. Please do not modify it. To request review of this BRFA, please start a new section at WT:BRFA. The result of the discussion was
Approved.
Operator: The Earwig (talk · contribs · SUL · edit count · logs · page moves · block log · rights log · ANI search)
Time filed: 04:43, Tuesday, December 1, 2015 (UTC)
Automatic, Supervised, or Manual: Supervised
Programming language(s): Python with earwigbot
Source code available: infobox_station.py
Function overview: Replace {{Infobox China station}} and {{Infobox Japan station}} with {{Infobox station}}
Links to relevant discussions (where appropriate): China TfD, Japan TfD, previous BRFA
Edit period(s): One-time run
Estimated number of pages affected: ~7000 ([1][2])
Exclusion compliant (Yes/No): Yes
Already has a bot flag (Yes/No): Yes
Function details: Fairly straightforward, I hope. Most of the logic is handled by the sandboxes, and has been discussed in the previous BRFA; the bot just replaces the template invocation with its sandbox ({{Infobox China station/sandbox}} and {{Infobox Japan station/sandbox}}) and substitutes it. There is some additional logic to inserting categories which were previously automatically added by the template using two helpers (1 and 2).
Discussion
editApproved for trial (50 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. — we've been over this fairly extensively, to the point where it's time to just do a few and ensure everything turns out okay. :P --slakr\ talk / 03:56, 2 December 2015 (UTC)[reply]
Trial complete. Edits. Bit of an annoying cat issue going on; adds a category twice in at least one case.
Don't think I can fix this using the current implementation.I can, of course, but it will take a little bit of work. That other category shouldn't be there in the first place. I also fixed the extra newline issue seen in a few edits. — Earwig talk 11:39, 5 December 2015 (UTC)[reply]- At a glance, it seems you possibly need to just check if the category was already on the page before actually adding it in _add_cats (unless I'm missing something here). Hazard SJ 20:57, 5 December 2015 (UTC)[reply]
- _add_cats has no knowledge of the categories it's actually adding (that's handled by Template:Infobox China station/sandbox/cats), hence the trouble. — Earwig talk 20:58, 5 December 2015 (UTC)[reply]
- Fixed. A bit dirty (using action=parse), but it should work. — Earwig talk 11:42, 6 December 2015 (UTC)[reply]
- _add_cats has no knowledge of the categories it's actually adding (that's handled by Template:Infobox China station/sandbox/cats), hence the trouble. — Earwig talk 20:58, 5 December 2015 (UTC)[reply]
- At a glance, it seems you possibly need to just check if the category was already on the page before actually adding it in _add_cats (unless I'm missing something here). Hazard SJ 20:57, 5 December 2015 (UTC)[reply]
Approved. If there are any further issues they can be easily dealt. Magioladitis (talk) 12:43, 8 December 2015 (UTC)[reply]
- The above discussion is preserved as an archive of the debate. Please do not modify it. To request review of this BRFA, please start a new section at WT:BRFA.