Xqt has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/109622


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(-)


  git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core 
refs/changes/22/109622/1

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: newchange
Gerrit-Change-Id: I7001045bb6e649719404f6c6193c6b33af39e88b
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Xqt <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to