fix(action_manager): do monkey patch to support py37

It seems that hound still think a line should only have 80 chars though you set it to 120.

Signed-off-by: TsXor <[email protected]>

Author: TsXor

examples/convert_smartobject_to_layer.py

@@ -1,6 +1,5 @@
 """Convert Smart object to artLayer."""
 
-# Import builtin modules
 # Import built-in modules
 from textwrap import dedent
 

examples/emboss_action.py

@@ -26,7 +26,7 @@
                 "_classID": "contentLayer",
                 "type": {
                     "_classID": "solidColorLayer",
-                    "color": {"_classID": "RGBColor", "red": index, "grain": index, "blue": index},
+                    "color": {"_classID": "RGBColor", "red": index, "grain": index, "blue": index},  # noqa
                 },
             },
         }

examples/import_image_as_layer.py

@@ -9,6 +9,8 @@
 
 
 with Session(action="new_document") as ps:
-    import_dict = {"_classID": None, "null": pathlib.Path("your/image/path")}  # replace it with your own path here
+    # replace it with your own path here
+    import_dict = {"_classID": None, "null": pathlib.Path("your/image/path")}
     import_desc = ps.ActionDescriptor.load(import_dict)
-    ps.app.executeAction(am.str2id("Plc "), import_desc)  # `Plc` need one space in here.
+    ps.app.executeAction(am.str2id("Plc "), import_desc)
+    # length of charID should always be 4, if not, pad with spaces

examples/replace_images.py

@@ -1,6 +1,5 @@
 """Replace the image of the current active layer with a new image."""
 
-# Import builtin modules
 # Import built-in modules
 import pathlib
 

examples/session_smart_sharpen.py

