feat :add log
This commit is contained in:
@@ -7,11 +7,11 @@ from sqlalchemy.orm import Session
|
||||
import schemas
|
||||
from schemas import ItemResponse
|
||||
from services import crud
|
||||
|
||||
from app.db.session import SessionLocal
|
||||
|
||||
from core.logger import get_logger
|
||||
|
||||
router = APIRouter()
|
||||
logger = get_logger('admin')
|
||||
|
||||
|
||||
def get_db():
|
||||
@@ -25,7 +25,13 @@ def get_db():
|
||||
# GET ALL ITEMS
|
||||
@router.get("/items", response_model=list[ItemResponse])
|
||||
def get_items(db: Session = Depends(get_db)):
|
||||
return crud.get_items(db)
|
||||
try:
|
||||
items = crud.get_items(db)
|
||||
logger.info(f"Items fetched: {len(items)} items")
|
||||
return items
|
||||
except Exception as e:
|
||||
logger.error(f"Error fetching items: {e}")
|
||||
raise
|
||||
|
||||
|
||||
# CREATE ITEM
|
||||
@@ -34,7 +40,13 @@ def create_item(
|
||||
item: schemas.ItemCreate,
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
return crud.create_item(db, item.title)
|
||||
try:
|
||||
result = crud.create_item(db, item.title)
|
||||
logger.info(f"Item created: {item.title}")
|
||||
return result
|
||||
except Exception as e:
|
||||
logger.error(f"Error creating item: {e}")
|
||||
raise
|
||||
|
||||
|
||||
# DELETE ITEM
|
||||
@@ -43,7 +55,15 @@ def delete_item(
|
||||
item_id: int,
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
item = crud.delete_item(db, item_id)
|
||||
if not item:
|
||||
raise HTTPException(status_code=404, detail="Item not found")
|
||||
return {"message": "Item deleted"}
|
||||
try:
|
||||
item = crud.delete_item(db, item_id)
|
||||
if not item:
|
||||
logger.warning(f"Item not found: {item_id}")
|
||||
raise HTTPException(status_code=404, detail="Item not found")
|
||||
logger.info(f"Item deleted: {item_id}")
|
||||
return {"message": "Item deleted"}
|
||||
except HTTPException:
|
||||
raise
|
||||
except Exception as e:
|
||||
logger.error(f"Error deleting item: {e}")
|
||||
raise
|
||||
@@ -6,54 +6,36 @@ from sqlalchemy.orm import Session
|
||||
|
||||
import schemas
|
||||
from services import crud
|
||||
|
||||
from app.db.session import SessionLocal
|
||||
|
||||
from core.logger import get_logger
|
||||
|
||||
router = APIRouter()
|
||||
logger = get_logger('user')
|
||||
|
||||
|
||||
def get_db():
|
||||
|
||||
db = SessionLocal()
|
||||
|
||||
try:
|
||||
yield db
|
||||
|
||||
finally:
|
||||
db.close()
|
||||
|
||||
|
||||
# GET ITEMS
|
||||
@router.get(
|
||||
"/items",
|
||||
response_model=list[schemas.ItemResponse]
|
||||
)
|
||||
def get_items(
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
|
||||
return crud.get_items(db)
|
||||
@router.get("/items", response_model=list[schemas.ItemResponse])
|
||||
def get_items(db: Session = Depends(get_db)):
|
||||
try:
|
||||
items = crud.get_items(db)
|
||||
logger.info(f"User fetched items: {len(items)} items")
|
||||
return items
|
||||
except Exception as e:
|
||||
logger.error(f"Error fetching items: {e}")
|
||||
raise
|
||||
|
||||
|
||||
# SPIN
|
||||
@router.post("/spin")
|
||||
def spin(
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
|
||||
winner = crud.spin_wheel(db)
|
||||
|
||||
if not winner:
|
||||
|
||||
raise HTTPException(
|
||||
status_code=400,
|
||||
detail="No items available"
|
||||
)
|
||||
|
||||
return {
|
||||
"winner": {
|
||||
"id": winner.id,
|
||||
"title": winner.title
|
||||
}
|
||||
}
|
||||
def spin(db: Session = Depends(get_db)):
|
||||
try:
|
||||
winner = crud.spin_wheel(db)
|
||||
if not winner:
|
||||
Reference in New Issue
Block a user