Alex S.H. Lin has submitted this change and it was merged.
Change subject: [BUGFIX] Fix for ProcessTitle() and ReplaceLink(), pep8 changes
......................................................................
[BUGFIX] Fix for ProcessTitle() and ReplaceLink(), pep8 changes
- mightBeLoc must be migtBeLcl in ProcessTitle()
- fix for list comprehensive variable d --> s
- i18n.twtranslate may have a third parameter for the dictionary
- pep8 changes (indentation fix)
Change-Id: I7001045bb6e649719404f6c6193c6b33af39e88b
---
M scripts/casechecker.py
1 file changed, 10 insertions(+), 10 deletions(-)
Approvals:
Alex S.H. Lin: Verified; Looks good to me, approved
diff --git a/scripts/casechecker.py b/scripts/casechecker.py
index 98532bd..495575c 100644
--- a/scripts/casechecker.py
+++ b/scripts/casechecker.py
@@ -4,7 +4,7 @@
with mixed latin and cyrilic alphabets.
"""
#
-# (C) Pywikibot team, 2006-2013
+# (C) Pywikibot team, 2006-2014
#
# Distributed under the terms of the MIT license.
#
@@ -284,8 +284,8 @@
% len(self.knownWords))
if len(self.knownWords) > 0:
pywikibot.log(u'Whitelist: %s'
- % u', '.join([self.MakeLink(i, False)
- for i in self.knownWords]))
+ % u', '.join([self.MakeLink(i, False)
+ for i in self.knownWords]))
else:
pywikibot.output(u'Whitelist is not known for language %s'
% self.site.lang)
@@ -380,8 +380,8 @@
## % (title, newTitle),
## ['Yes',
'No'],
## ['y', 'n'])
- editSummary = i18n.twtranslate(self.site,
-
"casechecker-rename")
+ editSummary = i18n.twtranslate(
+ self.site, "casechecker-rename")
dst = self.Page(newTitle)
if 'redirect' in page:
@@ -549,7 +549,7 @@
if badWord in self.alwaysInLocal:
mightBeLat = False
elif badWord in self.alwaysInLatin:
- mightBeLoc = False
+ mightBeLcl = False
if mightBeLcl:
mapLcl[badWord] = badWord.translate(self.latToLclDict)
@@ -770,7 +770,7 @@
% (i18n.twtranslate(
self.site,
"casechecker-replacement-summary"),
-
self.site.mediawiki_message(u"Comma-separator").join(msg)))
+
self.site.mediawiki_message(u"comma-separator").join(msg)))
return True
except KeyboardInterrupt:
raise
@@ -780,7 +780,8 @@
return False
def MakeMoveSummary(self, fromTitle, toTitle):
- return i18n.twtranslate(self.site, "casechecker-replacement-linklist")
% {'source': fromTitle, 'target': toTitle}
+ return i18n.twtranslate(self.site, "casechecker-replacement-linklist",
+ {'source': fromTitle, 'target':
toTitle})
def MakeLink(self, title, colorcode=True):
prf = u'' if self.Page(title).namespace() == 0 else u':'
@@ -803,7 +804,7 @@
def ReplaceLink(self, text, oldtxt, newtxt):
frmParts = [s.strip(self.stripChars)
- for d in self.wordBreaker.split(oldtxt)]
+ for s in self.wordBreaker.split(oldtxt)]
toParts = [s.strip(self.stripChars)
for s in self.wordBreaker.split(newtxt)]
@@ -817,7 +818,6 @@
toParts[i][0].lower() + toParts[i][1:])
text = text.replace(frmParts[i][0].upper() + frmParts[i][1:],
toParts[i][0].upper() + toParts[i][1:])
-
return text
--
To view, visit
https://gerrit.wikimedia.org/r/109622
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7001045bb6e649719404f6c6193c6b33af39e88b
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Alex S.H. Lin <alexsh(a)mail2000.com.tw>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: Meno25 <meno25mail(a)gmail.com>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: Siebrand <siebrand(a)wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>