]> git.ipfire.org Git - thirdparty/psycopg.git/commitdiff
Drop .register() method from Dumper/Loader classes
authorDaniele Varrazzo <daniele.varrazzo@gmail.com>
Sun, 11 Jul 2021 02:16:25 +0000 (04:16 +0200)
committerDaniele Varrazzo <daniele.varrazzo@gmail.com>
Sun, 11 Jul 2021 12:43:58 +0000 (14:43 +0200)
Use the register_dumper/loader() method on the adapters, readily
available from the context. A bit of a violation of Demeter law, but
much more intuitive and simplifies the Dumper/Loader interface (the
methods were outside the protocol).

psycopg/psycopg/adapt.py

index d2d61daabc3585cd6fe6357e8916121240d5b18c..08578c865ad94ee3eafa5c4205f0431dae19c809 100644 (file)
@@ -87,18 +87,6 @@ class Dumper(ABC):
         """
         return self
 
-    @classmethod
-    def register(
-        this_cls, cls: Union[type, str], context: Optional[AdaptContext] = None
-    ) -> None:
-        """
-        Configure *context* to use this dumper to convert object of type *cls*.
-        """
-        from . import postgres
-
-        adapters = context.adapters if context else postgres.adapters
-        adapters.register_dumper(cls, this_cls)
-
 
 class Loader(ABC):
     """
@@ -118,18 +106,6 @@ class Loader(ABC):
         """Convert a PostgreSQL value to a Python object."""
         ...
 
-    @classmethod
-    def register(
-        cls, oid: Union[int, str], context: Optional[AdaptContext] = None
-    ) -> None:
-        """
-        Configure *context* to use this loader to convert values with OID *oid*.
-        """
-        from . import postgres
-
-        adapters = context.adapters if context else postgres.adapters
-        adapters.register_loader(oid, cls)
-
 
 Transformer: Type["proto.Transformer"]