Page MenuHomePhabricator

protect.py only unprotects pages because it cannot properly determine which protection types are valid
Closed, ResolvedPublic

Description

Line 105 of protect.py should have self.protections.items() instead of self.protections, or else protection arguments are not passed to api call.

Event Timeline

Sn1per raised the priority of this task from to High.
Sn1per updated the task description. (Show Details)
Sn1per added a project: Pywikibot.
Sn1per subscribed.
Restricted Application added subscribers: Aklapper, Unknown Object (MLST). · View Herald TranscriptJan 18 2015, 2:29 AM
Sn1per renamed this task from protect.py doesn't work to protect.py only unprotects pages and cannot properly determine which protection types are valid.Jan 18 2015, 2:33 AM
Sn1per renamed this task from protect.py only unprotects pages and cannot properly determine which protection types are valid to protect.py only unprotects pages because it cannot properly determine which protection types are valid.
Sn1per claimed this task.
Sn1per set Security to None.

:P

gerritbot subscribed.

Change 185661 had a related patch set uploaded (by Sn1per):
Properly determine the valid protection types

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

Patch-For-Review

Change 185661 merged by jenkins-bot:
Properly determine the valid protection types

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