From c54342bb9812a17f8331ae0251e2fd5951b7b2a8 Mon Sep 17 00:00:00 2001 From: Lars Lindqvist Date: Tue, 8 Jan 2019 19:38:46 +0100 Subject: [PATCH] Refactor `get_filename` Use string formatting instead of `str(0) + str(index)` and don't assume file extension is three characters. --- instaloader/instaloader.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/instaloader/instaloader.py b/instaloader/instaloader.py index ad88749..2a3e22a 100644 --- a/instaloader/instaloader.py +++ b/instaloader/instaloader.py @@ -255,8 +255,7 @@ class Instaloader: return None else: def get_filename(index): - return filename if index == 0 else (filename[:-4] + '_old_' + - (str(0) if index < 10 else str()) + str(index) + filename[-4:]) + return filename if index == 0 else '{0}_old_{2:02}{1}'.format(*os.path.splitext(filename), index) i = 0 while os.path.isfile(get_filename(i)):