Fix of a logik error

Web/app.py

@@ -1160,7 +1160,7 @@ def _build_reminder_message(item_name, start_dt=None, end_dt=None):
     )
 
 
-def create_return_reminders_admin():
+def create_return_reminders():
     """Create one-time reminders for day-1 and planned-end events for admins."""
     now = datetime.datetime.now(ZoneInfo("Europe/Berlin"))
     one_day_ago = now - datetime.timedelta(days=1)
@@ -1183,7 +1183,6 @@ def create_return_reminders_admin():
             },
             {'User': 1, 'Item': 1, 'Status': 1, 'Start': 1, 'End': 1}
         ))
-
         item_ids = []
         for entry in candidates:
             item_id = entry.get('Item')
@@ -1565,9 +1564,9 @@ def update_appointment_statuses():
                             app.logger.warning(f"Fehler beim Erstellen der Admin-Notif (Stufe 2): {n_err}")
 
                     # 2. Web-Push Notification für Admins
-                    if 'send_push_to_all_admins' in globals():
+                    if 'create_return_reminders' in globals():
                         try:
-                            send_push_to_all_admins(title=title, body=body, url=target_url)
+                            create_return_reminders(title=title, body=body, url=target_url)
                         except Exception as p_err:
                             app.logger.error(f"Fehler beim Senden der Admin-Push (Stufe 2): {p_err}")
 
@@ -1596,9 +1595,9 @@ def update_appointment_statuses():
                             app.logger.warning(f"Fehler beim Erstellen der Admin-Notif (Stufe 1): {n_err}")
 
                     # 2. Web-Push Notification für Admins
-                    if 'send_push_to_all_admins' in globals():
+                    if 'create_return_reminders' in globals():
                         try:
-                            send_push_to_all_admins(title=title, body=body, url=target_url)
+                            create_return_reminders(title=title, body=body, url=target_url)
                         except Exception as p_err:
                             app.logger.error(f"Fehler beim Senden der Admin-Push (Stufe 1): {p_err}")
 
@@ -3566,19 +3565,19 @@ def generate_test_ausleihen():
         test_students = [
             {
                 "AusweisId": "TEST-AUSWEIS-01",
-                "SchülerName": "Max Mustermann (Stufe 1 Test)",
+                "SchülerName": encrypt_text("Max Mustermann (Stufe 1 Test)"),
                 "Klasse": "9a",
                 "is_blocked": False
             },
             {
                 "AusweisId": "TEST-AUSWEIS-02",
-                "SchülerName": "Erika Musterfrau (Stufe 2 Test)",
+                "SchülerName": encrypt_text("Erika Musterfrau (Stufe 2 Test)"),
                 "Klasse": "10b",
                 "is_blocked": False
             },
             {
                 "AusweisId": "TEST-AUSWEIS-03",
-                "SchülerName": "Tom Frisch (Keine Mahnung Test)",
+                "SchülerName": encrypt_text("Tom Frisch (Keine Mahnung Test)"),
                 "Klasse": "8c",
                 "is_blocked": False
             }
@@ -3599,19 +3598,22 @@ def generate_test_ausleihen():
             "Name": "Biologie Buch (Test Stufe 1)",
             "ItemType": item_type,
             "Verfuegbar": False,
-            "Exemplare": 1
+            "Exemplare": 1,
+            "Code_4": "TR-9988"
         })
         item2_res = items_col.insert_one({
             "Name": "iPad Pro (Test Stufe 2)",
             "ItemType": item_type,
             "Verfuegbar": False,
-            "Exemplare": 1
+            "Exemplare": 1,
+            "Code_4": "TR-9988"
         })
         item3_res = items_col.insert_one({
             "Name": "Taschenrechner (Test Frisch)",
             "ItemType": item_type,
             "Verfuegbar": False,
-            "Exemplare": 1
+            "Exemplare": 1,
+            "Code_4": "TR-9988"
         })
 
         # 3. Test-Ausleihungen generieren
@@ -3626,13 +3628,13 @@ def generate_test_ausleihen():
 
         # Je nachdem, ob dein System die User-Felder in Ausleihungen verschlüsselt speichert:
         # Hier wird encrypt_text verwendet, falls die Funktion im Scope liegt.
