Wikipedia:Bots/Requests for approval/Tom.Bot 3
- 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: Tom.Reding (talk · contribs · SUL · edit count · logs · page moves · block log · rights log · ANI search)
Time filed: 20:50, Wednesday, February 28, 2018 (UTC)
Automatic, Supervised, or Manual: automatic
Programming language(s): C# & AWB
Source code available: User:Tom.Bot/Task3 code
Function overview: Add {{Taxonbar|from=Q#}}
to pages with taxonomic infoboxes
Links to relevant discussions (where appropriate): WT:TREE#Taxonbar addition requirements, Template talk:Taxonbar#Specifying the from/from1 parameter, Taxonbar/doc
Edit period(s): One-time bulk run, with smaller followup runs as needed
Estimated number of pages affected: ~190,000 50,000~100,000 based on preliminary stats
Exclusion compliant (Yes/No): Yes
Already has a bot flag (Yes/No): Yes
Function details: Add {{Taxonbar|from=Q#}}
, where Q#
is the Wikidata item, to pages that:
- transclude either {{Automatic taxobox}}, {{Taxobox}}, {{Speciesbox}}, {{Subspeciesbox}}, or {{Infraspeciesbox}}, and
- contain at least 1 taxonomic ID listed here
or here, but excluding TPL, EOL, & GBIF taxonomic IDs from the count, per WT:TREE#Taxonbar addition requirements, and - don't already contain a {{Taxonbar}}.
Cosmetic changes piggybacked IIF {{Taxonbar}} is added:
- {{Reflist}}, {{Commons}}, {{Wikispecies}}, {{FishBase}}, {{Portal}}, {{Portal bar}}, {{Authority control}}, and {{DEFAULTSORT}} are all standardized as shown
- 1 blank line is added before {{Taxonbar}} when appropriate
- if DNE, a blank line is added between the end of the infobox & the first line of body-text
- capitalize
|status_system=IUCN<2.3|3.1>
- remove
|image_size=
& variants, as the parameter DNE on any affected infobox - remove
|image_width=<null>
& aliases, as|image_upright=
is available & preferred on all affected infoboxes - {{Italic title}} is moved from !top of the page to the top of the page and standardized as shown
- Remove {{Clear}}s below cats, as {{Taxonbar}} effectively self-clears via
listCell:css('width', '100%')
in Module:Navbox - WP:GenFixes on
Known issue:
- When a navbar/navbox is misplaced on the page, i.e. below categories, WP:GenFixes will place it below the {{Taxonbar}}. The correct placement of {{Taxonbar}} is below all navbars (and above {{Authority control}}). These minority instances (~3.5%) will be skipped until the issue is resolved.
Fixed (diff). If, after GenFixes, any template other than {{DEFAULTSORT}}, {{Authority control}}, {{Clear}}, {{Use DMY dates}}/etc., {{Italic title}}, {{Good article}}, or {{Featured article}} appear after {{Taxonbar}}, then the page is skipped, for manual review. The diff represents the vast majority (~95%) of error cases, so the estimated skip rate will be < 1%, if not ≪ 1%.
Discussion
[edit] Approved for trial (50 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. Don't really see any holdup for this. I'll have some questions, but I need to see the bot in action first. Headbomb {t · c · p · b} 22:10, 9 March 2018 (UTC)[reply]
Trial complete. Edits ~ Tom.Reding (talk ⋅dgaf) 22:44, 9 March 2018 (UTC)[reply]
Known issue (minor) added to description. ~ Tom.Reding (talk ⋅dgaf) 16:01, 12 March 2018 (UTC)[reply]
Fixed. ~ Tom.Reding (talk ⋅dgaf) 21:47, 12 March 2018 (UTC)[reply]
Approved for extended trial (25 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. to test the updated logic. Headbomb {t · c · p · b} 03:58, 17 March 2018 (UTC)[reply]
Trial complete. Edits. The first (newest) 6 were chosen for removal of {{Clear}}. The next oldest 6 were chosen for moving {{Italic title}} to top. The last (oldest) 13 were chosen for the updated logic (always placing {{Taxonbar}} below even-misplaced navbars). ~ Tom.Reding (talk ⋅dgaf) 17:52, 17 March 2018 (UTC)[reply]
@Tom.Reding: In edits like [1], italics title should be on its own line. If that's done, I'm fine with approving. However, I feel it would be best to remove both italics title and |name=
from the infobox (see Template:Taxobox#Italic page titles), so that the italicization is done automatically via the infobox, e.g. [2]. However, that's my personal opinion, and I'd like the project's feedback on this before mandating this. Headbomb {t · c · p · b} 18:05, 17 March 2018 (UTC)[reply]
- Ehhh, I didn't think I'd be getting into the weeds with moving {{Italic title}}. That's a highly conditional effect apparently, and I'd best leave it alone & just let GenFixes do what it wants with it. Will strike from the description. ~ Tom.Reding (talk ⋅dgaf) 18:20, 17 March 2018 (UTC)[reply]
- @Tom.Reding: You could also just add a find
\{\{italic title\}\}\{\{
replace{{italic title}}\n{{
and that would take care of that. I'm also entirely fine not messing with the auto-italicization from the infobox as part of this bot run though. Headbomb {t · c · p · b} 18:27, 17 March 2018 (UTC)[reply]- OK,
fixed! ~ Tom.Reding (talk ⋅dgaf) 18:34, 17 March 2018 (UTC)[reply]
Approved for extended trial (10 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. to check for the italic title thing. Headbomb {t · c · p · b} 19:14, 17 March 2018 (UTC)[reply]
Trial complete. Edits. The first (newest) 5 show {{Italic title}} moves, and the other 5 show in-place corrections. This edit represents a known issue in rdcheck's regex, and has been corrected per the workaround (I decided to re-capture the template #R regexes today to grab any new #Rs that might have shown up since I first started coding this, and accidentally pasta'd over the workaround
(?=\s*[\|\}])
). ~ Tom.Reding (talk ⋅dgaf) 20:27, 17 March 2018 (UTC)[reply]
- OK,
- @Tom.Reding: You could also just add a find
Approved. Headbomb {t · c · p · b} 20:37, 17 March 2018 (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.