fixed expiry not specified

chris-ch · Oct 22, 2019 · 13637af · 13637af
1 parent 980df50
commit 13637af
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/setup.py b/setup.py
@@ -1,6 +1,6 @@
 from setuptools import setup
 
-__version = '0.5.3'
+__version = '0.5.4'
 
 INSTALL_REQUIRE = ['requests>=2.20.0']
 

diff --git a/src/webscrapetools/keyvalue.py b/src/webscrapetools/keyvalue.py
@@ -79,7 +79,8 @@ def invalidate_expired_entries(as_of_date: datetime=None) -> None:
     :param as_of_date: fake current date (for dev only)
     :return:
     """
-    if not _get_expiry():
+    expiry_periods, expiry_unit = _get_expiry()
+    if not expiry_periods:
         return
 
     index_name = _fileindex_name()
@@ -90,7 +91,6 @@ def invalidate_expired_entries(as_of_date: datetime=None) -> None:
     if as_of_date is None:
         as_of_date = datetime.today()
 
-    expiry_periods, expiry_unit = _get_expiry()
     if expiry_unit == 'day':
         expiry_date = as_of_date - timedelta(days=expiry_periods)