jenkins-bot has submitted this change and it was merged.
Change subject: Simplify logic
......................................................................
Simplify logic
Change-Id: I6df49a96da63d193422b906de17ed804046aa89e
---
M scripts/flickrripper.py
1 file changed, 1 insertion(+), 4 deletions(-)
Approvals:
Merlijn van Deen: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/flickrripper.py b/scripts/flickrripper.py
index 5d721ee..9b0c60d 100644
--- a/scripts/flickrripper.py
+++ b/scripts/flickrripper.py
@@ -392,10 +392,7 @@
""" Loop over a set of Flickr photos. """
result = []
retry = False
- if not start_id:
- found_start_id = True
- else:
- found_start_id = False
+ found_start_id = not start_id
#
http://www.flickr.com/services/api/flickr.groups.pools.getPhotos.html
# Get the photos in a group
--
To view, visit
https://gerrit.wikimedia.org/r/86628
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6df49a96da63d193422b906de17ed804046aa89e
Gerrit-PatchSet: 2
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Legoktm <legoktm.wikipedia(a)gmail.com>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: jenkins-bot