Skip to content

Commit 7891393

Browse files
committed
bug fixed, is_deleted in self._model.__table__.columns:
1 parent 57430f4 commit 7891393

File tree

4 files changed

+32
-27
lines changed

4 files changed

+32
-27
lines changed

docker-compose.yml

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -44,32 +44,32 @@ services:
4444
- redis-data:/data
4545

4646
# #-------- uncomment to create first superuser --------
47-
# create_superuser:
48-
# build:
49-
# context: .
50-
# dockerfile: Dockerfile
51-
# env_file:
52-
# - ./src/.env
53-
# depends_on:
54-
# - db
55-
# command: python -m src.scripts.create_first_superuser
56-
# volumes:
57-
# - ./src:/code/src
47+
create_superuser:
48+
build:
49+
context: .
50+
dockerfile: Dockerfile
51+
env_file:
52+
- ./src/.env
53+
depends_on:
54+
- db
55+
command: python -m src.scripts.create_first_superuser
56+
volumes:
57+
- ./src:/code/src
5858

5959
# #-------- uncomment to run tests --------
60-
# pytest:
61-
# build:
62-
# context: .
63-
# dockerfile: Dockerfile
64-
# env_file:
65-
# - ./src/.env
66-
# depends_on:
67-
# - db
68-
# - create_superuser
69-
# - redis
70-
# command: python -m pytest
71-
# volumes:
72-
# - ./src:/code/src
60+
pytest:
61+
build:
62+
context: .
63+
dockerfile: Dockerfile
64+
env_file:
65+
- ./src/.env
66+
depends_on:
67+
- db
68+
- create_superuser
69+
- redis
70+
command: python -m pytest
71+
volumes:
72+
- ./src:/code/src
7373

7474
volumes:
7575
postgres-data:

src/app/api/v1/users.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,3 +117,8 @@ async def erase_db_user(
117117

118118
db_user = await crud_users.db_delete(db=db, username=username)
119119
return {"message": "User deleted from the database"}
120+
121+
@router.get("/deleted_users")
122+
async def read_users(request: Request, db: Annotated[AsyncSession, Depends(async_get_db)]):
123+
users = await crud_users.get_multi(db=db, schema_to_select=UserRead, is_deleted=True)
124+
return users

src/app/crud/crud_base.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -228,9 +228,9 @@ async def delete(
228228
-------
229229
None
230230
"""
231-
db_row = db_row or await self.get(db=db, **kwargs)
231+
db_row = db_row or await self.exists(db=db, **kwargs)
232232
if db_row:
233-
if "is_deleted" in db_row:
233+
if "is_deleted" in self._model.__table__.columns:
234234
object_dict = {
235235
"is_deleted": True,
236236
"deleted_at": datetime.utcnow()

src/app/crud/helper.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ def _extract_matching_columns_from_schema(model: Type[Base], schema: Type[BaseMo
2222
"""
2323
column_list = list(model.__table__.columns)
2424
if schema is not None:
25-
schema_fields = schema.__fields__.keys()
25+
schema_fields = schema.model_fields.keys()
2626
column_list = []
2727
for column_name in schema_fields:
2828
if hasattr(model, column_name):

0 commit comments

Comments
 (0)