Changeset ad5df3a in rattail

Timestamp:

08/01/2022 09:12:25 PM (6 months ago)

Author:

Lance Edgar <lance@…>

Branches:

master

Children:

d006a6d

Parents:

0ce1f4a

Message:

Remove "basic db cache" for config from DB settings table

turns out that won't even work right

refs #8

File:

• rattail/config.py (modified) (3 diffs)

rattail/config.py

@@ -143 +143 @@
         self.versioning_has_been_enabled = False
 
-        # implement a basic db caching mechanism, if configured
-        self.basic_db_cache = self.getbool('rattail.config',
-                                           'basic_db_cache',
-                                           default=False,
-                                           usedb=False)
-        if self.basic_db_cache:
-            self.cache_expiry_seconds = 30
-            self.cached_settings = {}
-
     def read_file(self, path, recurse=True, require=False):
         """
@@ -308 +299 @@
         now = app.localtime(tzinfo=False)
 
-        # first try to return value from simple cache
-        if self.basic_db_cache:
-            cached = self.cached_settings.get(name)
-            if cached:
-                time, value = cached
-                if (now - time).total_seconds() <= self.cache_expiry_seconds:
-                    return value
-
         close = False
         if not session:
@@ -326 +309 @@
         if close:
             session.close()
-
-        # add value to simple cache
-        if self.basic_db_cache:
-            self.cached_settings[name] = (now, value)
 
         return value