jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/430929 )
Change subject: [IMPR] use str.format method usability
......................................................................
[IMPR] use str.format method usability
use str.format instead of concating indexed variables
Change-Id: Ic1c053da3e6c0a53c663ba4d4e8bf6284789f469
---
M scripts/surnames_redirects.py
1 file changed, 6 insertions(+), 10 deletions(-)
Approvals:
Zhuyifei1999: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/surnames_redirects.py b/scripts/surnames_redirects.py
index ab19916..5fadd38 100755
--- a/scripts/surnames_redirects.py
+++ b/scripts/surnames_redirects.py
@@ -16,7 +16,7 @@
Example: "python pwb.py surnames_redirects -start:B"
"""
#
-# (C) Pywikibot team, 2017
+# (C) Pywikibot team, 2017-2018
#
# Distributed under the terms of the MIT license.
#
@@ -64,26 +64,22 @@
if self.getOption('surnames_last'):
name_parts = name.split(', ')
if len(name_parts) == 2 and len(name.split(' ')) <= 3:
- possible_names.append(name_parts[1] + ' ' + name_parts[0])
+ possible_names.append('{1} {0}'.format(*name_parts))
else:
words = name.split()
if len(words) == 2 and name == name.title():
- possible_names.append(words[1] + ', ' + words[0])
+ possible_names.append('{1}, {0}'.format(*words))
elif len(words) == 3:
# title may have at most one non-titlecased word
if len(SequenceMatcher(None, name,
name.title()).get_matching_blocks()) <= 3:
- possible_names.append(words[1] + ' ' +
- words[2] + ', ' +
- words[0])
- possible_names.append(words[2] + ', ' +
- words[0] + ' ' +
- words[1])
+ possible_names.append('{1} {2}, {0}'.format(*words))
+ possible_names.append('{2}, {0} {1}'.format(*words))
for possible_name in possible_names:
# append disambiguation inside parenthesis if there is one
if len(split_title) == 2:
- possible_name += ' (' + split_title[1]
+ possible_name += ' ({1}'.format(*split_title)
new_page = pywikibot.Page(site, possible_name)
if new_page.exists():
--
To view, visit
https://gerrit.wikimedia.org/r/430929
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic1c053da3e6c0a53c663ba4d4e8bf6284789f469
Gerrit-Change-Number: 430929
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Zhuyifei1999 <zhuyifei1999(a)gmail.com>
Gerrit-Reviewer: Zoranzoki21 <zorandori4444(a)gmail.com>
Gerrit-Reviewer: jenkins-bot <>