jenkins-bot merged this change.

View Change

Approvals: Dvorapa: Looks good to me, approved jenkins-bot: Verified
[IMPR] Improvements for askForHints (4)

- combine checkings

Change-Id: If8b9cc468c53b7215124c84eca80f19d43d2b124
---
M scripts/interwiki.py
1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/scripts/interwiki.py b/scripts/interwiki.py
index e9c6264..d4e5e76 100755
--- a/scripts/interwiki.py
+++ b/scripts/interwiki.py
@@ -1167,16 +1167,13 @@

def askForHints(self, counter):
"""Ask for hints to other sites."""
- if not self.workonme: # we don't work on it anyway
- return
-
- if not (
- (self.untranslated or self.conf.askhints)
- and not self.hintsAsked
- and self.originPage
- and self.originPage.exists()
- and not self.originPage.isRedirectPage()
- and not self.originPage.isCategoryRedirect()):
+ if (not self.workonme # we don't work on it anyway
+ or not self.untranslated and not self.conf.askhints
+ or self.hintsAsked
+ or not self.originPage
+ or not self.originPage.exists()
+ or self.originPage.isRedirectPage()
+ or self.originPage.isCategoryRedirect()):
return

self.hintsAsked = True
@@ -1189,8 +1186,10 @@

while True:
newhint = pywikibot.input('Give a hint (? to see pagetext):')
+
if not newhint:
break
+
if newhint == '?':
t += self.conf.showtextlinkadd
pywikibot.output(self.originPage.get()[:t])

To view, visit change 512335. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: If8b9cc468c53b7215124c84eca80f19d43d2b124
Gerrit-Change-Number: 512335
Gerrit-PatchSet: 8
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Dvorapa <dvorapa@seznam.cz>
Gerrit-Reviewer: John Vandenberg <jayvdb@gmail.com>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot (75)
Gerrit-CC: D3r1ck01 <xsavitar.wiki@aol.com>