lazy: Properly override getattr calls

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

diff --git a/tasklib/lazy.py b/tasklib/lazy.py
index 0bd93323762c7f8d265dd6ae455118d27f5b8f80..04ccdf5216f7bee3f3b5c83d24b304c3e095bc22 100644 (file)
--- a/tasklib/lazy.py
+++ b/tasklib/lazy.py
@@ -29,7 +29,7 @@ class LazyUUIDTask(object):
         # Getattr is called only if the attribute could not be found using
         # normal means
         self.replace()
-        return self.name
+        return getattr(self, name)
 
     def __eq__(self, other):
         if other['uuid']:
@@ -39,6 +39,16 @@ class LazyUUIDTask(object):
     def __hash__(self):
         return self['uuid'].__hash__()
 
+    def __repr__(self):
+        return "LazyUUIDTask: {0}".format(self._uuid)
+
+    @property
+    def saved(self):
+        """
+        Implementation of the 'saved' property. Always returns True.
+        """
+        return True
+
     def replace(self):
         """
         Performs conversion to the regular Task object, referenced by the
@@ -66,12 +76,24 @@ class LazyUUIDTaskSet(object):
     def __getattr__(self, name):
         # Getattr is called only if the attribute could not be found using
         # normal means
-        self.replace()
-        return self.name
+
+        if name.startswith('__'):
+            # If some internal method was being search, do not convert
+            # to TaskQuerySet just because of that
+            raise AttributeError
+        else:
+            self.replace()
+            return getattr(self, name)
+
+    def __repr__(self):
+        return "LazyUUIDTaskSet([{0}])".format(', '.join(self._uuids))
 
     def __eq__(self, other):
         return set(t['uuid'] for t in other) == self._uuids
 
+    def __ne__(self, other):
+        return not (self == other)
+
     def __contains__(self, task):
         return task['uuid'] in self._uuids
 
@@ -79,9 +101,24 @@ class LazyUUIDTaskSet(object):
         return len(self._uuids)
 
     def __iter__(self):
-        self.replace()
-        for task in self:
-            yield task
+        for uuid in self._uuids:
+            yield LazyUUIDTask(self._tw, uuid)
+
+    def __sub__(self, other):
+        return LazyUUIDTaskSet(self._tw,
+                               self._uuids - set(t['uuid'] for t in other))
+
+    def __isub__(self, other):
+        self._uuids -= set(t['uuid'] for t in other)
+        return self
+
+    def __or__(self, other):
+        return LazyUUIDTaskSet(self._tw,
+                               self._uuids | set(t['uuid'] for t in other))
+
+    def __ior__(self, other):
+        self._uuids |= set(t['uuid'] for t in other)
+        return self
 
     def replace(self):
         """