jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/538724 )
Change subject: Make sure the user is logged in before attempting a delete
......................................................................
Make sure the user is logged in before attempting a delete
When BotPasswords is used, even if you have already run the login.py
script, you will still end up logged out when you run the delete.py
script later. This patch provides a workaround so that delete.py remains
usable while we revamp the BotPasswords related processes to make the
login session info persistant.
Bug: T233608
Change-Id: Ib3bc9005ab58ff19ce14acf9df4127a7d32eb746
---
M scripts/delete.py
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/delete.py b/scripts/delete.py
index 18cbec7..b291e5a 100755
--- a/scripts/delete.py
+++ b/scripts/delete.py
@@ -214,6 +214,8 @@
self.current_page, ns_names))
return # Not an orphan, do not delete.
+ if self.site.user() is None:
+ self.site.login()
self.current_page.delete(self.summary,
not self.getOption('always'),
self.getOption('always'),
--
To view, visit
https://gerrit.wikimedia.org/r/538724
To unsubscribe, or for help writing mail filters, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib3bc9005ab58ff19ce14acf9df4127a7d32eb746
Gerrit-Change-Number: 538724
Gerrit-PatchSet: 5
Gerrit-Owner: Huji <huji.huji(a)gmail.com>
Gerrit-Reviewer: Huji <huji.huji(a)gmail.com>
Gerrit-Reviewer: Revi <wiki(a)ymhong.com>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot (75)