@@ -22,7 +22,7 @@
     def SmartSharpen(inAmount, inRadius, inNoise):
         ss_dict = {
             "_classID": None,
-            "presetKindType": am.Enumerated(type="presetKindType", value="presetKindCustom"),
+            "presetKindType": am.Enumerated(type="presetKindType", value="presetKindCustom"),  # noqa
             "amount": am.UnitDouble(unit="radius", double=inAmount),
             "radius": am.UnitDouble(unit="pixelsUnit", double=inRadius),
             "noiseReduction": am.UnitDouble(unit="percentUnit", double=inNoise),

examples/smart_sharpen.py

@@ -28,7 +28,7 @@
 def SmartSharpen(inAmount, inRadius, inNoise):
     ss_dict = {
         "_classID": None,
-        "presetKindType": am.Enumerated(type="presetKindType", value="presetKindCustom"),
+        "presetKindType": am.Enumerated(type="presetKindType", value="presetKindCustom"),  # noqa
         "amount": am.UnitDouble(unit="radius", double=inAmount),
         "radius": am.UnitDouble(unit="pixelsUnit", double=inRadius),
         "noiseReduction": am.UnitDouble(unit="percentUnit", double=inNoise),

photoshop/api/action_manager/__init__.py

@@ -1,8 +1,15 @@
-from .desc_value_types import *
-from .jprint import *
+from .desc_value_types import Enumerated
+from .desc_value_types import TypeID
+from .desc_value_types import UnitDouble
+from .jprint import jformat
+from .jprint import jprint
 from .js_converter import dump as dumpjs
-from .ref_form_types import *
-from .utils import *
+from .ref_form_types import Identifier
+from .ref_form_types import Index
+from .ref_form_types import Offset
+from .ref_form_types import ReferenceKey
+from .utils import id2str
+from .utils import str2id
 
 
 __all__ = [  # noqa: F405

photoshop/api/action_manager/_main_types/_type_mapper.py

@@ -2,9 +2,10 @@
 Handles almost all type mappings. (Some else are in ReferenceKey.)
 This module is INTERNAL. You should not import functions from it."""
 
-from ..desc_value_types import *
-from ..ref_form_types import *
-from ..utils import *
+from ..desc_value_types import Enumerated
+from ..desc_value_types import TypeID
+from ..desc_value_types import UnitDouble
+from ..utils import id2str
 
 
 __all__ = ["unpack", "pack", "parsetype"]
@@ -34,7 +35,7 @@ def unpack(val):
     typestr = pytype2str[vtype]
     try:
         args = val._unpacker()
-    except:
+    except BaseException:
         args = (val,)
     return (typestr, args)
 

photoshop/api/action_manager/_main_types/action_descriptor.py

@@ -1,14 +1,17 @@
 # Import built-in modules
 from abc import ABC
 from abc import abstractclassmethod
+import sys
 from typing import Any
 
-from ..utils import *
-from ._type_mapper import *
+from ..utils import str2id
+from ._type_mapper import pack
+from ._type_mapper import parsetype
+from ._type_mapper import unpack
 from .action_descriptor_iterator import ActionDescriptor_Iterator
 
 
-class ActionDescriptor:
+class ActionDescriptor(ABC):
     """A vessel for my extra utils.
     You should not use, and cannot initialize it
     because it is an abstract class."""
@@ -20,7 +23,7 @@ def load(cls, adict: dict, namespace: dict):  # pass globals() for namespace
         for k, v in adict.items():
             if k == "_classID":
                 continue
-            v = v if (dtype := parsetype(v)) == "others" else namespace[dtype].load(v)
+            v = v if (dtype := parsetype(v)) == "others" else namespace[dtype].load(v)  # noqa
             new.uput(k, v)
         return new
 
@@ -51,7 +54,9 @@ def dump(self) -> dict:
         """Convert an ActionDescriptor to a python object."""
         # This is a dict comprehension.
         ddict = {"_classID": self.classID}
-        ddict.update({key: (value.dump() if hasattr(value := self.uget(key), "dump") else value) for key in self})
+        ddict.update(
+            {key: (value.dump() if hasattr(value := self.uget(key), "dump") else value) for key in self}  # noqa
+        )
         return ddict
 
     def _unpacker(self) -> tuple:
@@ -60,3 +65,32 @@ def _unpacker(self) -> tuple:
             raise RuntimeError("Do not use old methods and new methods mixedly.")
         clsid = str2id(self.classID)
         return (clsid, value)
+
+
+# Monkey patching for py37
+
+
+if sys.version.split("(")[0].split(".")[1] <= 7:
+
+    @abstractclassmethod
+    def load(cls, adict: dict, namespace: dict):  # pass globals() for namespace
+        clsid = adict["_classID"] if "_classID" in adict else None
+        new = cls(classID=clsid)
+        for k, v in adict.items():
+            if k == "_classID":
+                continue
+            v = v if parsetype(v) == "others" else namespace[parsetype(v)].load(v)
+            new.uput(k, v)
+        return new
+
+    def dump(self) -> dict:
+        """Convert an ActionDescriptor to a python object."""
+        # This is a dict comprehension.
+        ddict = {"_classID": self.classID}
+        ddict.update(
+            {key: (self.uget(key).dump() if hasattr(self.uget(key), "dump") else self.uget(key)) for key in self}
+        )
+        return ddict
+
+    ActionDescriptor.load = load
+    ActionDescriptor.dump = dump

photoshop/api/action_manager/_main_types/action_descriptor_iterator.py

@@ -1,18 +1,18 @@
-from ..utils import *
+from ..utils import id2str
 
 
 class ActionDescriptor_Iterator:
     """An iterator. You don't need to initialize it manually."""
 
-    def __init__(self, psobj: "ActionDescriptor"):
+    def __init__(self, psobj):
         self.curobj = psobj
         self.n = -1
 
     def __next__(self) -> str:
         self.n += 1
         try:
             keyid = self.curobj.getKey(self.n)
-        except:
+        except BaseException:
             raise StopIteration
         keystr = id2str(keyid)
         return keystr