Mise à jour du projet

2026-04-02 22:30:40 +02:00
parent 98825db072
commit 085cf33114
6 changed files with 448 additions and 588 deletions
--- a/server/db/database.js
+++ b/server/db/database.js
@@ -1,8 +1,10 @@
 /**
 * ============================================
 * DATABASE.JS - Gestion MariaDB
- * Smart Parking v2.0
- * AJOUTÉ : expireReservations() — libère auto les places
+ * Smart Parking v3.0
+ * AJOUTÉ : checkReservationConflict()
+ *          → vérifie les conflits d'horaire
+ *            au lieu de bloquer toute la place
 * ============================================
 */

@@ -123,7 +125,7 @@ async function initDatabase() {
                    [i, `SENSOR_${String(i).padStart(3, '0')}`, 'free']
                );
            }
-            console.log('✅ 10 places créées (toutes libres)');
+            console.log('✅ 10 places créées');
        }

    } catch (err) {
@@ -227,6 +229,34 @@ async function createReservation(userId, spotId, date, startTime, endTime, durat
    return { id: result.insertId };
 }

+/**
+ * ⭐ NOUVELLE FONCTION — Vérification des conflits d'horaire
+ *
+ * Problème corrigé : avant, quand une place était réservée,
+ * elle restait bloquée pour TOUS les jours et TOUTES les heures.
+ *
+ * Maintenant on vérifie uniquement s'il y a une réservation
+ * qui se chevauche sur la MÊME date et le MÊME créneau horaire.
+ *
+ * Exemple :
+ *   Place 2 réservée aujourd'hui 10h-11h ✅
+ *   Place 2 réservée aujourd'hui 14h-15h ✅ (pas de conflit)
+ *   Place 2 réservée demain 10h-11h       ✅ (pas de conflit)
+ *   Place 2 réservée aujourd'hui 10h30-11h30 ❌ (conflit !)
+ */
+async function checkReservationConflict(spotId, date, startTime, endTime) {
+    const [rows] = await pool.query(`
+        SELECT id FROM reservations
+        WHERE spot_id = ?
+          AND date = ?
+          AND status IN ('active', 'pending')
+          AND start_time < ?
+          AND end_time > ?
+    `, [spotId, date, endTime, startTime]);
+
+    return rows.length > 0;  // true = conflit, false = créneau libre
+}
+
 async function getReservationById(id) {
    const [rows] = await pool.query(
        `SELECT r.*, s.number AS spot_number
@@ -270,16 +300,10 @@ async function updateReservationStatus(id, status) {
 }

 /**
- * ⭐ NOUVELLE FONCTION — Expiration automatique des réservations
- *
- * Cherche toutes les réservations actives dont la date+heure de fin
- * est déjà dépassée, les passe en "completed" et libère les places.
- *
- * Appelée toutes les 60 secondes par server.js.
- * Cela résout le problème des places qui restent "réservées" indéfiniment.
+ * Expiration automatique des réservations
+ * Appelée toutes les 60 secondes par server.js
 */
 async function expireReservations() {
-    // Trouver les réservations actives dont l'heure de fin est passée
    const [expiredRows] = await pool.query(`
        SELECT r.id, r.spot_id, r.user_id, s.number AS spot_number
        FROM reservations r
@@ -289,20 +313,14 @@ async function expireReservations() {
    `);

    for (const res of expiredRows) {
-        // Passer la réservation en "completed"
        await pool.query(
            "UPDATE reservations SET status = 'completed' WHERE id = ?",
            [res.id]
        );
-
-        // Libérer la place (la passer en "free")
-        // (le capteur Arduino prendra le relais ensuite si une voiture est encore là)
        await pool.query(
            "UPDATE spots SET status = 'free', last_update = NOW() WHERE id = ?",
            [res.spot_id]
        );
-
-        // Ajouter à l'historique
        await pool.query(
            'INSERT INTO history (action, details, user_id) VALUES (?, ?, ?)',
            [
@@ -364,7 +382,7 @@ async function getStats(days = 7) {
 }

 // ============================================
-// MQTT EVENTS
+// MQTT
 // ============================================

 async function recordMqttEvent(topic, message) {
@@ -388,18 +406,13 @@ module.exports = {
    pool,
    initDatabase,
    closeDatabase,
-    // Utilisateurs
    createUser, getUserByEmail, getUserById, getAllUsers, updateUser, deleteUser,
-    // Places
    createSpot, getAllSpots, getSpotById, updateSpotStatus, deleteAllSpots,
-    // Réservations
-    createReservation, getReservationById, getReservationsByUser,
+    createReservation, checkReservationConflict,
+    getReservationById, getReservationsByUser,
    getAllReservations, updateReservationStatus,
-    expireReservations,   // ← NOUVEAU
-    // Historique
+    expireReservations,
    addHistory, getHistory,
-    // Stats
    recordStats, getStats,
-    // MQTT
    recordMqttEvent
 };
--- a/server/routes/api.js
+++ b/server/routes/api.js
@@ -1,9 +1,10 @@
 /**
 * ============================================
 * API ROUTES - Routes de l'API REST
- * Smart Parking - BTS CIEL IR
- * CORRIGÉ : annulation libère bien la place
- *           ajout route /complete pour l'admin
+ * Smart Parking v3.0
+ * CORRIGÉ : réservation vérifie les conflits
+ *           d'horaire au lieu de bloquer la place
+ *           définitivement
 * ============================================
 */

@@ -110,11 +111,7 @@ router.post('/spots/init', authenticateToken, requireAdmin, async (req, res) =>
        const spotCount = Math.min(Math.max(parseInt(req.body.count) || 10, 5), 50);
        await db.deleteAllSpots();
        for (let i = 1; i <= spotCount; i++) {
-            let status = 'free';
-            const rand = Math.random();
-            if (rand > 0.85) status = 'reserved';
-            else if (rand > 0.60) status = 'occupied';
-            await db.createSpot(i, `SENSOR_${String(i).padStart(3, '0')}`, status);
+            await db.createSpot(i, `SENSOR_${String(i).padStart(3, '0')}`, 'free');
        }
        await db.addHistory('Réinitialisation places', `${spotCount} places créées`, req.user.id);
        res.json({ success: true, message: `${spotCount} places créées` });
@@ -145,21 +142,71 @@ router.get('/reservations/all', authenticateToken, requireAdmin, async (req, res
    }
 });

+/**
+ * POST /api/reservations
+ *
+ * CORRIGÉ : on ne bloque plus la place entière définitivement.
+ * On vérifie uniquement s'il y a un CONFLIT d'horaire sur
+ * la même date et le même créneau.
+ *
+ * Exemple de ce qui est maintenant possible :
+ *   Place 2 — 10h-11h aujourd'hui    ✅
+ *   Place 2 — 14h-15h aujourd'hui    ✅ (pas de conflit)
+ *   Place 2 — 10h-11h demain         ✅ (pas de conflit)
+ *   Place 2 — 10h30-11h30 aujourd'hui ❌ (conflit !)
+ */
 router.post('/reservations', authenticateToken, async (req, res) => {
    try {
        const { spotId, date, startTime, endTime, duration, vehicle, price } = req.body;
+
        if (!spotId || !date || !startTime || !endTime || !duration || !price)
            return res.status(400).json({ success: false, message: 'Tous les champs sont requis' });
+
        const spot = await db.getSpotById(spotId);
-        if (!spot || spot.status !== 'free')
-            return res.status(409).json({ success: false, message: "Cette place n'est plus disponible" });
+        if (!spot)
+            return res.status(404).json({ success: false, message: 'Place introuvable' });
+
+        // CORRIGÉ : bloquer uniquement si une voiture est physiquement là
+        if (spot.status === 'occupied')
+            return res.status(409).json({ success: false, message: "Une voiture est déjà sur cette place" });
+
+        // CORRIGÉ : vérifier les conflits d'horaire au lieu du statut global
+        const conflict = await db.checkReservationConflict(spotId, date, startTime, endTime);
+        if (conflict)
+            return res.status(409).json({
+                success: false,
+                message: `Cette place est déjà réservée sur ce créneau. Choisissez un autre horaire ou une autre date.`
+            });
+
        const paymentCode = 'PARK' + Date.now().toString().slice(-8);
        const result = await db.createReservation(
            req.user.id, spotId, date, startTime, endTime, duration, vehicle, price, paymentCode
        );
-        await db.updateSpotStatus(spotId, 'reserved');
-        await db.addHistory('Nouvelle réservation', `Place ${spot.number} - ${price}EUR`, req.user.id);
-        res.status(201).json({ success: true, message: 'Réservation créée', data: { id: result.id, paymentCode } });
+
+        // On ne change le statut de la place QUE si la réservation est pour aujourd'hui
+        // et que l'heure de début est maintenant ou dans moins de 30 minutes
+        const now      = new Date();
+        const today    = now.toISOString().split('T')[0];
+        const resStart = new Date(`${date}T${startTime}`);
+        const diffMin  = (resStart - now) / 60000;
+
+        if (date === today && diffMin <= 30) {
+            await db.updateSpotStatus(spotId, 'reserved');
+        }
+        // Pour une réservation future, le statut de la place reste inchangé
+        // Le timer d'expiration (server.js) le mettra à jour au bon moment
+
+        await db.addHistory(
+            'Nouvelle réservation',
+            `Place ${spot.number} réservée le ${date} de ${startTime} à ${endTime} — ${price}EUR`,
+            req.user.id
+        );
+
+        res.status(201).json({
+            success: true,
+            message: 'Réservation créée',
+            data: { id: result.id, paymentCode }
+        });
    } catch (err) {
        console.error('❌ Erreur create reservation:', err.message);
        res.status(500).json({ success: false, message: 'Erreur serveur' });
@@ -168,7 +215,6 @@ router.post('/reservations', authenticateToken, async (req, res) => {

 /**
 * PUT /api/reservations/:id/cancel
- * CORRIGÉ : libère désormais la place associée
 */
 router.put('/reservations/:id/cancel', authenticateToken, async (req, res) => {
    try {
@@ -179,10 +225,10 @@ router.put('/reservations/:id/cancel', authenticateToken, async (req, res) => {
            return res.status(403).json({ success: false, message: 'Accès refusé' });

        await db.updateReservationStatus(req.params.id, 'cancelled');
-        await db.updateSpotStatus(reservation.spot_id, 'free');   // ← BUG CORRIGÉ ICI
+        await db.updateSpotStatus(reservation.spot_id, 'free');
        await db.addHistory(
            'Annulation réservation',
-            `Reservation #${req.params.id} annulee - place ${reservation.spot_id} liberee`,
+            `Réservation #${req.params.id} annulée — place ${reservation.spot_number} libérée`,
            req.user.id
        );
        res.json({ success: true, message: 'Réservation annulée' });
@@ -193,7 +239,7 @@ router.put('/reservations/:id/cancel', authenticateToken, async (req, res) => {
 });

 /**
- * PUT /api/reservations/:id/complete  (admin uniquement)
+ * PUT /api/reservations/:id/complete (admin)
 */
 router.put('/reservations/:id/complete', authenticateToken, requireAdmin, async (req, res) => {
    try {
@@ -205,7 +251,7 @@ router.put('/reservations/:id/complete', authenticateToken, requireAdmin, async
        await db.updateSpotStatus(reservation.spot_id, 'free');
        await db.addHistory(
            'Réservation terminée',
-            `Reservation #${req.params.id} terminee - place ${reservation.spot_id} liberee`,
+            `Réservation #${req.params.id} terminée — place ${reservation.spot_number} libérée`,
            req.user.id
        );
        res.json({ success: true, message: 'Réservation terminée' });
@@ -221,11 +267,11 @@ router.put('/reservations/:id/complete', authenticateToken, requireAdmin, async

 router.get('/stats', authenticateToken, async (req, res) => {
    try {
-        const spots = await db.getAllSpots();
-        const total = spots.length;
-        const free  = spots.filter(s => s.status === 'free').length;
-        const occupied = spots.filter(s => s.status === 'occupied').length;
-        const reserved = spots.filter(s => s.status === 'reserved').length;
+        const spots        = await db.getAllSpots();
+        const total        = spots.length;
+        const free         = spots.filter(s => s.status === 'free').length;
+        const occupied     = spots.filter(s => s.status === 'occupied').length;
+        const reserved     = spots.filter(s => s.status === 'reserved').length;
        const occupancyRate = total > 0 ? Math.round(((occupied + reserved) / total) * 100) : 0;
        res.json({ success: true, data: { total, free, occupied, reserved, occupancyRate } });
    } catch (err) {
@@ -244,7 +290,7 @@ router.get('/history', authenticateToken, requireAdmin, async (req, res) => {
 });

 router.get('/status', (_req, res) => {
-    res.json({ success: true, message: 'Smart Parking API operationnelle', version: '1.0.0', timestamp: new Date().toISOString() });
+    res.json({ success: true, message: 'Smart Parking API opérationnelle', version: '3.0.0', timestamp: new Date().toISOString() });
 });

 module.exports = router;