jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/575279 )
Change subject: the id of the photo as last resort
......................................................................
the id of the photo as last resort
Change-Id: I9b0240eded9fc3c5a7932e9bc5eb24447f052af1
---
M scripts/flickrripper.py
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/flickrripper.py b/scripts/flickrripper.py
index 904525d..dc9c058 100755
--- a/scripts/flickrripper.py
+++ b/scripts/flickrripper.py
@@ -187,8 +187,8 @@
description = description[:items]
title = cleanUpTitle(description)
else:
- title = ''
- # Should probably have the id of the photo as last resort.
+ # Use the id of the photo as last resort.
+ title = photoInfo.find('photo').attrib['id']
if pywikibot.Page(site, 'File:{} - {} - {}.jpg'
.format(title, project, username)).exists():
--
To view, visit
https://gerrit.wikimedia.org/r/575279
To unsubscribe, or for help writing mail filters, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I9b0240eded9fc3c5a7932e9bc5eb24447f052af1
Gerrit-Change-Number: 575279
Gerrit-PatchSet: 3
Gerrit-Owner: Shizhao <shizhao(a)gmail.com>
Gerrit-Reviewer: D3r1ck01 <xsavitar.wiki(a)aol.com>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot (75)