fix: correct comparison logic in compare_json.py

changes were not being handled properly for nested structures

Author: leonpwd

src/compare_json.py

@@ -20,32 +20,54 @@ def load_notes_json(filepath):
         return []
 
 def save_notes_json(data, filepath):
-    data.replace('�', 'é').replace('é', 'é').replace('è', 'è').replace('�', 'Á')
+    # La variable data est une liste, pas une string, donc on ne peut pas appeler replace dessus
     with open(filepath, "w", encoding="utf-8") as f:
         json.dump(data, f, ensure_ascii=False, indent=2)
     # Restreindre les permissions du fichier JSON (lecture/écriture propriétaire uniquement)
     os.chmod(filepath, 0o600)
 
 def find_new_notes(old_notes, new_notes):
+    """
+    Détecte les changements entre old_notes et new_notes.
+    Retourne une liste de changements : [matiere, section, note, ponderation]
+    
+    Détecte :
+    - Nouvelles notes ajoutées
+    - Notes modifiées
+    - Pondérations modifiées
+    """
     changes = []
     old_map = {(m['matiere'], m['coef']): m for m in old_notes}
+    
     for new_matiere in new_notes:
         key = (new_matiere['matiere'], new_matiere['coef'])
         old_matiere = old_map.get(key)
+        
         if not old_matiere:
+            # Nouvelle matière, ignorer (on ne notifie que les notes)
             continue
+        
         for section in ["Projet", "Contrôle Continu", "Examen"]:
             old_section = old_matiere['sections'].get(section, [])
             new_section = new_matiere['sections'].get(section, [])
+            
             for idx, new_block in enumerate(new_section):
                 old_block = old_section[idx] if idx < len(old_section) else None
                 new_notes_list = new_block.get("notes", [])
                 old_notes_list = old_block.get("notes", []) if old_block else []
-                for n_idx, new_note in enumerate(new_notes_list):
-                    old_note = old_notes_list[n_idx] if n_idx < len(old_notes_list) else None
-                    if old_note != new_note:
+                
+                # Créer un ensemble des notes existantes pour comparaison
+                old_notes_set = {(n.get("note", ""), n.get("pondération", "")) for n in old_notes_list}
+                
+                # Parcourir toutes les nouvelles notes
+                for new_note in new_notes_list:
+                    new_note_tuple = (new_note.get("note", ""), new_note.get("pondération", ""))
+                    
+                    # Si la note n'existe pas dans old_notes_set, c'est un changement
+                    if new_note_tuple not in old_notes_set:
                         matiere_tronquee = new_matiere['matiere'].split('/')[0].strip()
                         note = new_note.get("note", "")
                         ponderation = new_note.get("pondération", "")
                         changes.append([matiere_tronquee, section, note, ponderation])
+    
     return changes