python-general-be-lib 0.8.1__tar.gz → 0.8.2__tar.gz

{python_general_be_lib-0.8.1/python_general_be_lib.egg-info → python_general_be_lib-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-general-be-lib
-Version: 0.8.1
+Version: 0.8.2
 Summary: General purpose backend library — SQLAlchemy CRUD/Geometry repositories, FastAPI exceptions, Pydantic base models, logger utilities.
 Author-email: Andrea Di Placido <a.diplacido@arpes.it>, "Arpes S.r.l." <it.admin@arpes.it>
 License: MIT

{python_general_be_lib-0.8.1 → python_general_be_lib-0.8.2}/general/interface/base/declarative_base.py

@@ -45,15 +45,19 @@ class Base:
             default_values = dict()
         mapper: Mapper = inspect(cls)
 
-        column_properties = [prop for prop in mapper.column_attrs if include_only is None or prop.key in include_only]
+        # ``mapper.columns`` è indicizzato per **nome di attributo** e non forza la
+        # configurazione dei mapper: ``column_attrs``/``attrs``/``relationships`` la
+        # forzerebbero, e chiamare ``to_pydantic`` a import-time — il pattern
+        # documentato — romperebbe le relationship con forward-ref non ancora
+        # risolvibili (stessa ragione per cui ``relationship_attributes`` è lazy).
+        selected = [(name, column) for name, column in mapper.columns.items()
+                    if include_only is None or name in include_only]
 
         model_name = f"{cls.__name__}Model" if not model_name else model_name
         fields = dict()
-        field_values = {prop.key: None if full_optional else ... for prop in column_properties}
+        field_values = {name: None if full_optional else ... for name, _ in selected}
         field_values.update(default_values)
-        for prop in column_properties:
-            col = prop.columns[0]
-            field_name = prop.key
+        for field_name, col in selected:
             if isinstance(col.type, Geometry):
                 field_type = Optional[WKBElement] if col.nullable or full_optional else WKBElement
             else:

{python_general_be_lib-0.8.1 → python_general_be_lib-0.8.2}/general/interface/repository/crud_repository.py

@@ -41,7 +41,9 @@ def _reduced_model(entity: Type[Base], columns: frozenset[str]) -> Type[BaseMode
     ricostruire il TypeAdapter — mancherebbe il bersaglio a ogni richiesta, e il suo
     `maxsize=None` tratterebbe una voce per classe generata.
     """
-    ordinate = [prop.key for prop in inspect(entity).column_attrs if prop.key in columns]
+    # ``mapper.columns`` è indicizzato per nome di attributo e non forza la
+    # configurazione dei mapper (a differenza di ``column_attrs``/``attrs``).
+    ordinate = [name for name in inspect(entity).columns.keys() if name in columns]
     digest = hashlib.sha1("|".join(ordinate).encode()).hexdigest()[:8]
     return entity.to_pydantic(
         model_name=f"{entity.__name__}_{digest}",
@@ -131,7 +133,7 @@ class CrudRepository[Entity: Base | Type[Base]]:
 
 
     def _retrieve_parsing_model(self, columns=None, exclude_columns=None):
-        selected_columns = set(columns) if columns else set(self.entity.__mapper__.attrs.keys())  # o columns_names corretto
+        selected_columns = set(columns) if columns else set(inspect(self.entity).columns.keys())
         if exclude_columns:
             selected_columns -= set(exclude_columns)
         if columns is None and not exclude_columns:

{python_general_be_lib-0.8.1 → python_general_be_lib-0.8.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.backends.legacy:build"
 
 [project]
 name = "python-general-be-lib"
-version = "0.8.1"
+version = "0.8.2"
 description = "General purpose backend library — SQLAlchemy CRUD/Geometry repositories, FastAPI exceptions, Pydantic base models, logger utilities."
 readme = "README.md"
 license = { text = "MIT" }

{python_general_be_lib-0.8.1 → python_general_be_lib-0.8.2/python_general_be_lib.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-general-be-lib
-Version: 0.8.1
+Version: 0.8.2
 Summary: General purpose backend library — SQLAlchemy CRUD/Geometry repositories, FastAPI exceptions, Pydantic base models, logger utilities.
 Author-email: Andrea Di Placido <a.diplacido@arpes.it>, "Arpes S.r.l." <it.admin@arpes.it>
 License: MIT

{python_general_be_lib-0.8.1 → python_general_be_lib-0.8.2}/setup.py

@@ -2,7 +2,7 @@ from setuptools import setup, find_packages
 
 setup(
     name="python-general-be-lib",
-    version="0.8.1",
+    version="0.8.2",
     packages=find_packages(include=["general*"]),
     python_requires=">=3.12",
     install_requires=[

{python_general_be_lib-0.8.1 → python_general_be_lib-0.8.2}/tests/test_caricamento_parziale.py

@@ -234,3 +234,51 @@ def test_paging_find_usa_lo_stesso_modello_ridotto(engine: Engine, repo: CrudRep
         pagina = repo.paging_find(columns=["nome"], limit=20)
 
     assert set(pagina.data[0].model_dump()) == {"nome"}
+
+
+# ------------------------------------------- configurazione dei mapper a import-time
+
+
+def test_to_pydantic_non_forza_la_configurazione_dei_mapper() -> None:
+    """``to_pydantic`` deve funzionare su un'entità con forward-ref non ancora risolvibili.
+
+    Il pattern documentato è generare i modelli a import-time
+    (``UserModel = User.to_pydantic(...)``), quando le entità referenziate per nome
+    dalle ``relationship`` possono non essere ancora state importate. Toccare
+    ``mapper.column_attrs``, ``mapper.attrs`` o ``mapper.relationships`` innesca
+    ``configure_mappers()`` su **tutto** il registry e fa fallire l'import con::
+
+        InvalidRequestError: When initializing mapper Mapper[Rel(...)],
+        expression 'Azienda' failed to locate a name
+
+    ``mapper.columns`` invece non la forza, ed è già indicizzato per nome di attributo.
+    È la stessa ragione per cui ``CrudRepository.relationship_attributes`` è una
+    property lazy.
+    """
+    from sqlalchemy import ForeignKey
+    from sqlalchemy.orm import relationship
+
+    class RelAziendaUtente(Base):
+        __tablename__ = "rel_azienda_utente_probe"
+
+        id = Column(Integer, primary_key=True)
+        id_azienda = Column(Integer, ForeignKey("azienda_probe.id"))
+        # riferimento per nome a una classe definita più avanti: irrisolvibile adesso
+        azienda = relationship("AziendaProbe")
+
+    modello = RelAziendaUtente.to_pydantic(full_optional=True)
+    assert set(modello.model_fields) == {"id", "id_azienda"}
+
+    ridotto = RelAziendaUtente.to_pydantic(full_optional=True, include_only=["id_azienda"])
+    assert set(ridotto.model_fields) == {"id_azienda"}
+
+    # la classe mancante arriva dopo: da qui in poi la configurazione può avvenire
+    class AziendaProbe(Base):
+        __tablename__ = "azienda_probe"
+
+        id = Column(Integer, primary_key=True)
+
+    from sqlalchemy.orm import configure_mappers
+
+    configure_mappers()
+    assert RelAziendaUtente.azienda.property.mapper.class_ is AziendaProbe