44
55from sqlalchemy import select
66from sqlalchemy .orm import Session
7+ from sqlalchemy .exc import SQLAlchemyError
78from models .player_model import PlayerModel
89from schemas .player_schema import Player
910
@@ -26,7 +27,7 @@ def create(orm_session: Session, player_model: PlayerModel):
2627 try :
2728 orm_session .commit ()
2829 return True
29- except Exception as error :
30+ except SQLAlchemyError as error :
3031 print (f"Error trying to create the Player: { error } " )
3132 orm_session .rollback ()
3233 return False
@@ -35,13 +36,13 @@ def create(orm_session: Session, player_model: PlayerModel):
3536
3637
3738def retrieve_all (orm_session : Session ):
38- """Retrieves all the Players from the database.
39+ """Retrieves all the players from the database.
3940
4041 Args:
4142 orm_session: The SQLAlchemy ORM session instance.
4243
4344 Returns:
44- A collection with all the Players .
45+ A collection with all the players .
4546 """
4647 # https://docs.sqlalchemy.org/en/20/changelog/migration_20.html#migration-20-query-usage
4748 statement = select (Player )
@@ -105,7 +106,7 @@ def update(orm_session: Session, player_model: PlayerModel):
105106 try :
106107 orm_session .commit ()
107108 return True
108- except Exception as error :
109+ except SQLAlchemyError as error :
109110 print (f"Error trying to update the Player: { error } " )
110111 orm_session .rollback ()
111112 return False
@@ -128,7 +129,7 @@ def delete(orm_session: Session, player_id: int):
128129 try :
129130 orm_session .commit ()
130131 return True
131- except Exception as error :
132+ except SQLAlchemyError as error :
132133 print (f"Error trying to delete the Player: { error } " )
133134 orm_session .rollback ()
134135 return False
0 commit comments