cannot fix issue with elapsed timer on admin dashboard. fixing issues with QR code editing.

Grey = cant click
Red = current
Green = complete
blue = skipped?

app/api/admin/clues/[clueId]/route.js

@@ -1,5 +1,5 @@
 import { getToken } from "next-auth/jwt";
-import { queryOne, run } from "@/lib/db";
+import { queryOne, query, run } from "@/lib/db";
 
 export const dynamic = "force-dynamic"; // Disable caching
 
@@ -32,39 +32,120 @@ export async function DELETE(request, context) {
       return Response.json({ error }, { status });
     }
 
-    const { clueId } = context.params;
+    // Get clueId parameter - properly awaited
+    const { clueId } = await context.params;
 
     // Validate clueId is a number
     if (isNaN(parseInt(clueId))) {
       return Response.json({ error: "Invalid clue ID" }, { status: 400 });
     }
 
-    // Check if clue exists
-    const clue = await queryOne("SELECT id FROM clues WHERE id = ?", [clueId]);
+    // Always set shouldRenumber to true - no longer read from request body
+    const shouldRenumber = true;
+
+    // Check if clue exists and get its QR code value
+    const clue = await queryOne("SELECT id, qr_code FROM clues WHERE id = ?", [
+      clueId,
+    ]);
     if (!clue) {
       return Response.json({ error: "Clue not found" }, { status: 404 });
     }
 
-    // Delete clue record
-    await run("DELETE FROM clues WHERE id = ?", [clueId]);
+    console.log(
+      `Starting deletion of clue ${clueId} with QR code ${clue.qr_code}`
+    );
 
