diff --git a/brancheneinstufung.py b/brancheneinstufung.py index ec8953a9..9098d17a 100644 --- a/brancheneinstufung.py +++ b/brancheneinstufung.py @@ -654,11 +654,11 @@ def is_valid_wikipedia_article_url(wiki_url): def process_wiki_updates_from_chatgpt(sheet_handler, data_processor, row_limit=None): """ Identifiziert Zeilen (S nicht OK/Updated/Copied/Invalid), prüft ob U eine *valide* und *andere* Wiki-URL ist. - FALLS JA: Kopiert U->M, markiert S='X (URL Copied)', U='URL übernommen', löscht TS/Version, setzt ReEval-Flag A. - FALLS NEIN: Löscht U, markiert S='X (Invalid Suggestion)'. + - Wenn ja: Kopiert U->M, markiert S='X (URL Copied)', U='URL übernommen', löscht TS/Version, setzt ReEval-Flag A. + - Wenn nein (U keine URL, U==M, oder U ungültig): LÖSCHT den Inhalt von U und markiert S als 'X (Invalid Suggestion)'. Verarbeitet maximal row_limit Zeilen. """ - debug_print("Starte Modus: Wiki-Updates (URL-Validierung & bedingtes Update)...") # Titel angepasst + debug_print("Starte Modus: Wiki-Updates (URL-Validierung & Löschen ungültiger Vorschläge)...") if not sheet_handler.load_data(): return all_data = sheet_handler.get_all_data_with_headers() @@ -666,16 +666,33 @@ def process_wiki_updates_from_chatgpt(sheet_handler, data_processor, row_limit=N header_rows = 5 data_rows = all_data[header_rows:] - # Indizes holen - required_keys = ["Chat Wiki Konsistenzprüfung", "Chat Vorschlag Wiki Artikel", "Wiki URL", "Wikipedia Timestamp", "Wiki Verif. Timestamp", "Timestamp letzte Prüfung", "Version", "ReEval Flag"] - col_indices = {}; all_keys_found = True - for key in required_keys: idx = COLUMN_MAP.get(key); col_indices[key] = idx; if idx is None: debug_print(f"FEHLER: Key '{key}' fehlt!"); all_keys_found = False - if not all_keys_found: return debug_print("Breche Wiki-Updates ab.") + # --- Indizes holen (Korrigierte Schleife) --- + required_keys = [ + "Chat Wiki Konsistenzprüfung", "Chat Vorschlag Wiki Artikel", "Wiki URL", + "Wikipedia Timestamp", "Wiki Verif. Timestamp", "Timestamp letzte Prüfung", "Version", + "ReEval Flag" # Spalte A für ReEval-Flag + ] + col_indices = {} + all_keys_found = True + # --- KORRIGIERTE SYNTAX --- + for key in required_keys: + idx = COLUMN_MAP.get(key) + col_indices[key] = idx # Speichere den Index (kann auch None sein) + if idx is None: + debug_print(f"FEHLER: Schlüssel '{key}' für Spaltenindex fehlt in COLUMN_MAP!") + all_keys_found = False + # --- ENDE KORRIGIERTE SYNTAX --- + + if not all_keys_found: + debug_print("Breche Wiki-Updates ab, da Spaltenindizes fehlen.") + return + # --- Ende Indizes holen --- all_sheet_updates = [] processed_rows_count = 0 # Zählt alle Zeilen, die geprüft werden updated_url_count = 0 # Zählt Zeilen, wo URL kopiert wurde cleared_suggestion_count = 0 # Zählt Zeilen, wo Vorschlag gelöscht wurde + error_rows_count = 0 # Behalte Fehlerzählung bei for idx, row in enumerate(data_rows): row_num_in_sheet = idx + header_rows + 1 @@ -684,73 +701,75 @@ def process_wiki_updates_from_chatgpt(sheet_handler, data_processor, row_limit=N debug_print(f"Zeilenlimit ({row_limit}) erreicht.") break - def get_value(key): idx = col_indices.get(key); return row[idx] if idx is not None and len(row) > idx else "" - konsistenz_s = get_value("Chat Wiki Konsistenzprüfung"); vorschlag_u = get_value("Chat Vorschlag Wiki Artikel"); url_m = get_value("Wiki URL") + # --- Hilfsfunktion für sicheren Zugriff --- + def get_value(key): + index = col_indices.get(key) + # Prüfe ob Index existiert UND ob die Zeile lang genug ist + if index is not None and len(row) > index: + return row[index] + # Falls Index nicht existiert ODER Zeile zu kurz, leeren String zurückgeben + # debug_print(f"Warnung Zeile {row_num_in_sheet}: Index für '{key}' ({index}) nicht gefunden oder Zeilenlänge ({len(row)}) zu kurz.") + return "" + # --- Ende Hilfsfunktion --- - # Bedingungen für eine mögliche Aktion + konsistenz_s = get_value("Chat Wiki Konsistenzprüfung") + vorschlag_u = get_value("Chat Vorschlag Wiki Artikel") + url_m = get_value("Wiki URL") + + # Bedingungen prüfen + is_update_candidate = False; new_url = "" konsistenz_s_upper = konsistenz_s.strip().upper() vorschlag_u_cleaned = vorschlag_u.strip() url_m_cleaned = url_m.strip() - - # Prüfe nur Zeilen, deren Status eine Aktion erfordert - if konsistenz_s_upper in ["OK", "X (UPDATED)", "X (URL COPIED)", "X (INVALID SUGGESTION)", ""]: - continue # Diese Zeile braucht keine Prüfung/Aktion in diesem Modus - - processed_rows_count += 1 # Diese Zeile wird jetzt geprüft - debug_print(f"\n--- Prüfe Zeile {row_num_in_sheet} (Status S='{konsistenz_s}') ---") - - is_update_candidate = False - new_url = "" + condition1_status_nok = konsistenz_s_upper not in ["OK", "X (UPDATED)", "X (URL COPIED)", "X (INVALID SUGGESTION)", ""] condition2_u_is_url = vorschlag_u_cleaned.lower().startswith(("http://", "https://")) and "wikipedia.org/wiki/" in vorschlag_u_cleaned.lower() - condition3_u_differs_m = False - condition4_u_is_valid = False + condition3_u_differs_m = False; condition4_u_is_valid = False - if condition2_u_is_url: + if condition1_status_nok and condition2_u_is_url: new_url = vorschlag_u_cleaned condition3_u_differs_m = new_url != url_m_cleaned if condition3_u_differs_m: - debug_print(f" Prüfe Validität von URL in U: {new_url}...") - condition4_u_is_valid = is_valid_wikipedia_article_url(new_url) - is_update_candidate = condition4_u_is_valid # Kandidat nur, wenn URL valide UND anders ist + # debug_print(f"Zeile {row_num_in_sheet}: Potenzieller Kandidat. Prüfe Validität von URL: {new_url}...") # Weniger Lärm + condition4_u_is_valid = is_valid_wikipedia_article_url(new_url) # Annahme: Funktion existiert + # if not condition4_u_is_valid: debug_print(f"Zeile {row_num_in_sheet}: URL '{new_url}' ist KEIN valider Artikel.") # Weniger Lärm - # --- Aktionen basierend auf Prüfung --- - row_updates = [] # Updates für DIESE Zeile - s_l=sheet_handler._get_col_letter(col_indices["Chat Wiki Konsistenzprüfung"]+1) - u_l=sheet_handler._get_col_letter(col_indices["Chat Vorschlag Wiki Artikel"]+1) + is_update_candidate = condition1_status_nok and condition2_u_is_url and condition3_u_differs_m and condition4_u_is_valid + clear_invalid_suggestion = condition1_status_nok and not is_update_candidate + # --- Verarbeitung des Kandidaten ODER Löschen des Vorschlags --- if is_update_candidate: - # Fall A: URL ist valide und anders -> URL kopieren, Status setzen, TS löschen, ReEval setzen - debug_print(f" => Aktion: Update durchführen. Kopiere URL nach M, lösche Timestamps, setze ReEval-Flag.") - updated_url_count += 1 - m_l=sheet_handler._get_col_letter(col_indices["Wiki URL"]+1) - an_l=sheet_handler._get_col_letter(col_indices["Wikipedia Timestamp"]+1) - ax_l=sheet_handler._get_col_letter(col_indices["Wiki Verif. Timestamp"]+1) - ao_l=sheet_handler._get_col_letter(col_indices["Timestamp letzte Prüfung"]+1) - ap_l=sheet_handler._get_col_letter(col_indices["Version"]+1) - a_l=sheet_handler._get_col_letter(col_indices["ReEval Flag"]+1) - + # Fall 1: Gültiges Update durchführen + debug_print(f"Zeile {row_num_in_sheet}: Update-Kandidat VALIDIERUNG ERFOLGREICH. Setze ReEval-Flag und bereite Updates vor.") + processed_rows_count += 1 # Zähle geprüfte Zeile + updated_url_count += 1 # Zähle erfolgreiches Update + # Updates sammeln (M, S, U, Timestamps/Version löschen, A setzen) + m_l=sheet_handler._get_col_letter(col_indices["Wiki URL"]+1); s_l=sheet_handler._get_col_letter(col_indices["Chat Wiki Konsistenzprüfung"]+1); u_l=sheet_handler._get_col_letter(col_indices["Chat Vorschlag Wiki Artikel"]+1); an_l=sheet_handler._get_col_letter(col_indices["Wikipedia Timestamp"]+1); ax_l=sheet_handler._get_col_letter(col_indices["Wiki Verif. Timestamp"]+1); ao_l=sheet_handler._get_col_letter(col_indices["Timestamp letzte Prüfung"]+1); ap_l=sheet_handler._get_col_letter(col_indices["Version"]+1); a_l=sheet_handler._get_col_letter(col_indices["ReEval Flag"]+1) row_updates = [ - {'range': f'{m_l}{row_num_in_sheet}', 'values': [[new_url]]}, # Neue URL - {'range': f'{s_l}{row_num_in_sheet}', 'values': [["X (URL Copied)"]]}, # Neuer Status - {'range': f'{u_l}{row_num_in_sheet}', 'values': [["URL übernommen"]]}, # Marker - {'range': f'{an_l}{row_num_in_sheet}', 'values': [[""]]}, {'range': f'{ax_l}{row_num_in_sheet}', 'values': [[""]]}, # TS löschen - {'range': f'{ao_l}{row_num_in_sheet}', 'values': [[""]]}, {'range': f'{ap_l}{row_num_in_sheet}', 'values': [[""]]}, - {'range': f'{a_l}{row_num_in_sheet}', 'values': [["x"]]}, # ReEval setzen + {'range': f'{m_l}{row_num_in_sheet}', 'values': [[new_url]]}, + {'range': f'{s_l}{row_num_in_sheet}', 'values': [["X (URL Copied)"]]}, + {'range': f'{u_l}{row_num_in_sheet}', 'values': [["URL übernommen"]]}, + {'range': f'{an_l}{row_num_in_sheet}', 'values': [[""]]}, + {'range': f'{ax_l}{row_num_in_sheet}', 'values': [[""]]}, + {'range': f'{ao_l}{row_num_in_sheet}', 'values': [[""]]}, + {'range': f'{ap_l}{row_num_in_sheet}', 'values': [[""]]}, + {'range': f'{a_l}{row_num_in_sheet}', 'values': [["x"]]}, ] - else: - # Fall B: Status war NOK, aber U war keine valide/andere URL -> U löschen, Status anpassen - debug_print(f" => Aktion: Vorschlag in U löschen/markieren.") - cleared_suggestion_count += 1 - row_updates = [ - {'range': f'{s_l}{row_num_in_sheet}', 'values': [["X (Invalid Suggestion)"]]}, # Neuer Status - {'range': f'{u_l}{row_num_in_sheet}', 'values': [[""]]} # Leere Spalte U - ] - # Keine Timestamps löschen, kein ReEval-Flag - - # Füge die Updates für diese Zeile zur Gesamtliste hinzu - if row_updates: all_sheet_updates.extend(row_updates) + elif clear_invalid_suggestion: + # Fall 2: Ungültigen Vorschlag löschen/markieren + debug_print(f"Zeile {row_num_in_sheet}: Status S war '{konsistenz_s}', aber Vorschlag U ('{vorschlag_u_cleaned}') ist ungültig/identisch. Lösche U und setze Status S.") + processed_rows_count += 1 # Zähle geprüfte Zeile + cleared_suggestion_count += 1 + s_l=sheet_handler._get_col_letter(col_indices["Chat Wiki Konsistenzprüfung"]+1) + u_l=sheet_handler._get_col_letter(col_indices["Chat Vorschlag Wiki Artikel"]+1) + row_updates = [ + {'range': f'{s_l}{row_num_in_sheet}', 'values': [["X (Invalid Suggestion)"]]}, + {'range': f'{u_l}{row_num_in_sheet}', 'values': [[""]]} + ] + all_sheet_updates.extend(row_updates) + # Kein ReEval-Flag setzen + # --- Batch Update am Ende --- if all_sheet_updates: debug_print(f"BEREIT ZUM SENDEN: Batch-Update für {processed_rows_count} geprüfte Zeilen ({len(all_sheet_updates)} Zellen)...")