Wikipedia:Bots/Requests for approval/Legobot 13


 * 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 Symbol keep vote.svg Approved

Legobot 13
Operator:

Time filed: 15:32, Tuesday July 3, 2012 (UTC)

Automatic, Supervised, or Manual: Automatic unsupervised

Programming language(s): Pywikipedia rewrite branch

Source code available: here

Function overview: Creates a list of incorrectly moved pages for WP:AFC

Links to relevant discussions (where appropriate): request on my talk page

Edit period(s): Hourly

Estimated number of pages affected: 1

Exclusion compliant (Yes/No): Yes

Already has a bot flag (Yes/No): Yes

Function details:
 * Grabs the latest 100 page moves
 * Checks if page move matches the set of rules
 * new page matches 'Articles for creation/*'
 * old page matches 'Wikipedia talk:Articles for creation/' and new page matches 'Wikipedia talk:'
 * old page matches 'Wikipedia:Articles for creation/' and new page matches 'Wikipedia:'


 * If any moves matched those rules, update Articles for creation/Wrongly moved submissions

Discussion
It looks good. I have just a couple of thoughts: Is this correct, or am I missing something? Perhaps you can set a higher lelimit and set lestart to the last time the bot was run. —  The   Earwig   (talk) 18:31, 3 July 2012 (UTC)
 * If there are fewer than 100 moves in the past hour, the bot might add the same move to the list twice.
 * If there are more than 100 moves in the past hour, the bot might skip an incorrect move.
 * That jumped out at me as well reading through the request. I don't know anything about bot request so I don't know if that's usual. Why can't it check every page moves since last ran up to a set time length? KTC (talk) 20:16, 3 July 2012 (UTC)
 * I agree with KTC above, why can't it check every page move? --Nathan2055talk - contribs 03:49, 8 July 2012 (UTC)
 * Thats a great idea. I've updated the code so that it will check every move of the past hour and then log based on those. LegoKontribsTalkM 10:07, 10 July 2012 (UTC)

BAG assistance needed - No action taken on the request by the BAG for almost a week now. --Nathan2055talk - contribs 17:29, 10 July 2012 (UTC)

Kind of a "duh" task, but let's go for a just to make sure. —  The   Earwig   (talk) 20:13, 10 July 2012 (UTC)
 * - The bot hasn't turned on and begun processing the list yet. --Nathan2055talk - contribs 17:08, 11 July 2012 (UTC)
 * Thanks for pointing that out, the bot wasn't logged in. I've fixed that now. LegoKontribsTalkM 17:22, 11 July 2012 (UTC)

(for a while now, sorry!). As the timestamps show, the first 3 edits were manually run by me since my cron jobs were having issues on the toolserver. Since then I've switched it over to my local machine which is working fine (the last edit). LegoKontribsTalkM 20:32, 22 July 2012 (UTC)


 * I don't think it should be tagging moves like this one. The latter two rules should only match if the new title does not match what you tested the old title against. I did a bit of a code review; see that here. —  Earwig   talk 21:43, 22 July 2012 (UTC)
 * Good point. And thanks for the review, I've merged in your pull request. LegoKontribsTalkM 22:24, 22 July 2012 (UTC)
 * BAGAssistanceNeeded Can I get an approval now? I've left the bot running right now (so nothing is missed) and it's doing fine . Thanks, LegoKontribsTalkM 20:58, 28 July 2012 (UTC)

Sure; nice work. —  Earwig   talk 21:03, 28 July 2012 (UTC)
 * 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.