Page MenuHomePhabricator

movepages.py: arguments recognition error
Closed, ResolvedPublicBUG REPORT

Description

There is broken the movepages.py script, arguments recognition error. Since the 28 March commit (github).

for arg in local_args:
    opt, _, value = arg.partition(':')
    if opt.startswith('-'):
        continue
    opt = opt[1:]
    if opt == 'pairsfile':

The line #200 opt.startswith('-'): continue ignores all arguments, because they must start with '-'.

Event Timeline

Xqt triaged this task as High priority.

Change 789894 had a related patch set uploaded (by Xqt; author: Xqt):

[pywikibot/core@master] [bugfix] Enable arguments

https://gerrit.wikimedia.org/r/789894

Change 789895 had a related patch set uploaded (by Xqt; author: Xqt):

[pywikibot/core@stable] [bugfix] Enable arguments

https://gerrit.wikimedia.org/r/789895

Change 789894 merged by jenkins-bot:

[pywikibot/core@master] [bugfix] Enable arguments

https://gerrit.wikimedia.org/r/789894

Change 789895 merged by jenkins-bot:

[pywikibot/core@stable] [7.2.1] Fix argument parsing in movepages.py

https://gerrit.wikimedia.org/r/789895

@Vladis13: I backported the patch to the stable release