jenkins-bot has submitted this change and it was merged.
Change subject: Bug 55118: use the family name and site code to get sources
......................................................................
Bug 55118: use the family name and site code to get sources
instead of the language code only
This prevents the script from importing a source for xxwiki
for data imported from xxwikivoyage or xxwikisource (bug 62037)
Change-Id: I29df011b82fd8687d4c38def62d0c0ea3a1b7994
---
M scripts/harvest_template.py
1 file changed, 11 insertions(+), 11 deletions(-)
Approvals:
Merlijn van Deen: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/harvest_template.py b/scripts/harvest_template.py
index 2507544..79adb7c 100755
--- a/scripts/harvest_template.py
+++ b/scripts/harvest_template.py
@@ -50,14 +50,14 @@
self.repo = pywikibot.Site().data_repository()
self.cacheSources()
- def getSource(self, lang):
+ def getSource(self, site):
"""
- Get the source for the specified language,
+ Get the source for the specified site,
if possible
"""
- if lang in self.source_values:
- source = pywikibot.Claim(self.repo, 'p143')
- source.setTarget(self.source_values.get(lang))
+ if site.family.name in self.source_values and site.code in
self.source_values[site.family.name]:
+ source = pywikibot.Claim(self.repo, 'P143')
+
source.setTarget(self.source_values.get(site.family.name).get(site.code))
return source
def cacheSources(self):
@@ -65,12 +65,12 @@
Fetches the sources from the onwiki list
and stores it internally
"""
- page = pywikibot.Page(self.repo, u'Wikidata:List of wikis/python')
+ page = pywikibot.Page(self.repo, u'List of wikis/python', ns=4)
self.source_values = json.loads(page.get())
- self.source_values = self.source_values['wikipedia']
- for source_lang in self.source_values:
- self.source_values[source_lang] = pywikibot.ItemPage(self.repo,
-
self.source_values[source_lang])
+ for family_code, family in self.source_values.iteritems():
+ for source_lang in family:
+ self.source_values[family_code][source_lang] =
pywikibot.ItemPage(self.repo,
+
family[source_lang])
def run(self):
"""
@@ -152,7 +152,7 @@
pywikibot.output('Adding %s --> %s' %
(claim.getID(), claim.getTarget()))
item.addClaim(claim)
# A generator might yield pages from multiple
sites
- source = self.getSource(page.site.language())
+ source = self.getSource(page.site)
if source:
claim.addSource(source, bot=True)
--
To view, visit https://gerrit.wikimedia.org/r/118974
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I29df011b82fd8687d4c38def62d0c0ea3a1b7994
Gerrit-PatchSet: 2
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Ricordisamoa <[email protected]>
Gerrit-Reviewer: Ladsgroup <[email protected]>
Gerrit-Reviewer: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: Xqt <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
Pywikibot-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikibot-commits