improve messages and error handling

app.py

@@ -91,19 +91,31 @@ def rueckgabe(idBuch, grund='rueckgabe'):
     - grund (str): The reason for the return (default: 'rueckgabe').
 
     Returns:
-    - int: 0 if the book was not found or already returned.
-
+    - dict: Information about the return, including the students who had the book.
+    """
+    sql = f"""
+    SELECT a.`idBuch`, a.`idKind`, a.`ausleihe`, a.`rueckgabe`, a.`rueckGrund`,
+           CONCAT(k.`Vorname`, ' ', k.`Nachnamen`) AS student_name
+    FROM `{AUSLEIHE_TABLE}` a
+    JOIN `kind` k ON a.`idKind` = k.`idKind`
+    WHERE a.`idBuch` = %s AND a.`rueckgabe` is NULL;
     """
-    sql = f"SELECT `idBuch`, `idKind`, `ausleihe`, `rueckgabe`, `rueckGrund` FROM `{AUSLEIHE_TABLE}` WHERE `idBuch` = %s AND `rueckgabe` is NULL;"
     result = execute_query(sql, (idBuch,))
     if len(result) == 0:
-        return 0
+        return {"success": False, "message": "Buch nicht gefunden oder bereits zurückgegeben"}
+    
     # return the book
     sql = f"UPDATE `{AUSLEIHE_TABLE}` SET `rueckgabe` = NOW(), `rueckGrund` = %s WHERE `idBuch` = %s AND `rueckgabe` is NULL;"
     execute_query(sql, (grund, idBuch))
-    #pprint.pprint(result)
-    # return if the book was returned or not and who had it before
-    return result
+    
+    student_names = [row['student_name'] for row in result]
+    students_str = ", ".join(student_names)
+    
+    return {
+        "success": True,
+        "message": f"Buch zurückgegeben von: {students_str}",
+        "student_names": student_names
+    }
 
 @app.get("/borrow/{idBuch}/{idKind}")
 def ausleihe(idBuch, idKind):
@@ -117,22 +129,26 @@ def ausleihe(idBuch, idKind):
     Returns:
     - dict: A dictionary containing the result of the borrowing operation.
     """
-    rueckgabe_result = rueckgabe(idBuch, grund="neu-ausleihe")
-    
-    message = "Buch erfolgreich ausgeliehen"
-    if rueckgabe_result:
-        # Get the name of the previous borrower
-        prev_borrower_id = rueckgabe_result[0]['idKind']
-        sql = "SELECT CONCAT(Vorname, ' ', Nachnamen) AS full_name FROM kind WHERE idKind = %s;"
-        prev_borrower_name = execute_query(sql, (prev_borrower_id,))[0]['full_name']
-        message += f". Zuvor ausgeliehen von {prev_borrower_name}"
-    
-    # Insert new borrowing record
-    sql = f"INSERT INTO `{AUSLEIHE_TABLE}` (`idBuch`, `idKind`, `ausleihe`) VALUES (%s, %s, NOW());"
-    execute_query(sql, (idBuch, idKind))
-    
-    return {"message": message}
-    
+    try:
+        rueckgabe_result = rueckgabe(idBuch, grund="neu-ausleihe")
+        
+        message = "Buch erfolgreich ausgeliehen"
+        if rueckgabe_result.get("success", False):
+            # Get the name of the previous borrower
+            prev_borrower_id = rueckgabe_result["student_names"][0]
+            sql = "SELECT CONCAT(Vorname, ' ', Nachnamen) AS full_name FROM kind WHERE idKind = %s;"
+            prev_borrower_name = execute_query(sql, (prev_borrower_id,))[0]['full_name']
+            message += f". Zuvor ausgeliehen von {prev_borrower_name}"
+        
+        # Insert new borrowing record
+        sql = f"INSERT INTO `{AUSLEIHE_TABLE}` (`idBuch`, `idKind`, `ausleihe`) VALUES (%s, %s, NOW());"
+        execute_query(sql, (idBuch, idKind))
+        
+        return {"success": True, "message": message}
+    except Exception as e:
+        LOG.error(f"Error in ausleihe: {str(e)}")
+        return {"success": False, "message": f"Fehler beim Ausleihen des Buches: {str(e)}"}
+
 @app.get("/borrowed/{idKind}")
 def ausgeliehen(idKind):
     """

index.html

@@ -125,11 +125,17 @@
             fetch(`/return/${bookId}`)
                 .then(response => response.json())
                 .then(result => {
-                    if (result === 0) {
-                        setMessage(`Buch (Nummer: ${bookId}) nicht gefunden oder bereits zurückgegeben`);
-                    } else {
-                        setMessage(`Buch (Nummer: ${bookId}) erfolgreich zurückgegeben`);
+                    if (result.success) {
+                        let message = `Buch (Nummer: ${bookId}) erfolgreich zurückgegeben.`;
+                        if (result.student_names.length === 1) {
+                            message += ` Ausgeliehen von: ${result.student_names[0]}`;
+                        } else if (result.student_names.length > 1) {
+                            message += ` Ausgeliehen von: ${result.student_names.join(', ')}`;
+                        }
+                        setMessage(message);
                         document.getElementById('returnBookId').value = ''; // Clear input field
+                    } else {
+                        setMessage(result.message);
                     }
                 });
         }