Wikipedia:Bots/Requests for approval/RjwilmsiBot 7

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.

RjwilmsiBot 7

Operator: Rjwilmsi (talk · contribs)

Time filed: 17:53, Wednesday May 11, 2011 (UTC)

Automatic or Manual: Automatic unsupervised

Programming language(s): AWB, Custom module

Source code available: AWB

Function overview: Correct parameter names in citation templates

Links to relevant discussions (where appropriate):

Edit period(s): On release of database dump

Estimated number of pages affected: ~11,000 now, expect to be less on each later database dump

Exclusion compliant (Y/N): Y

Already has a bot flag (Y/N): Y

Function details: The task is to correct parameter names within citation templates, on mainspace. Using a database scan of existing citation use, I've extracted a list of parameter names that don't exist in the citation templates, and created a list of the corresponding correct parameter names. The list is User:Rjwilmsi/BadCiteParams. Templates supported are {{cite web}}, {{cite book}}, {{cite journal}}, {{cite news}} and {{citation}} and their redirects.

The task is to use the list to correct the parameter names using the following logic:

rename |bad name= → |good name= provided that |good name= doesn't already have a value in the citation
- perform a specific check per-template type to ensure |good name= is valid for it
- if the value of |bad name= contains the value of |good name= then remove |bad name= as it's redundant
include renaming of |bad name= → |good name= even if |bad name= has no value
skip the edit if the only changes are to remove an unused |bad name=, or rename a |bad name= with no value – this avoids making non-visible changes
if |bad name= and |good name= are present with different values, don't change (I'll consider fixing manually later)

Edge cases: there are a very small number of edge cases, where for example |published= can mean |publisher= or |date=. Those that I've noticed will be excluded from the bot task and processed manually; any others found can be treated similarly.

Discussion

Approved for trial (50 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. MBisanz ^talk 07:59, 13 May 2011 (UTC)[reply]

Trial complete. Also, as examples of the exception handling Egoless_programming#cite_ref-1 was skipped as |isbn= was already there, so renaming or removing |ibn= wouldn't have been visible, and Waiting_to_Exhale:_Original_Soundtrack_Album#cite_ref-exhaleriaa_34-2 was skipped as renaming |authro= to |author= when the parameter had no value wouldn't have been visible either. Rjwilmsi 22:05, 13 May 2011 (UTC)[reply]

Contribs approximately here in contribution history. Rjwilmsi 15:03, 14 May 2011 (UTC)[reply]

Approved. MBisanz ^talk 02:16, 16 May 2011 (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.