Skip to content

Commit 01506bb

Browse files
committed
fix none getattr
1 parent f7e9a49 commit 01506bb

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

rental_backend/routes/item.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,11 @@ async def get_items(
4242
if is_available is not None:
4343
query = query.filter(Item.is_available == is_available)
4444

45-
column = getattr(Item, order_by, Item.id)
45+
new_order = order_by if order_by is not None else "id"
46+
column = getattr(Item, new_order)
4647
if order == "desc":
4748
query = query.order_by(column.desc())
48-
elif order == "asc":
49+
else:
4950
query = query.order_by(column.asc())
5051

5152
items = query.all()

0 commit comments

Comments
 (0)