Update app.py

app.py

@@ -11,14 +11,14 @@ import re
 quiz_history = []
 
 # ============================
-# Extract sentences (FIXED)
+# Extract sentences
 # ============================
 def extract_sentences(text):
     sentences = re.split(r'[.!?\n]', text)
     return [s.strip() for s in sentences if len(s.strip()) > 20]
 
 # ============================
-# Generate quiz (SAFE)
+# Generate quiz
 # ============================
 def generate_quiz(text, num_questions):
     try:
@@ -130,23 +130,22 @@ def submit_answer(selected, quiz, index, score, end_time):
     return show_question(quiz, index, score, end_time)
 
 # ============================
-# Prepare quiz (FIXED 🔥)
+# Prepare quiz (TEXT + PDF FIXED)
 # ============================
 def prepare_quiz(text, pdf_library, selected_pdf, num_questions, timer_minutes):
 
-    # FIX: handle None
     if text is None:
         text = ""
 
-    # PDF override
+    # Use PDF if selected
     if selected_pdf and pdf_library and selected_pdf in pdf_library:
         text = pdf_library[selected_pdf]
 
-    # FINAL CHECK
+    print("FINAL TEXT LENGTH:", len(text))  # debug
+
     if not text.strip():
         return "⚠️ Provide text or select a PDF.", None, None, None, None, None, None, ""
 
-    # Timer
     try:
         timer_minutes = float(timer_minutes)
         if timer_minutes <= 0:
@@ -157,14 +156,14 @@ def prepare_quiz(text, pdf_library, selected_pdf, num_questions, timer_minutes):
     quiz, msg = generate_quiz(text, num_questions)
 
     if not quiz:
-        return msg, None, None, None, None, None, None, ""
+        return "❌ Could not generate quiz. Try better text or PDF.", None, None, None, None, None, None, ""
 
     end_time = time.time() + timer_minutes * 60
 
     return show_question(quiz, 0, 0, end_time)
 
 # ============================
-# Load PDFs
+# Load PDFs (FIXED)
 # ============================
 def load_multiple_pdfs(files):
     if files is None:
@@ -176,14 +175,21 @@ def load_multiple_pdfs(files):
     for file in files:
         try:
             text = ""
-            with open(file.name, "rb") as f:
-                pdf = PdfReader(f)
-                for page in pdf.pages:
-                    page_text = page.extract_text()
-                    if page_text:
-                        text += page_text + " "
+
+            pdf = PdfReader(file.name)
+
+            for page in pdf.pages:
+                page_text = page.extract_text()
+                if page_text:
+                    text += page_text + " "
+
+            print(f"{file.name} TEXT LENGTH:", len(text))  # debug
 
             name = file.name.split("/")[-1]
+
+            if not text.strip():
+                text = "This PDF has no readable text."
+
             pdf_texts[name] = text.strip()
             names.append(name)
 
@@ -276,11 +282,7 @@ with gr.Blocks(title="Smart Quiz Generator") as app:
             download_file = gr.File()
 
     # EVENTS
-    pdf_input.change(
-        load_multiple_pdfs,
-        inputs=pdf_input,
-        outputs=[pdf_library_state, pdf_selector]
-    )
+    pdf_input.change(load_multiple_pdfs, inputs=pdf_input, outputs=[pdf_library_state, pdf_selector])
 
     start_btn.click(
         prepare_quiz,