Update app.py

app.py

@@ -54,93 +54,19 @@ app.layout = dbc.Container([
     ], className="my-3"),
     dbc.Button("Split PDF", id='split-button', color="primary", className="mt-3", disabled=True),
     dbc.Progress(id='progress-bar', className="my-3"),
-    dbc.Spinner(html.Div(id='processing-spinner'), color="primary", type="grow"),  # Add spinner
+    dbc.Spinner(html.Div(id='processing-spinner'), color="primary", type="grow"),
     dbc.Button("Download ZIP", id='download-button', color="success", className="mt-3", disabled=True),
     dcc.Download(id="download-zip"),
     html.Div(id='log-output', style={'whiteSpace': 'pre-line'}),
 ], fluid=True)
 
-@callback(
-    Output('pdf-name', 'children'),
-    Output('split-button', 'disabled'),
-    Input('upload-pdf', 'contents'),
-    State('upload-pdf', 'filename')
-)
-def update_output(contents, filename):
-    if contents is not None:
-        logger.info(f"PDF file uploaded: {filename}")
-        return f"Selected file: {filename}", False
-    return "No file selected", True
-
-@callback(
-    Output('ranges-container', 'children'),
-    Input('add-range', 'n_clicks'),
-    Input({'type': 'remove-range', 'index': ALL}, 'n_clicks'),
-    State('ranges-container', 'children'),
-    prevent_initial_call=True
-)
-def manage_ranges(add_clicks, remove_clicks, existing_ranges):
-    ctx = dash.callback_context
-    triggered_id = ctx.triggered[0]['prop_id'].split('.')[0]
-
-    if triggered_id == 'add-range':
-        logger.info("Adding new range input")
-        new_index = len(existing_ranges)
-        new_range = dbc.Row([
-            dbc.Col(dbc.Input(id={'type': 'range-input', 'index': new_index}, type='text', placeholder='Enter page range (e.g., 1-3)'), width=10),
-            dbc.Col(dbc.Button("Remove", id={'type': 'remove-range', 'index': new_index}, color="danger", size="sm"), width=2),
-        ], className="mb-2")
-        existing_ranges.append(new_range)
-    elif 'remove-range' in triggered_id:
-        remove_index = json.loads(triggered_id)['index']
-        logger.info(f"Removing range input at index {remove_index}")
-        existing_ranges = [range for range in existing_ranges if json.loads(range['props']['children'][1]['props']['children']['props']['id'])['index'] != remove_index]
-
-    return existing_ranges
-
-def process_pdf(contents, filename, ranges):
-    global generated_file, progress
-    progress = 0
-    logger.info(f"Starting PDF processing for file: {filename}")
-    
-    try:
-        content_type, content_string = contents.split(',')
-        decoded = base64.b64decode(content_string)
-        pdf = PdfReader(io.BytesIO(decoded))
-        logger.info(f"PDF loaded successfully. Total pages: {len(pdf.pages)}")
-        
-        writers = []
-        for range_str in ranges:
-            start, end = map(int, range_str.split('-'))
-            logger.info(f"Processing range: {start}-{end}")
-            writer = PdfWriter()
-            for i in range(start - 1, min(end, len(pdf.pages))):
-                writer.add_page(pdf.pages[i])
-            writers.append(writer)
-        
-        zip_buffer = io.BytesIO()
-        with zipfile.ZipFile(zip_buffer, 'w', zipfile.ZIP_DEFLATED) as zf:
-            for i, writer in enumerate(writers):
-                progress = (i + 1) / len(writers) * 100
-                logger.info(f"Creating split PDF {i+1}/{len(writers)}")
-                pdf_buffer = io.BytesIO()
-                writer.write(pdf_buffer)
-                pdf_buffer.seek(0)
-                zf.writestr(f"split_{i+1}.pdf", pdf_buffer.getvalue())
-        
-        zip_buffer.seek(0)
-        generated_file = zip_buffer.getvalue()
-        progress = 100
-        logger.info("PDF processing completed successfully")
-    except Exception as e:
-        logger.error(f"Error processing PDF: {str(e)}", exc_info=True)
-        progress = -1
+# ... (keep all other callbacks and functions as they were) ...
 
 @callback(
     Output('progress-bar', 'value'),
     Output('download-button', 'disabled'),
     Output('log-output', 'children'),
-    Output('processing-spinner', 'children'),  # Add output for spinner
+    Output('processing-spinner', 'children'),
     Input('split-button', 'n_clicks'),
     State('upload-pdf', 'contents'),
     State('upload-pdf', 'filename'),
@@ -165,7 +91,7 @@ def split_pdf(n_clicks, contents, filename, ranges):
     Output('progress-bar', 'value', allow_duplicate=True),
     Output('download-button', 'disabled', allow_duplicate=True),
     Output('log-output', 'children', allow_duplicate=True),
-    Output('processing-spinner', 'children', allow_duplicate=True),  # Add output for spinner
+    Output('processing-spinner', 'children', allow_duplicate=True),
     Input('progress-bar', 'value'),
     prevent_initial_call=True
 )
@@ -176,7 +102,7 @@ def update_progress(value):
         return 100, False, "PDF splitting completed. Click 'Download ZIP' to get your files.", ""
     elif progress == -1:
         logger.error("PDF splitting failed")
-        return 0, True, "Error occurred during PDF splitting. Check console for details.", ""
+        return 0, False, "Error occurred during PDF splitting. Check console for details.", ""
     else:
         return progress, True, f"Processing... {progress:.0f}% complete", "Processing..."