From 941493cb16833446770db33eb706e0c57e09ae37 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Fri, 21 Nov 2025 14:17:11 +0000
Subject: [PATCH] =?UTF-8?q?Fix:=20R=C3=A9trocompatibilit=C3=A9=20pour=20le?=
 =?UTF-8?q?=20champ=20priority?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Rend le code compatible avec les bases de données qui n'ont pas encore
exécuté la migration priority.

Changements :
- Route GET /api/todos utilise un fallback si priority n'existe pas
- Route PATCH /api/todos/:id/priority retourne un message explicite
- Les tâches s'affichent même sans migration
- Message clair pour guider l'utilisateur vers la migration

L'application fonctionne maintenant avec ou sans le champ priority.
---
 routes/todos.routes.js | 63 ++++++++++++++++++++++++++++++------------
 1 file changed, 46 insertions(+), 17 deletions(-)

diff --git a/routes/todos.routes.js b/routes/todos.routes.js
index c89d892..501ecb5 100644
--- a/routes/todos.routes.js
+++ b/routes/todos.routes.js
@@ -16,14 +16,31 @@ router.use(authenticateToken);
  */
 router.get('/', async (req, res) => {
   try {
-    const todos = await getAll(`
-      SELECT id, text, completed, priority, created_at
-      FROM global_todos
-      WHERE user_id = $1
-      ORDER BY priority DESC, created_at DESC
-    `, [req.user.id]);
+    // Essayer d'abord avec le champ priority (nouvelle version)
+    try {
+      const todos = await getAll(`
+        SELECT id, text, completed, priority, created_at
+        FROM global_todos
+        WHERE user_id = $1
+        ORDER BY priority DESC, created_at DESC
+      `, [req.user.id]);
 
-    res.json(todos);
+      res.json(todos);
+      return;
+    } catch (priorityError) {
+      // Si le champ priority n'existe pas, utiliser l'ancienne requête
+      logger.info('Champ priority non trouvé, utilisation de la requête sans priority');
+      const todos = await getAll(`
+        SELECT id, text, completed, created_at
+        FROM global_todos
+        WHERE user_id = $1
+        ORDER BY created_at DESC
+      `, [req.user.id]);
+
+      // Ajouter priority=false par défaut pour compatibilité frontend
+      const todosWithPriority = todos.map(t => ({ ...t, priority: false }));
+      res.json(todosWithPriority);
+    }
   } catch (error) {
     logger.error('Erreur lors de la récupération des todos:', error);
     res.status(500).json({ error: 'Erreur serveur' });
@@ -159,17 +176,29 @@ router.patch('/:id/toggle', async (req, res) => {
 router.patch('/:id/priority', async (req, res) => {
   try {
     // Vérifier que le todo appartient à l'utilisateur
-    const todo = await getOne('SELECT id, priority FROM global_todos WHERE id = $1 AND user_id = $2', [req.params.id, req.user.id]);
-    if (!todo) {
-      return res.status(404).json({ error: 'Todo non trouvé' });
+    try {
+      const todo = await getOne('SELECT id, priority FROM global_todos WHERE id = $1 AND user_id = $2', [req.params.id, req.user.id]);
+      if (!todo) {
+        return res.status(404).json({ error: 'Todo non trouvé' });
+      }
+
+      const newPriority = todo.priority ? 0 : 1;
+      await runQuery('UPDATE global_todos SET priority = $1 WHERE id = $2', [newPriority, req.params.id]);
+
+      logger.info(`Todo global ${newPriority ? 'marqué prioritaire' : 'démarqué prioritaire'} (ID: ${req.params.id}) par ${req.user.username}`);
+
+      res.json({ message: 'Priorité modifiée avec succès', priority: newPriority === 1 });
+    } catch (priorityError) {
+      // Si le champ priority n'existe pas encore, retourner un message d'erreur explicite
+      if (priorityError.message && priorityError.message.includes('priority')) {
+        logger.warn('Tentative d\'utilisation de la fonctionnalité priority sans migration');
+        return res.status(400).json({
+          error: 'Fonctionnalité priority non disponible',
+          message: 'Veuillez exécuter la migration: npm run db:migrate:priority'
+        });
+      }
+      throw priorityError;
     }
-
-    const newPriority = todo.priority ? 0 : 1;
-    await runQuery('UPDATE global_todos SET priority = $1 WHERE id = $2', [newPriority, req.params.id]);
-
-    logger.info(`Todo global ${newPriority ? 'marqué prioritaire' : 'démarqué prioritaire'} (ID: ${req.params.id}) par ${req.user.username}`);
-
-    res.json({ message: 'Priorité modifiée avec succès', priority: newPriority === 1 });
   } catch (error) {
     logger.error('Erreur lors du toggle de la priorité:', error);
     res.status(500).json({ error: 'Erreur serveur' });