Listing or returning items with UoW raises sqlalchemy.orm.exc.DetachedInstanceError #63

Activity

1. sevetseh28 commented on Mar 22, 2023

   

   sevetseh28

   on Mar 22, 2023

   Last edited by sevetseh28

   Author

   More actions

   Maybe using schemas.Item as a DTO like this, and convert:

   def item_to_schema(item: models.Item) -> schemas.Item:
       return schemas.Item(id=item.id, title=item.title, description=item.description)

   @app.get("/items/", response_model=list[schemas.Item])
   async def read_items(uow: AbstractUnitOfWork = Depends(get_uow)) -> List[schemas.Item]:
       with uow:
           items = uow.repo.list()
           # Convert items to schemas.Item DTOs
           items_dtos = [item_to_schema(item) for item in items]
       return items_dtos

   wdyt?
2. instanceofmel commented on Mar 6, 2024

   

   instanceofmel

   on Mar 6, 2024

   More actions

   A little late, but for future readers:

   You have multiple options in this case, including the one @sevetseh28 provided. Other options:

   • Return your list inside the with block
   • Set expire_on_commit=False when creating the session. This will disable the reloading of attributes after commit. I think it's safe to disable it because a handler shouldn't reload anything after commiting anyways as the aggregate must be transactionally consistent and complete before committing.
   • Use a view as described in Chapter 12