jenkins-bot has submitted this change and it was merged.

Change subject: [bugfix] fix syntax error for py 2.6 ......................................................................

[bugfix] fix syntax error for py 2.6

- {} cant't be used to create a set in py 2.6 - assign expression to a variable "refs" and reuse it - Pepify some doc strings

Bug: T57138 Change-Id: Ibb3ecd1720d8d0172a68dff0441b36c3d8459947 --- M harvest_template.py 1 file changed, 10 insertions(+), 11 deletions(-)

Approvals: John Vandenberg: Looks good to me, approved jenkins-bot: Verified

diff --git a/harvest_template.py b/harvest_template.py index 8cb29d4..b372caf 100644 --- a/harvest_template.py +++ b/harvest_template.py @@ -14,8 +14,8 @@

""" # -# (C) 2013 Multichill, Amir -# (C) 2013 Pywikibot team +# (C) Multichill, Amir, 2013 +# (C) Pywikibot team, 2013-2016 # # Distributed under the terms of MIT License. # @@ -28,11 +28,12 @@

class HarvestRobot: - """ - A bot to add Wikidata claims - """ + """A bot to add Wikidata claims.""" + def __init__(self, generator, templateTitle, fields): """ + Constructor. + Arguments: * generator - A generator that yields Page objects. * templateTitle - The template to work on @@ -84,9 +85,7 @@ self.procesPage(page)

def procesPage(self, page): - """ - Proces a single page - """ + """Process a single page/item.""" item = pywikibot.DataPage(page) pywikibot.output('Processing %s' % page) if not item.exists(): @@ -127,12 +126,12 @@ pywikibot.output('Adding %s --> %s' % (claim, linkedItem.getID())) - if self.setSource(self.site().language()): + refs = self.setSource(self.site().language()) + if refs: item.editclaim( str(claim), linkedItem.getID(), - refs={self.setSource( - self.site().language())}) + refs=set(refs)) else: item.editclaim(str(claim), linkedItem.getID())