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)