TaskWarrior: Use config propery instead of get_config call

[etc/taskwarrior.git] / tasklib / backends.py

diff --git a/tasklib/backends.py b/tasklib/backends.py
index 32a8018acb142ddb1257e0fe3916089bde2ecf70..fbe3ba336e53b50f92a08874c0d1a9b9cec5e7c9 100644 (file)
--- a/tasklib/backends.py
+++ b/tasklib/backends.py
@@ -103,6 +103,7 @@ class TaskWarrior(Backend):
         if not os.path.exists(self.taskrc_location):
             self.taskrc_location = '/'
 
         if not os.path.exists(self.taskrc_location):
             self.taskrc_location = '/'
 

+        self._config = None

         self.version = self._get_version()
         self.config = {
             'confirmation': 'no',
         self.version = self._get_version()
         self.config = {
             'confirmation': 'no',


@@ -229,7 +230,13 @@ class TaskWarrior(Backend):
 
     # Public interface
 
 
     # Public interface
 

-    def get_config(self):
+    @property
+    def config(self):
+        # First, check if memoized information is available
+        if self._config:
+            return copy.deepcopy(self._config)
+
+        # If not, fetch the config using the 'show' command

         raw_output = self.execute_command(
             ['show'],
             config_override={'verbose': 'nothing'}
         raw_output = self.execute_command(
             ['show'],
             config_override={'verbose': 'nothing'}


@@ -243,7 +250,10 @@ class TaskWarrior(Backend):
             if match:
                 config[match.group('key')] = match.group('value').strip()
 
             if match:
                 config[match.group('key')] = match.group('value').strip()
 

-        return config
+        # Memoize the config dict
+        self._config = config
+
+        return copy.deepcopy(config)

 
     def execute_command(self, args, config_override=None, allow_failure=True,
                         return_all=False):
 
     def execute_command(self, args, config_override=None, allow_failure=True,
                         return_all=False):