Merge pull request #141 from psd-tools/v1.8.28

v1.8.28

CHANGES.rst

@@ -1,3 +1,8 @@
+1.8.28 (2019-09-09)
+-------------------
+
+- [api] fix `Effects.__repr__()` when data is empty.
+
 1.8.27 (2019-08-29)
 -------------------
 

src/psd_tools/api/effects.py

@@ -48,15 +48,15 @@ def __init__(self, layer):
     @property
     def scale(self):
         """Scale value."""
-        return self._data.get(Key.Scale).value
+        return self._data.get(Key.Scale).value if self._data else None
 
     @property
     def enabled(self):
         """Whether if all the effects are enabled.
 
         :rtype: bool
         """
-        return bool(self._data.get(b'masterFXSwitch'))
+        return bool(self._data.get(b'masterFXSwitch')) if self._data else False
 
     @property
     def items(self):
@@ -78,9 +78,10 @@ def __getitem__(self, key):
     #     return self._items.__delitem__(key)
 
     def __repr__(self):
-        return '%s(%s%s)' % (
-            self.__class__.__name__, '' if self.enabled else 'disabled ',
-            ' '.join(x.__class__.__name__.lower() for x in self)
+        return '%s(%s)' % (
+            self.__class__.__name__,
+            ' '.join(x.__class__.__name__.lower()
+                     for x in self) if self._data else ''
         )
 
 

src/psd_tools/version.py

@@ -1 +1 @@
-__version__ = '1.8.27'
+__version__ = '1.8.28'

tests/psd_tools/api/test_effects.py

@@ -21,6 +21,8 @@ def fixture():
 def test_effects(fixture):
     assert isinstance(fixture[0].effects.scale, float)
     assert fixture[0].effects.enabled is True
+    for layer in fixture:
+        assert layer.__repr__()
     for effect in fixture[0].effects:
         assert effect.enabled is True