mirror of
https://github.com/R0m1k3/noteflow.git
synced 2026-10-22 14:48:18 +02:00
Problème identifié : - Quand une nouvelle note était créée, elle écrasait la première note existante - La fonction runQuery() dans database-postgres.js retournait rowCount (1) au lieu de l'ID réel - Les requêtes INSERT n'utilisaient pas RETURNING pour récupérer l'ID généré Solution implémentée : 1. Ajout de RETURNING * à toutes les requêtes INSERT dans : - routes/notes.routes.js (notes, todos, images, fichiers, tags) - routes/todos.routes.js (todos globaux) - routes/users.routes.js (utilisateurs) - routes/rss.routes.js (flux RSS, résumés) - routes/settings.routes.js (paramètres) - routes/calendar.routes.js (événements, tokens OAuth) 2. Modification de runQuery() pour : - Retourner la ligne complète si RETURNING est présent - Inclure tous les champs de la ligne insérée (spread operator) - Ajouter des logs détaillés pour le debugging 3. Ajout de logs détaillés pour tracer : - Chaque création d'entité avec ses paramètres - L'ID retourné après insertion - Les erreurs potentielles Impact : - Les nouvelles notes ont maintenant leur vrai ID auto-incrémenté - Plus d'écrasement des notes existantes - Meilleure traçabilité avec les logs détaillés
172 lines
5.3 KiB
JavaScript
172 lines
5.3 KiB
JavaScript
// Routes de gestion des todos globaux (sidebar)
|
|
const express = require('express');
|
|
const router = express.Router();
|
|
const { body, validationResult } = require('express-validator');
|
|
|
|
const { getAll, getOne, runQuery } = require('../config/database');
|
|
const { authenticateToken } = require('../middleware/auth');
|
|
const logger = require('../config/logger');
|
|
|
|
// Toutes les routes nécessitent authentification
|
|
router.use(authenticateToken);
|
|
|
|
/**
|
|
* GET /api/todos
|
|
* Liste tous les todos globaux de l'utilisateur
|
|
*/
|
|
router.get('/', async (req, res) => {
|
|
try {
|
|
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]);
|
|
|
|
res.json(todos);
|
|
} catch (error) {
|
|
logger.error('Erreur lors de la récupération des todos:', error);
|
|
res.status(500).json({ error: 'Erreur serveur' });
|
|
}
|
|
});
|
|
|
|
/**
|
|
* POST /api/todos
|
|
* Créer un nouveau todo global
|
|
*/
|
|
router.post('/',
|
|
[body('text').trim().notEmpty().withMessage('Le texte est requis')],
|
|
async (req, res) => {
|
|
try {
|
|
const errors = validationResult(req);
|
|
if (!errors.isEmpty()) {
|
|
return res.status(400).json({ error: 'Données invalides', details: errors.array() });
|
|
}
|
|
|
|
const { text } = req.body;
|
|
|
|
logger.info(`[CREATE GLOBAL TODO] Début création - user_id: ${req.user.id}, text: "${text}"`);
|
|
|
|
const result = await runQuery(`
|
|
INSERT INTO global_todos (user_id, text)
|
|
VALUES ($1, $2)
|
|
RETURNING *
|
|
`, [req.user.id, text]);
|
|
|
|
logger.info(`[CREATE GLOBAL TODO] Todo global créé avec succès - ID: ${result.id}, user: ${req.user.username}`);
|
|
|
|
res.status(201).json({
|
|
id: result.id,
|
|
text,
|
|
completed: false,
|
|
created_at: new Date().toISOString()
|
|
});
|
|
} catch (error) {
|
|
logger.error('Erreur lors de la création du todo:', error);
|
|
res.status(500).json({ error: 'Erreur serveur' });
|
|
}
|
|
}
|
|
);
|
|
|
|
/**
|
|
* PUT /api/todos/:id
|
|
* Modifier un todo global
|
|
*/
|
|
router.put('/:id',
|
|
[
|
|
body('text').optional().trim().notEmpty(),
|
|
body('completed').optional().isBoolean()
|
|
],
|
|
async (req, res) => {
|
|
try {
|
|
const errors = validationResult(req);
|
|
if (!errors.isEmpty()) {
|
|
return res.status(400).json({ error: 'Données invalides', details: errors.array() });
|
|
}
|
|
|
|
const { text, completed } = req.body;
|
|
|
|
// Vérifier que le todo appartient à l'utilisateur
|
|
const todo = await getOne('SELECT id 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 updates = [];
|
|
const params = [];
|
|
let paramCount = 0;
|
|
|
|
if (text !== undefined) {
|
|
paramCount++;
|
|
updates.push(`text = $${paramCount}`);
|
|
params.push(text);
|
|
}
|
|
if (completed !== undefined) {
|
|
paramCount++;
|
|
updates.push(`completed = $${paramCount}`);
|
|
params.push(completed ? 1 : 0);
|
|
}
|
|
|
|
if (updates.length > 0) {
|
|
paramCount++;
|
|
params.push(req.params.id);
|
|
await runQuery(`UPDATE global_todos SET ${updates.join(', ')} WHERE id = $${paramCount}`, params);
|
|
}
|
|
|
|
res.json({ message: 'Todo modifié avec succès' });
|
|
} catch (error) {
|
|
logger.error('Erreur lors de la modification du todo:', error);
|
|
res.status(500).json({ error: 'Erreur serveur' });
|
|
}
|
|
}
|
|
);
|
|
|
|
/**
|
|
* PATCH /api/todos/:id/toggle
|
|
* Basculer l'état completed d'un todo
|
|
*/
|
|
router.patch('/:id/toggle', async (req, res) => {
|
|
try {
|
|
// Vérifier que le todo appartient à l'utilisateur
|
|
const todo = await getOne('SELECT id, completed 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 newCompleted = todo.completed ? 0 : 1;
|
|
await runQuery('UPDATE global_todos SET completed = $1 WHERE id = $2', [newCompleted, req.params.id]);
|
|
|
|
logger.info(`Todo global ${newCompleted ? 'complété' : 'réouvert'} (ID: ${req.params.id}) par ${req.user.username}`);
|
|
|
|
res.json({ message: 'Todo modifié avec succès', completed: newCompleted === 1 });
|
|
} catch (error) {
|
|
logger.error('Erreur lors du toggle du todo:', error);
|
|
res.status(500).json({ error: 'Erreur serveur' });
|
|
}
|
|
});
|
|
|
|
/**
|
|
* DELETE /api/todos/:id
|
|
* Supprimer un todo global
|
|
*/
|
|
router.delete('/:id', async (req, res) => {
|
|
try {
|
|
// Vérifier que le todo appartient à l'utilisateur
|
|
const todo = await getOne('SELECT id 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é' });
|
|
}
|
|
|
|
await runQuery('DELETE FROM global_todos WHERE id = $1', [req.params.id]);
|
|
|
|
logger.info(`Todo global supprimé (ID: ${req.params.id}) par ${req.user.username}`);
|
|
|
|
res.json({ message: 'Todo supprimé avec succès' });
|
|
} catch (error) {
|
|
logger.error('Erreur lors de la suppression du todo:', error);
|
|
res.status(500).json({ error: 'Erreur serveur' });
|
|
}
|
|
});
|
|
|
|
module.exports = router;
|