Wikipedia:Bots/Requests for approval/SD5bot 2
- 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: SD5 (talk · contribs · SUL · edit count · logs · page moves · block log · rights log · ANI search)
Time filed: 16:57, Sunday April 1, 2012 (UTC)
Automatic, Supervised, or Manual: Automatic
Programming language(s): Python (pywikipedia)
Source code available: Upon request
Function overview: Per the bot request and associated discussion - will replace {{ndash}} with {{spaced ndash}}.
Links to relevant discussions (where appropriate): Request - Discussion
Edit period(s): One time run
Estimated number of pages affected: ~15300
Exclusion compliant (Y/N): Y
Already has a bot flag (Y/N): N (currently have another request open that is in trial though, see here)
Function details: Will replace the template {{ndash}} with {{spaced ndash}}, per the above discussion. List.
Discussion
[edit]- Hi! You said in your bot request that the source code would be available upon request. Is the script you plan to use not the typical find and replace one that comes as standard with Pywikipedia? The Helpful One 20:10, 1 April 2012 (UTC)[reply]
- Hello Helpful One! It's a replacement script that I first wrote as a "test" quite a while ago when I first came across pywikipedia. I've since changed it quite a bit. Only main functional difference from replace.py I think is that the replacement regex(es) and options such as cat or list scanning etc are inside the script. Like a "lite" replace.py. I'd be fine with posting it somewhere if needed. SD5 20:19, 1 April 2012 (UTC)[reply]
Technical Approved for trial (30 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. Please use a descriptive edit summary, since this change might be unexpected. I had to look through the discussions to see why this wasn't a WP:NOTBROKEN case. — HELLKNOWZ ▎TALK 20:29, 1 April 2012 (UTC)[reply]
- Trial complete. - SD5 21:13, 1 April 2012 (UTC)[reply]
Because of https://en.wikipedia.org/w/index.php?title=Talk:Constructed_language&diff=prev&oldid=485051850#.22secret_language.22_and_.22political_language.22 and https://en.wikipedia.org/w/index.php?title=Talk:Agatha_Christie&diff=prev&oldid=485051493 I suggest that the intention of these uses was an {{ndash}} rather than a {{spaced ndash}}, so don't replace " {{ndash}}" or " {{ndash}}" (not that you'll ever come across that one). Josh Parris 23:14, 1 April 2012 (UTC)[reply]
- In fact, seeing https://en.wikipedia.org/w/index.php?title=Dual_wield&diff=prev&oldid=485051841 and https://en.wikipedia.org/w/index.php?title=Bermuda&diff=prev&oldid=485052157#cite_note-3 makes me think that "{{ndash}} " ought to be replaced with {{spaced ndash}}. Josh Parris 23:26, 1 April 2012 (UTC)[reply]
- {{BAGAssistanceNeeded}} I changed the regex so that:
- "Lorem{{ndash}}Ipsum" would be replaced with "Lorem{{spaced ndash}}Ipsum"
- "Test{{ndash}} this" would be replaced with "Test{{spaced ndash}}this"
- Everything else would be left alone. SD5 00:38, 2 April 2012 (UTC)[reply]
- Let's have another Approved for trial (20 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. Josh Parris 00:43, 2 April 2012 (UTC)[reply]
- Trial complete. - SD5 00:51, 2 April 2012 (UTC)[reply]
- Let's have another Approved for trial (20 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. Josh Parris 00:43, 2 April 2012 (UTC)[reply]
- {{BAGAssistanceNeeded}} I changed the regex so that:
Once you're finished you're going to have to tell the requestor/whoever changes the template (you?) that there are {{ndash}}es left that are meant to be ndashes.
Looks good Approved. Josh Parris 01:05, 2 April 2012 (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.