jenkins-bot merged this change.

View Change

Approvals: D3r1ck01: Looks good to me, but someone else must approve Dalba: Looks good to me, approved jenkins-bot: Verified
[IMPR] Improvements for askForHints (3)

- decrease nested code

Change-Id: I8aea761ed32046fd843e734205479ac1989f84d9
---
M scripts/interwiki.py
1 file changed, 31 insertions(+), 29 deletions(-)

diff --git a/scripts/interwiki.py b/scripts/interwiki.py
index 36bcc7f..e9c6264 100755
--- a/scripts/interwiki.py
+++ b/scripts/interwiki.py
@@ -1170,43 +1170,45 @@
if not self.workonme: # we don't work on it anyway
return

- if ((self.untranslated or self.conf.askhints)
+ 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()):
+ return

- self.hintsAsked = True
- if not self.conf.untranslated:
- return
+ self.hintsAsked = True
+ if not self.conf.untranslated:
+ return

- t = self.conf.showtextlink
- if t:
+ t = self.conf.showtextlink
+ if t:
+ pywikibot.output(self.originPage.get()[:t])
+
+ 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])
-
- 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])
- elif ':' not in newhint:
- pywikibot.output(fill(
- 'Please enter a hint in the format language:pagename '
- 'or type nothing if you do not have a hint.'))
- else:
- links = titletranslate.translate(
- self.originPage,
- hints=[newhint],
- auto=self.conf.auto,
- removebrackets=self.conf.hintnobracket)
- for link in links:
- page = pywikibot.Page(link)
- self.addIfNew(page, counter, None)
- if self.conf.hintsareright:
- self.hintedsites.add(page.site)
+ elif ':' not in newhint:
+ pywikibot.output(fill(
+ 'Please enter a hint in the format language:pagename '
+ 'or type nothing if you do not have a hint.'))
+ else:
+ links = titletranslate.translate(
+ self.originPage,
+ hints=[newhint],
+ auto=self.conf.auto,
+ removebrackets=self.conf.hintnobracket)
+ for link in links:
+ page = pywikibot.Page(link)
+ self.addIfNew(page, counter, None)
+ if self.conf.hintsareright:
+ self.hintedsites.add(page.site)

def batchLoaded(self, counter):
"""

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

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8aea761ed32046fd843e734205479ac1989f84d9
Gerrit-Change-Number: 540854
Gerrit-PatchSet: 2
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: D3r1ck01 <xsavitar.wiki@aol.com>
Gerrit-Reviewer: Dalba <dalba.wiki@gmail.com>
Gerrit-Reviewer: Dvorapa <dvorapa@seznam.cz>
Gerrit-Reviewer: Framawiki <framawiki@tools.wmflabs.org>
Gerrit-Reviewer: jenkins-bot (75)