-    // Also delete any references in team_clues table
-    await run(`
-      CREATE TABLE IF NOT EXISTS team_clues (
-        team_id INTEGER NOT NULL,
-        clue_id INTEGER NOT NULL,
-        found_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
-        PRIMARY KEY (team_id, clue_id),
-        FOREIGN KEY (team_id) REFERENCES teams(id),
-        FOREIGN KEY (clue_id) REFERENCES clues(id)
-      )
-    `);
-    await run("DELETE FROM team_clues WHERE clue_id = ?", [clueId]);
+    // Start a transaction for atomicity
+    try {
+      await run("BEGIN TRANSACTION");
 
-    return Response.json({
-      success: true,
-      message: "Clue deleted successfully",
-    });
+      console.log("Transaction started");
+
+      // First, make sure the team_clues table exists and clear any references
+      await run(`
+        CREATE TABLE IF NOT EXISTS team_clues (
+          team_id INTEGER NOT NULL,
+          clue_id INTEGER NOT NULL,
+          found_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
+          PRIMARY KEY (team_id, clue_id),
+          FOREIGN KEY (team_id) REFERENCES teams(id) ON DELETE CASCADE,
+          FOREIGN KEY (clue_id) REFERENCES clues(id) ON DELETE CASCADE
+        )
+      `);
+
+      // Delete any references in team_clues table
+      const deleteTeamClues = await run(
+        "DELETE FROM team_clues WHERE clue_id = ?",
+        [clueId]
+      );
+      console.log(
+        `Deleted team clues result: ${deleteTeamClues.changes} references cleared`
+      );
+
+      // Delete clue record
+      const deleteResult = await run("DELETE FROM clues WHERE id = ?", [
+        clueId,
+      ]);
+      console.log(`Deleted clue result: ${deleteResult.changes} rows affected`);
+
+      if (deleteResult.changes === 0) {
+        throw new Error(`Clue ${clueId} was not deleted`);
+      }
+
+      // If renumbering is enabled (which is always true now)
+      if (shouldRenumber) {
+        // Always use the sequential renumbering approach
+        // This ensures QR codes are sequential regardless of DB IDs
+        console.log("Renumbering all QR codes sequentially");
+
+        // Get all remaining clues ordered by ID
+        const allClues = await query("SELECT id FROM clues ORDER BY id ASC");
+        console.log(`Found ${allClues.length} clues to renumber`);
+
+        // Update each clue to have sequential QR[index+1] format
+        for (let i = 0; i < allClues.length; i++) {
+          const clue = allClues[i];
+          const sequentialQrCode = `QR${i + 1}`;
+
+          const updateResult = await run(
+            "UPDATE clues SET qr_code = ? WHERE id = ?",
+            [sequentialQrCode, clue.id]
+          );
+
+          console.log(
+            `Updated clue ${clue.id} to ${sequentialQrCode}: ${updateResult.changes} rows affected`
+          );
+        }
+      }
+
+      // Commit the transaction
+      await run("COMMIT");
+      console.log("Transaction committed successfully");
+
+      return Response.json({
+        success: true,
+        message: "Clue deleted successfully",
+        renumbered: shouldRenumber,
+      });
+    } catch (transactionError) {
+      // If any error occurs, roll back the transaction
+      console.error("Error during deletion transaction:", transactionError);
+      try {
+        await run("ROLLBACK");
+        console.log("Transaction rolled back");
+      } catch (rollbackError) {
+        console.error("Error while rolling back transaction:", rollbackError);
+      }
+
+      // Provide more specific error message based on the error type
+      let errorMessage = "Failed to delete clue: " + transactionError.message;
+      if (transactionError.message.includes("SQLITE_CONSTRAINT")) {
+        errorMessage =
+          "Database constraint violation. The clue may be referenced by other records.";
+      }
+
+      return Response.json({ error: errorMessage }, { status: 500 });
+    }
   } catch (error) {
     console.error("Admin clue deletion error:", error);
     return Response.json(
@@ -83,16 +164,17 @@ export async function GET(request, context) {
       return Response.json({ error }, { status });
     }
 
-    const { clueId } = context.params;
+    // Get clueId parameter - properly awaited
+    const { clueId } = await context.params;
 
     // Validate clueId is a number
     if (isNaN(parseInt(clueId))) {
       return Response.json({ error: "Invalid clue ID" }, { status: 400 });
     }
 
-    // Get the specific clue - removed coordinates
+    // Get the specific clue - now including qr_code
     const clue = await queryOne(
-      "SELECT id, title, description, location FROM clues WHERE id = ?",
+      "SELECT id, title, description, location, qr_code FROM clues WHERE id = ?",
       [clueId]
     );
 
@@ -119,15 +201,16 @@ export async function PUT(request, context) {
       return Response.json({ error }, { status });
     }
 
-    const { clueId } = context.params;
+    // Get clueId parameter - properly awaited
+    const { clueId } = await context.params;
 
     // Validate clueId is a number
     if (isNaN(parseInt(clueId))) {
       return Response.json({ error: "Invalid clue ID" }, { status: 400 });
     }
 
-    // Parse the request body - removed coordinates
-    const { title, description, location } = await request.json();
+    // Parse the request body
+    const { title, description, location, qr_code } = await request.json();
 
     // Validate required fields
     if (!title || !description) {
@@ -137,20 +220,33 @@ export async function PUT(request, context) {
       );
     }
 
+    // Validate QR code format if provided
+    if (qr_code && !qr_code.match(/^QR\d+$/i)) {
+      return Response.json(
+        { error: "QR code must follow the format QR[number], e.g. QR1, QR2" },
+        { status: 400 }
+      );
+    }
+
     // Check if clue exists
-    const clue = await queryOne("SELECT id FROM clues WHERE id = ?", [clueId]);
+    const clue = await queryOne("SELECT id, qr_code FROM clues WHERE id = ?", [
+      clueId,
+    ]);
     if (!clue) {
       return Response.json({ error: "Clue not found" }, { status: 404 });
     }
 
-    // Update the clue - removed coordinates
+    // Allow custom QR code - no longer enforcing QR{id} format
+    const finalQrCode = qr_code || `QR${clueId}`;
+
+    // Update the clue with the provided or default QR code
     await run(
       `
       UPDATE clues 
-      SET title = ?, description = ?, location = ?
+      SET title = ?, description = ?, location = ?, qr_code = ?
       WHERE id = ?
     `,
-      [title, description, location || "", clueId]
+      [title, description, location || "", finalQrCode, clueId]
     );
 
     return Response.json({

app/api/admin/clues/route.js

@@ -39,13 +39,14 @@ export async function GET(request) {
         title TEXT NOT NULL,
         description TEXT NOT NULL,
         location TEXT,
+        qr_code TEXT DEFAULT '',
         created_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP
       )
     `);
 
-    // Fetch all clues with sequential ordering
+    // Fetch all clues with sequential ordering - IMPORTANT: explicitly include qr_code field
     const clues = await query(`
-      SELECT id, title, description, location, created_at
+      SELECT id, title, description, location, qr_code, created_at
       FROM clues 
       ORDER BY id ASC
     `);
@@ -77,34 +78,136 @@ export async function POST(request) {
       return Response.json({ error }, { status });
     }
 
-    // Parse the request body - simplified without coordinates
-    const { title, description, location = "" } = await request.json();
-
-    // Validate required fields
-    if (!title || !description) {
+    // Parse the request body - now accepting qr_code
+    let bodyData;
+    try {
+      bodyData = await request.json();
+    } catch (parseError) {
+      console.error("Error parsing JSON request body:", parseError);
       return Response.json(
-        { error: "Title and description are required" },
+        { error: "Invalid JSON in request body" },
         { status: 400 }
       );
     }
 
-    // Insert the new clue - simplified without coordinates
-    await run(
-      `
-      INSERT INTO clues (title, description, location)
-      VALUES (?, ?, ?)
-    `,
-      [title, description, location]
-    );
+    const { title, description, location = "", qr_code = "" } = bodyData;
 
-    // Get the inserted clue ID
-    const newClue = await queryOne("SELECT last_insert_rowid() as id");
+    // Add more detailed validation
+    if (!title) {
+      return Response.json({ error: "Title is required" }, { status: 400 });
+    }
+
+    if (!description) {
+      return Response.json(
+        { error: "Description is required" },
+        { status: 400 }
+      );
+    }
+
+    // Validate QR code format if provided
+    if (qr_code && !qr_code.match(/^QR\d+$/i)) {
+      return Response.json(
+        { error: "QR code must follow the format QR[number], e.g. QR1, QR2" },
+        { status: 400 }
+      );
+    }
+
+    // Make sure the clues table exists before inserting
+    try {
+      await run(`
+        CREATE TABLE IF NOT EXISTS clues (
+          id INTEGER PRIMARY KEY AUTOINCREMENT,
+          title TEXT NOT NULL,
+          description TEXT NOT NULL,
+          location TEXT,
+          qr_code TEXT DEFAULT '',
+          created_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP
+        )
+      `);
+
+      // Check if we need to add the qr_code column (for existing tables that don't have it)
+      try {
+        // First check if the column exists
+        const tableInfo = await query(`PRAGMA table_info(clues)`);
+        const qrCodeColumnExists = tableInfo.some(
+          (col) => col.name === "qr_code"
+        );
+
+        if (!qrCodeColumnExists) {
+          // If the column doesn't exist, add it with a default value
+          await run(`ALTER TABLE clues ADD COLUMN qr_code TEXT DEFAULT ''`);
+          console.log("Added qr_code column to clues table");
+        }
+      } catch (alterTableError) {
+        console.error(
+          "Error checking/modifying table structure:",
+          alterTableError
+        );
+        // Continue anyway - the insertion will either work or fail
+      }
+    } catch (tableError) {
+      console.error("Error ensuring table exists:", tableError);
+      return Response.json(
+        { error: "Database error: Could not ensure clues table exists" },
+        { status: 500 }
+      );
+    }
+
+    // Insert the new clue with QR code if provided
+    let result;
+    try {
+      if (qr_code) {
+        // If QR code is provided, use it
+        result = await run(
+          `INSERT INTO clues (title, description, location, qr_code) VALUES (?, ?, ?, ?)`,
+          [title, description, location, qr_code]
+        );
+      } else {
+        // Otherwise, just insert without QR code (will be set later)
+        result = await run(
+          `INSERT INTO clues (title, description, location) VALUES (?, ?, ?)`,
+          [title, description, location]
+        );
+      }
+    } catch (insertError) {
+      console.error("Database insert error:", insertError);
+      return Response.json(
+        { error: `Database error: ${insertError.message}` },
+        { status: 500 }
+      );
+    }
+
+    // Get the inserted clue ID and update QR codes
+    let newClueId;
+    try {
+      const newClue = await queryOne("SELECT last_insert_rowid() as id");
+      newClueId = newClue?.id;
+
+      // If no QR code was provided, update all QR codes to ensure sequential order
+      if (!qr_code && newClueId) {
+        // Update all clues to have sequential QR codes
+        const allClues = await query("SELECT id FROM clues ORDER BY id ASC");
+
+        // Update each clue to have QR[index+1] format (sequential without gaps)
+        for (let i = 0; i < allClues.length; i++) {
+          const clue = allClues[i];
+          const sequentialQrCode = `QR${i + 1}`;
+          await run("UPDATE clues SET qr_code = ? WHERE id = ?", [
+            sequentialQrCode,
+            clue.id,
+          ]);
+        }
+      }
+    } catch (updateError) {
+      console.error("Error updating QR codes:", updateError);
+      // Continue anyway, the clue was inserted successfully
+    }
 
     return Response.json(
       {
         success: true,
         message: "Clue added successfully",
-        clueId: newClue.id,
+        clueId: newClueId || null,
       },
       { status: 201 }
     );