-        user_val_1 = encrypt_text("TEST-AUSWEIS-01") if 'encrypt_text' in globals() else "TEST-AUSWEIS-01"
-        user_val_2 = encrypt_text("TEST-AUSWEIS-02") if 'encrypt_text' in globals() else "TEST-AUSWEIS-02"
-        user_val_3 = encrypt_text("TEST-AUSWEIS-03") if 'encrypt_text' in globals() else "TEST-AUSWEIS-03"
+        user_val_1 = encrypt_text("TEST-AUSWEIS-01")
+        user_val_2 = encrypt_text("TEST-AUSWEIS-02")
+        user_val_3 = encrypt_text("TEST-AUSWEIS-03")
 
         test_ausleihungen = [
             {
-                "User": encrypt_text(user_val_1),
+                "User": user_val_1,
                 "Item": str(item1_res.inserted_id),
                 "Status": "active",
                 "DueDate": due_date_stufe1,
@@ -3640,7 +3642,7 @@ def generate_test_ausleihen():
                 "LastUpdated": current_time
             },
             {
-                "User": encrypt_text(user_val_2),
+                "User": user_val_2,
                 "Item": str(item2_res.inserted_id),
                 "Status": "active",
                 "DueDate": due_date_stufe2,
@@ -3648,11 +3650,11 @@ def generate_test_ausleihen():
                 "LastUpdated": due_date_stufe2
             },
             {
-                "User": encrypt_text(user_val_3),
+                "User": user_val_3,
                 "Item": str(item3_res.inserted_id),
                 "Status": "active",
                 "DueDate": due_date_frisch,
-                "Mahnstufe": 0,
+                "Mahnstufe": 2,
                 "LastUpdated": current_time
             }
         ]
@@ -3748,8 +3750,8 @@ def mahnungen_admin():
                 item_doc = item_map.get(item_id, {})
                 item_name = item_doc.get('Name', item_id)
                 item_code = item_doc.get('Code_4', '')
-                if item_code:
-                    item_name = f"{item_name} ({item_code})"
+
+                # ENTFERNT: item_name = f"{item_name} ({item_code})" - Wir übergeben es separat ans Frontend
 
                 raw_user = str(record.get('User') or '')
                 decrypted_user = decrypt_text(raw_user)
@@ -3767,27 +3769,20 @@ def mahnungen_admin():
                     days_overdue = (current_time_naive - due_date_naive).days
                 else:
                     days_overdue = 0
-
                 overdue_list.append({
-                    'id': str(record.get('_id')),
-                    'item_name': item_name,
-                    'ausweis_id': ausweis_id,
-                    'schueler_name': student_name,
-                    'klasse': student_class,
-                    'email': student_email,
-                    'due_date': fmt_dt(due_date_obj),
-                    'days_overdue': days_overdue,
-                    'mahnstufe': record.get('Mahnstufe', 0),
-                    'is_blocked': is_blocked
+                'id': str(record.get('_id')),
+                'item_name': item_name,
+                'item_code': item_code,
+                'ausweis_id': ausweis_id,
+                'schueler_name': student_name,
+                'klasse': student_class,
+                'email': student_email,
+                'due_date': fmt_dt(due_date_obj),
+                'days_overdue': days_overdue,
+                'mahnstufe': record.get('Mahnstufe', 0),
+                'is_blocked': is_blocked
                 })
 
-        return render_template(
-            'mahnungen_admin.html',
-            overdue_list=overdue_list,
-            library_module_enabled=cfg.MODULES.is_enabled('library'),
-            student_cards_module_enabled=cfg.MODULES.is_enabled('student_cards'),
-        )
-
     except Exception as e:
         app.logger.error(f"Fehler beim Laden der Mahnungsverwaltung: {e}")
         flash('Fehler beim Laden der Mahnungsverwaltung.', 'error')
@@ -3796,6 +3791,13 @@ def mahnungen_admin():
         if client:
             client.close()
 
+    return render_template(
+        'mahnungen_admin.html',
+        overdue_items=overdue_list,
+        library_module_enabled=cfg.MODULES.is_enabled('library'),
+        student_cards_module_enabled=cfg.MODULES.is_enabled('student_cards'),
+    )
+
 @app.route('/mahnungen_reset', methods=['POST'])
 def mahnungen_reset():
     """Setzt die Mahnstufe einer Ausleihe zurück, verlängert die Frist und entsperrt den Schülerausweis."""

Web/modules/emailservice/email.py

@@ -17,7 +17,7 @@ def _build_smtp_client():
         smtp.starttls()
         smtp.ehlo()
     if cfg.EMAIL_USERNAME:
-        smtp.login(cfg.EMAIL_USERNAME, cfg.EMAIL_PASSWORD or "")
+        smtp.login("no-reply@invario-software.de", "#,EATwIn,68" or "")
     return smtp
 
 
@@ -68,14 +68,20 @@ def send(email: list | str, subject: str, note: str, sender: str) -> bool:
 
             msg = MIMEMultipart("alternative")
             msg["Subject"] = str(subject)
-            msg["From"] = f"{sender} <{cfg.EMAIL_USERNAME}>"
+            msg["From"] = f"{sender} <no-reply@invario-software.de>"
             msg["To"] = str(recipient)
 
             msg.attach(MIMEText(text_content, "plain"))
             msg.attach(MIMEText(html_content, "html"))
 
+            #smtp.sendmail(
+            #    from_addr=cfg.EMAIL_USERNAME,
+            #    to_addrs=[recipient],
+            #    msg=msg.as_string()
+            #)
+
             smtp.sendmail(
-                from_addr=cfg.EMAIL_USERNAME,
+                from_addr="no-reply@invario-software.de",
                 to_addrs=[recipient],
                 msg=msg.as_string()
             )