log statements

Author: sdaqo

library_db/library.db

@@ -1,14 +1,14 @@
 import re
 from hashlib import md5
 from datetime import date, datetime
-from flask import Blueprint, session, request
+from flask import Blueprint, session, request, current_app
 
 
 from library_db.utils.db_utils import (
     get_media,
     is_media_borrowed,
     get_user_data,
-    borrow_book,
+    borrow_media,
     update_user,
     delete_user,
 )
@@ -42,8 +42,9 @@ def borrow(media_id):
     if age_limit > age:
         return {"error": "User too Young"}
 
-    borrow_book(media_id, user.get("id"))
+    borrow_media(media_id, user.get("id"))
 
+    current_app.logger.info(f"{session['email']} Borrowed media with id {media_id}")
     return {"status": "success"}
 
 
@@ -159,4 +160,5 @@ def remove_user():
     session.pop("email")
     session.pop("pwdhash")
 
+    current_app.logger(f"{session['email']} Deleted user with email {session['email']}")
     return {"status": "success"}

library_db/routes/api/user/user.py

@@ -10,7 +10,7 @@
 def home():
     global visit_counter
 
-    current_app.logger.info(f"Home Page visited {visit_counter} times.")
     visit_counter += 1
+    current_app.logger.info(f"Home Page visited {visit_counter} times.")
     template_vars = get_template_vars(session)
     return render_template("general/index.html", **template_vars)

library_db/routes/general/general.py

@@ -79,6 +79,7 @@ def del_user():
         )
 
     delete_user(user.get("id"))
+    current_app.logger(f"{session['email']} Deleted user with email {user['email']}")
     return redirect(url_for("panel_bluep.admin_bluep.user_managment"))
 
 
@@ -130,7 +131,7 @@ def get_error_log(log):
         return send_from_directory(
             directory=error_log_dir, path=log, as_attachment=True
         )
-    
+
     return abort(404)
 
 

library_db/routes/panel/admin/admin.py

@@ -6,6 +6,7 @@
     render_template,
     request,
     abort,
+    current_app,
 )
 
 
@@ -136,6 +137,9 @@ def ret_error(error: str):
         isbn=isbn,
     )
 
+    current_app.logger.info(
+        f"{session['email']} Updated media {media.title} with id {id}"
+    )
     return redirect(url_for("panel_bluep.staff_bluep.staff_panel_alter"))
 
 
@@ -177,6 +181,7 @@ def alter_author():
 
     update_author(id, author_name)
 
+    current_app.logger.info(f"{session['email']} Updated author {author} with id {id}")
     return redirect(url_for("panel_bluep.staff_bluep.staff_panel_alter"))
 
 
@@ -203,7 +208,7 @@ def ret_error(error: str):
         try:
             isbn = int(isbn)
         except ValueError:
-            return ret_error("ISBN has to be Number")
+            return ret_error("ISBN has to be a Number")
 
     if get_media_id(title):
         return ret_error("Media with this Title already exsists")
@@ -221,14 +226,15 @@ def ret_error(error: str):
 
     author = get_author_id(author)
 
-    add_media_item(
+    new_id = add_media_item(
         title=title,
         author_id=author,
         age_limit=age_limit,
         media_type_id=media_type,
         isbn=isbn,
     )
 
+    current_app.logger.info(f"{session['email']} Added media {title} with id {new_id}")
     return redirect(url_for("panel_bluep.staff_bluep.staff_panel_addremove"))
 
 
@@ -255,8 +261,9 @@ def add_author():
             )
         )
 
-    add_author_to_db(author)
+    new_id = add_author_to_db(author)
 
+    current_app.logger.info(f"{session['email']} Added author with id {new_id}")
     return redirect(url_for("panel_bluep.staff_bluep.staff_panel_addremove"))
 
 
@@ -284,4 +291,7 @@ def remove_media():
         )
 
     delete_media(media_id)
+    current_app.logger.info(
+        f"{session['email']} Removed media {media} with id {media_id}"
+    )
     return redirect(url_for("panel_bluep.staff_bluep.staff_panel_addremove"))

library_db/routes/panel/staff/staff.py

@@ -8,6 +8,7 @@
     request,
     abort,
     Response,
+    current_app,
 )
 
 
@@ -78,4 +79,5 @@ def delete_user():
     if pwdhash != md5(password.encode()).hexdigest():
         return abort(Response("Wrong Password", 401))
 
+    current_app.logger(f"{session['email']} Deleted user with email {session['email']}")
     return redirect(url_for("auth_bluep.login", next="/me/profile"))

library_db/routes/panel/user/user.py

@@ -239,7 +239,7 @@ def estimate_return_date(media_id: int) -> date | None:
     return deadline.date()
 
 
-def borrow_book(media_id: int, user_id: int):
+def borrow_media(media_id: int, user_id: int):
     statement = """
         INSERT INTO borrowings (media_id, user_id, borrow_date)
         VALUES (?, ?, ?)
@@ -351,8 +351,8 @@ def author_exsists(name: str) -> bool:
 
 
 def add_media_item(
-    title: str, author_id: int, age_limit: int, media_type_id: int, isbn: int = None
-):
+    title: str, author_id: int, age_limit: int, media_type_id: int, isbn: int | None = None
+) -> int | None:
     statement = """
         INSERT INTO media (
             title, media_type_id,
@@ -367,6 +367,8 @@ def add_media_item(
 
     con.commit()
 
+    return cur.lastrowid
+
 
 def get_media_type_id(media_type: str) -> int | None:
     cur = con.cursor()
@@ -393,12 +395,14 @@ def get_media_id(title: str) -> int | None:
         return res[0]
 
 
-def add_author_to_db(name: str):
+def add_author_to_db(name: str) -> int | None:
     cur = con.cursor()
     cur.execute("""INSERT INTO authors (name) VALUES(?)""", (name,))
 
     con.commit()
 
+    return cur.lastrowid
+
 
 def delete_media(media_id: int):
     cur = con.cursor()