fix: consolidate pending fixes for PDF manager UI, dashboard filtering, and SQL syntax

- Removed corrupted 'arial.ttf'.
- Updated 'dashboard.py' to filter out 'final_pdf' sessions.
- Applied migrations in 'database.py' for session metadata.
- Fixed SQL syntax error in 'routes/library.py'.
- Set session type to 'final_pdf' in 'routes/upload.py'.
- Refined 'templates/pdf_manager.html' with search, dropdowns, and individual delete buttons.

dashboard.py

@@ -140,7 +140,7 @@ def dashboard():
                COUNT(CASE WHEN i.image_type = 'cropped' THEN 1 END) as question_count
         FROM sessions s
         LEFT JOIN images i ON s.id = i.session_id
-        WHERE s.user_id = ?
+        WHERE s.user_id = ? AND (s.session_type IS NULL OR s.session_type != 'final_pdf')
         GROUP BY s.id, s.created_at, s.original_filename, s.persist, s.name, s.session_type
         ORDER BY s.created_at DESC
     """, (current_user.id,)).fetchall()

database.py

@@ -263,6 +263,21 @@ def setup_database():
     except sqlite3.OperationalError:
         cursor.execute("ALTER TABLE questions ADD COLUMN chapter TEXT")
 
+    try:
+        cursor.execute("SELECT subject FROM sessions LIMIT 1")
+    except sqlite3.OperationalError:
+        cursor.execute("ALTER TABLE sessions ADD COLUMN subject TEXT")
+
+    try:
+        cursor.execute("SELECT tags FROM sessions LIMIT 1")
+    except sqlite3.OperationalError:
+        cursor.execute("ALTER TABLE sessions ADD COLUMN tags TEXT")
+
+    try:
+        cursor.execute("SELECT notes FROM sessions LIMIT 1")
+    except sqlite3.OperationalError:
+        cursor.execute("ALTER TABLE sessions ADD COLUMN notes TEXT")
+
     # --- Multi-user Migrations ---
     try:
         cursor.execute("SELECT user_id FROM sessions LIMIT 1")

routes/library.py

@@ -1,6 +1,8 @@
 import os
+import io
+import zipfile
 from datetime import datetime
-from flask import request, jsonify, render_template, redirect, url_for, send_file
+from flask import request, jsonify, render_template, redirect, url_for, send_file, current_app
 from .common import main_bp, get_db_connection, login_required, current_user
 from database import get_folder_tree, get_all_descendant_folder_ids
 from strings import METHOD_POST, METHOD_DELETE
@@ -222,7 +224,7 @@ def bulk_move_pdfs():
 def get_all_subjects_and_tags():
     conn = get_db_connection()
     subjects = [row['subject'] for row in conn.execute('SELECT DISTINCT subject FROM generated_pdfs WHERE subject IS NOT NULL AND user_id = ?', (current_user.id,)).fetchall()]
-    tags_query = conn.execute('SELECT DISTINCT tags FROM generated_pdfs WHERE tags IS NOT NULL AND tags != '' AND user_id = ?', (current_user.id,)).fetchall()
+    tags_query = conn.execute("SELECT DISTINCT tags FROM generated_pdfs WHERE tags IS NOT NULL AND tags != '' AND user_id = ?", (current_user.id,)).fetchall()
     all_tags = set()
     for row in tags_query:
         for t in row['tags'].split(','): all_tags.add(t.strip())
@@ -233,7 +235,7 @@ def get_all_subjects_and_tags():
 def get_metadata_suggestions():
     conn = get_db_connection()
     subjects = [row['subject'] for row in conn.execute('SELECT DISTINCT subject FROM generated_pdfs WHERE subject IS NOT NULL AND user_id = ?', (current_user.id,)).fetchall()]
-    tags_query = conn.execute('SELECT DISTINCT tags FROM generated_pdfs WHERE tags IS NOT NULL AND tags != '' AND user_id = ?', (current_user.id,)).fetchall()
+    tags_query = conn.execute("SELECT DISTINCT tags FROM generated_pdfs WHERE tags IS NOT NULL AND tags != '' AND user_id = ?", (current_user.id,)).fetchall()
     all_tags = set()
     for row in tags_query:
         for t in row['tags'].split(','): all_tags.add(t.strip())

routes/upload.py

@@ -193,7 +193,9 @@ def handle_final_pdf_upload():
     conn = get_db_connection()
     def process_and_save_pdf(file_content, original_filename):
         sid = str(uuid.uuid4())
-        conn.execute('INSERT INTO sessions (id, original_filename, user_id) VALUES (?, ?, ?)', (sid, original_filename, current_user.id))
+        # Associate session with user and mark as final_pdf
+        conn.execute('INSERT INTO sessions (id, original_filename, user_id, session_type) VALUES (?, ?, ?, ?)', 
+                     (sid, original_filename, current_user.id, 'final_pdf'))
         out_name = f"{sid}_{secure_filename(original_filename)}"
         with open(os.path.join(current_app.config['OUTPUT_FOLDER'], out_name), 'wb') as f: f.write(file_content)
         conn.execute('INSERT INTO generated_pdfs (session_id, filename, subject, tags, notes, source_filename, user_id) VALUES (?, ?, ?, ?, ?, ?, ?)', (sid, out_name, subject, tags, notes, original_filename, current_user.id))

templates/pdf_manager.html

@@ -43,6 +43,9 @@
     .table-hover .highlighted-row {
         background-color: #0d6efd !important;
     }
+    .no-caret::after {
+        display: none !important;
+    }
 
     /* Tom Select Dark Mode Fix */
     .ts-control {
@@ -78,7 +81,14 @@
     <!-- Header & Actions -->
     <div class="d-flex justify-content-between align-items-center flex-wrap gap-2 mb-4">
         <h1 class="mb-0">PDF Manager</h1>
-        <div class="d-flex flex-wrap gap-2">
+        <div class="d-flex flex-wrap gap-2 align-items-center">
+            <form action="/pdf_manager" method="get" class="d-flex gap-2">
+                <input type="text" name="search" class="form-control bg-dark text-white border-secondary" placeholder="Search..." value="{{ request.args.get('search', '') }}">
+                <button type="submit" class="btn btn-outline-secondary"><i class="bi bi-search"></i></button>
+                {% if request.args.get('search') %}
+                    <a href="/pdf_manager" class="btn btn-outline-danger"><i class="bi bi-x-lg"></i></a>
+                {% endif %}
+            </form>
             {% if all_view %}
                 <a href="/pdf_manager" class="btn btn-info">Show Folders</a>
             {% else %}
@@ -130,8 +140,18 @@
                     <h5 class="card-title mt-2">{{ folder.name }}</h5>
                     <p class="card-text text-muted">{{ folder.created_at.strftime('%Y-%m-%d %I:%M %p') }}</p>
                 </div>
-                <div class="card-footer text-center">
+                <div class="card-footer d-flex justify-content-between align-items-center">
                     <input type="checkbox" class="form-check-input item-checkbox" data-item-type="folder" value="{{ folder.id }}">
+                    <div class="dropdown">
+                        <button class="btn btn-sm btn-outline-secondary dropdown-toggle no-caret" type="button" data-bs-toggle="dropdown">
+                            <i class="bi bi-three-dots-vertical"></i>
+                        </button>
+                        <ul class="dropdown-menu dropdown-menu-dark dropdown-menu-end">
+                            <li><a class="dropdown-item rename-single-btn" href="#" data-id="{{ folder.id }}" data-type="folder"><i class="bi bi-pencil-square me-2"></i> Rename</a></li>
+                            <li><hr class="dropdown-divider"></li>
+                            <li><a class="dropdown-item text-danger delete-item-btn" href="#" data-item-type="folder" data-id="{{ folder.id }}"><i class="bi bi-trash me-2"></i> Delete</a></li>
+                        </ul>
+                    </div>
                 </div>
             </div>
         </div>
@@ -181,9 +201,20 @@
                 </div>
                 <div class="card-footer d-flex justify-content-between align-items-center">
                     <small class="text-muted">{{ pdf.created_at.strftime('%Y-%m-%d %I:%M %p') }}</small>
-                    <span>
+                    <div class="d-flex align-items-center gap-2">
                         {% if pdf.persist %}<i class="bi bi-pin-angle-fill text-primary" title="Persisted"></i>{% endif %}
-                    </span>
+                        <div class="dropdown">
+                            <button class="btn btn-sm btn-outline-secondary dropdown-toggle no-caret" type="button" data-bs-toggle="dropdown">
+                                <i class="bi bi-three-dots-vertical"></i>
+                            </button>
+                            <ul class="dropdown-menu dropdown-menu-dark dropdown-menu-end">
+                                <li><a class="dropdown-item edit-single-btn" href="#" data-id="{{ pdf.id }}"><i class="bi bi-info-circle me-2"></i> Details</a></li>
+                                <li><a class="dropdown-item" href="{{ url_for('main.download_file', filename=pdf.filename) }}"><i class="bi bi-download me-2"></i> Download</a></li>
+                                <li><hr class="dropdown-divider"></li>
+                                <li><a class="dropdown-item text-danger delete-item-btn" href="#" data-item-type="pdf" data-id="{{ pdf.id }}"><i class="bi bi-trash me-2"></i> Delete</a></li>
+                            </ul>
+                        </div>
+                    </div>
                 </div>
             </div>
         </div>
@@ -367,6 +398,77 @@
             cb.addEventListener('change', updateBulkButtons);
         });
 
+        // Single Deletion
+        document.querySelectorAll('.delete-item-btn').forEach(btn => {
+            btn.addEventListener('click', async (e) => {
+                e.preventDefault();
+                e.stopPropagation();
+                const itemType = btn.dataset.itemType;
+                const itemId = btn.dataset.id;
+                if (!confirm(`Are you sure you want to delete this ${itemType}?`)) return;
+
+                const url = itemType === 'folder' ? `/delete_folder/${itemId}` : `/delete_generated_pdf/${itemId}`;
+                const response = await fetch(url, { method: 'DELETE' });
+                if (response.ok) {
+                    location.reload();
+                } else {
+                    alert('Failed to delete item.');
+                }
+            });
+        });
+
+        // Single Rename
+        document.querySelectorAll('.rename-single-btn').forEach(btn => {
+            btn.addEventListener('click', (e) => {
+                e.preventDefault();
+                e.stopPropagation();
+                
+                const card = btn.closest('.card');
+                const titleElement = card.querySelector('.card-title') || card.querySelector('.fw-bold');
+                const currentName = titleElement.textContent.trim();
+                const itemType = btn.dataset.type || 'pdf';
+                const itemId = btn.dataset.id;
+
+                titleElement.innerHTML = `<input type="text" class="editable-input" value="${currentName}" />`;
+                const input = titleElement.querySelector('input');
+                input.focus();
+                input.select();
+
+                const saveChanges = async () => {
+                    const newName = input.value;
+                    if (newName && newName !== currentName) {
+                        const response = await fetch('/rename_item', {
+                            method: 'POST',
+                            headers: { 'Content-Type': 'application/json' },
+                            body: JSON.stringify({
+                                item_type: itemType,
+                                item_id: itemId,
+                                new_name: newName
+                            })
+                        });
+                        if (response.ok) location.reload();
+                        else {
+                            alert('Failed to rename.');
+                            titleElement.textContent = currentName;
+                        }
+                    } else {
+                        titleElement.textContent = currentName;
+                    }
+                };
+
+                input.addEventListener('blur', saveChanges);
+                input.addEventListener('keydown', ev => {
+                    if (ev.key === 'Enter') {
+                        ev.preventDefault();
+                        saveChanges();
+                    }
+                    if (ev.key === 'Escape') titleElement.textContent = currentName;
+                });
+            });
+        });
+
+        // Single Edit
+
         document.querySelectorAll('.folder-card').forEach(card => {
             card.addEventListener('click', e => {
                 // If the click target is the folder name link, let the link handle it