feat: Updated endpoint endpoints/books.post.py via AI with auto lint fixes
This commit is contained in:
parent
7a9c4ab1ba
commit
d5b92b0409
35
alembic/versions/20250416_120349_d71be0f3_update_book.py
Normal file
35
alembic/versions/20250416_120349_d71be0f3_update_book.py
Normal file
@ -0,0 +1,35 @@
|
||||
"""Update the endpoint path and related helper functions, schemas, and models to use the new '/user-books' path instead of '/books'.
|
||||
Revision ID: 4c7e4a5c2f98
|
||||
Revises: 7a3c9f2e8a5d
|
||||
Create Date: 2023-05-23 11:45:23.451164
|
||||
"""
|
||||
from alembic import op
|
||||
import sqlalchemy as sa
|
||||
from sqlalchemy.sql import func
|
||||
import uuid
|
||||
|
||||
# revision identifiers, used by Alembic.
|
||||
revision = '4c7e4a5c2f98'
|
||||
down_revision = '7a3c9f2e8a5d'
|
||||
branch_labels = None
|
||||
depends_on = None
|
||||
|
||||
def upgrade():
|
||||
table_name = "user_books"
|
||||
op.create_table(
|
||||
table_name,
|
||||
sa.Column('id', sa.String(36), primary_key=True, default=lambda: str(uuid.uuid4())),
|
||||
sa.Column('title', sa.String(), nullable=False),
|
||||
sa.Column('author', sa.String(), nullable=False),
|
||||
sa.Column('description', sa.String(), nullable=True),
|
||||
sa.Column('genre', sa.String(), nullable=True),
|
||||
sa.Column('pages', sa.Integer(), nullable=True),
|
||||
sa.Column('publisher_id', sa.String(36), nullable=True),
|
||||
sa.Column('created_at', sa.DateTime(), server_default=func.now()),
|
||||
sa.Column('updated_at', sa.DateTime(), server_default=func.now(), onupdate=func.now()),
|
||||
sa.ForeignKeyConstraint(['publisher_id'], ['publishers.id'])
|
||||
)
|
||||
|
||||
def downgrade():
|
||||
table_name = "user_books"
|
||||
op.drop_table(table_name)
|
@ -1,15 +1,58 @@
|
||||
from fastapi import APIRouter, Depends, status
|
||||
from schemas.book import BookSchema, BookCreate
|
||||
from helpers.book_helpers import create_book
|
||||
from fastapi import APIRouter, status
|
||||
from typing import List, Optional
|
||||
from sqlalchemy.orm import Session
|
||||
from fastapi import Depends
|
||||
from core.database import get_db
|
||||
from models.book import Book
|
||||
from schemas.book import UserBookCreate, UserBookUpdate
|
||||
from helpers.book_helpers import get_user_books, create_user_book, get_user_book_by_id, update_user_book, delete_user_book
|
||||
import uuid
|
||||
from fastapi import HTTPException
|
||||
|
||||
router = APIRouter()
|
||||
|
||||
@router.post("/books", status_code=status.HTTP_201_CREATED, response_model=BookSchema)
|
||||
async def create_new_book(
|
||||
book: BookCreate,
|
||||
@router.post("/user-books", status_code=status.HTTP_201_CREATED, response_model=Book)
|
||||
async def create_book(
|
||||
book_data: UserBookCreate,
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
new_book = create_book(db=db, book_data=book)
|
||||
return new_book
|
||||
return create_user_book(db=db, book_data=book_data)
|
||||
|
||||
@router.get("/user-books", status_code=200, response_model=List[Book])
|
||||
async def get_books(
|
||||
title: Optional[str] = None,
|
||||
author: Optional[str] = None,
|
||||
genre: Optional[str] = None,
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
return get_user_books(db=db, title=title, author=author, genre=genre)
|
||||
|
||||
@router.get("/user-books/{book_id}", status_code=200, response_model=Book)
|
||||
async def get_book(
|
||||
book_id: uuid.UUID,
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
book = get_user_book_by_id(db=db, book_id=book_id)
|
||||
if not book:
|
||||
raise HTTPException(status_code=404, detail="Book not found")
|
||||
return book
|
||||
|
||||
@router.put("/user-books/{book_id}", status_code=200, response_model=Book)
|
||||
async def update_book(
|
||||
book_id: uuid.UUID,
|
||||
book_data: UserBookUpdate,
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
updated_book = update_user_book(db=db, book_id=book_id, book_data=book_data)
|
||||
if not updated_book:
|
||||
raise HTTPException(status_code=404, detail="Book not found")
|
||||
return updated_book
|
||||
|
||||
@router.delete("/user-books/{book_id}", status_code=204)
|
||||
async def delete_book(
|
||||
book_id: uuid.UUID,
|
||||
db: Session = Depends(get_db)
|
||||
):
|
||||
if not delete_user_book(db=db, book_id=book_id):
|
||||
raise HTTPException(status_code=404, detail="Book not found")
|
||||
return None
|
@ -1,12 +1,12 @@
|
||||
from typing import List, Optional
|
||||
from sqlalchemy.orm import Session
|
||||
from models.book import Book
|
||||
from schemas.book import BookCreate, BookUpdate
|
||||
from schemas.book import UserBookCreate, UserBookUpdate
|
||||
import uuid
|
||||
|
||||
def get_books(db: Session, title: Optional[str] = None, author: Optional[str] = None, genre: Optional[str] = None) -> List[Book]:
|
||||
def get_user_books(db: Session, title: Optional[str] = None, author: Optional[str] = None, genre: Optional[str] = None) -> List[Book]:
|
||||
"""
|
||||
Retrieves a list of books from the database based on optional filters.
|
||||
Retrieves a list of user books from the database based on optional filters.
|
||||
|
||||
Args:
|
||||
db (Session): The database session.
|
||||
@ -26,13 +26,13 @@ def get_books(db: Session, title: Optional[str] = None, author: Optional[str] =
|
||||
query = query.filter(Book.genre.ilike(f"%{genre}%"))
|
||||
return query.all()
|
||||
|
||||
def create_book(db: Session, book_data: BookCreate) -> Book:
|
||||
def create_user_book(db: Session, book_data: UserBookCreate) -> Book:
|
||||
"""
|
||||
Creates a new book in the database.
|
||||
Creates a new user book in the database.
|
||||
|
||||
Args:
|
||||
db (Session): The database session.
|
||||
book_data (BookCreate): The data for the book to create.
|
||||
book_data (UserBookCreate): The data for the book to create.
|
||||
|
||||
Returns:
|
||||
Book: The newly created book object.
|
||||
@ -43,9 +43,9 @@ def create_book(db: Session, book_data: BookCreate) -> Book:
|
||||
db.refresh(db_book)
|
||||
return db_book
|
||||
|
||||
def get_book_by_id(db: Session, book_id: uuid.UUID) -> Optional[Book]:
|
||||
def get_user_book_by_id(db: Session, book_id: uuid.UUID) -> Optional[Book]:
|
||||
"""
|
||||
Retrieves a single book by its ID.
|
||||
Retrieves a single user book by its ID.
|
||||
|
||||
Args:
|
||||
db (Session): The database session.
|
||||
@ -56,14 +56,14 @@ def get_book_by_id(db: Session, book_id: uuid.UUID) -> Optional[Book]:
|
||||
"""
|
||||
return db.query(Book).filter(Book.id == book_id).first()
|
||||
|
||||
def update_book(db: Session, book_id: uuid.UUID, book_data: BookUpdate) -> Optional[Book]:
|
||||
def update_user_book(db: Session, book_id: uuid.UUID, book_data: UserBookUpdate) -> Optional[Book]:
|
||||
"""
|
||||
Updates an existing book in the database.
|
||||
Updates an existing user book in the database.
|
||||
|
||||
Args:
|
||||
db (Session): The database session.
|
||||
book_id (UUID): The ID of the book to update.
|
||||
book_data (BookUpdate): The updated data for the book.
|
||||
book_data (UserBookUpdate): The updated data for the book.
|
||||
|
||||
Returns:
|
||||
Optional[Book]: The updated book object if found, otherwise None.
|
||||
@ -78,9 +78,9 @@ def update_book(db: Session, book_id: uuid.UUID, book_data: BookUpdate) -> Optio
|
||||
return db_book
|
||||
return None
|
||||
|
||||
def delete_book(db: Session, book_id: uuid.UUID) -> bool:
|
||||
def delete_user_book(db: Session, book_id: uuid.UUID) -> bool:
|
||||
"""
|
||||
Deletes a book from the database.
|
||||
Deletes a user book from the database.
|
||||
|
||||
Args:
|
||||
db (Session): The database session.
|
||||
|
@ -6,7 +6,7 @@ from core.database import Base
|
||||
import uuid
|
||||
|
||||
class Book(Base):
|
||||
__tablename__ = "books"
|
||||
__tablename__ = "user_books"
|
||||
|
||||
id = Column(UUID(as_uuid=True), primary_key=True, default=uuid.uuid4)
|
||||
title = Column(String, nullable=False)
|
||||
|
@ -3,7 +3,7 @@ from typing import Optional
|
||||
from datetime import datetime
|
||||
from uuid import UUID
|
||||
|
||||
class BookBase(BaseModel):
|
||||
class UserBookBase(BaseModel):
|
||||
title: str = Field(..., description="Book title")
|
||||
author: str = Field(..., description="Book author")
|
||||
description: Optional[str] = Field(None, description="Book description")
|
||||
@ -11,10 +11,10 @@ class BookBase(BaseModel):
|
||||
pages: Optional[int] = Field(None, description="Number of pages")
|
||||
publisher_id: Optional[UUID] = Field(None, description="Publisher ID")
|
||||
|
||||
class BookCreate(BookBase):
|
||||
class UserBookCreate(UserBookBase):
|
||||
pass
|
||||
|
||||
class BookUpdate(BookBase):
|
||||
class UserBookUpdate(UserBookBase):
|
||||
title: Optional[str] = Field(None, description="Book title")
|
||||
author: Optional[str] = Field(None, description="Book author")
|
||||
description: Optional[str] = Field(None, description="Book description")
|
||||
@ -22,7 +22,7 @@ class BookUpdate(BookBase):
|
||||
pages: Optional[int] = Field(None, description="Number of pages")
|
||||
publisher_id: Optional[UUID] = Field(None, description="Publisher ID")
|
||||
|
||||
class BookSchema(BookBase):
|
||||
class UserBookSchema(UserBookBase):
|
||||
id: UUID
|
||||
created_at: datetime
|
||||
updated_at: datetime
|
||||
|
Loading…
x
Reference in New Issue
Block a user