gitea-admin/gh-christianlouis-docuelevate
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #427 from christianlouis/copilot/fix-processed-file-view-error

Browse Source 
fix(api): use stored DB paths for file preview and download
This commit is contained in:
Christian Krakau-Louis
2026-02-27 01:15:44 +01:00
committed by GitHub
parent 4ec270481d cded73481e
commit 0055de3c66
1 changed files with 24 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/api/files.py
+24 -14
View File
@@ -842,20 +842,25 @@ def get_file_preview(
raise HTTPException(status_code=404, detail=f"File with ID {file_id} not found") raise HTTPException(status_code=404, detail=f"File with ID {file_id} not found")
if version == "original": if version == "original":
# Return the original file from tmp # Use the stored original_file_path first (persisted copy), then fall back to local_filename (tmp)
if not file_record.local_filename or not os.path.exists(file_record.local_filename): file_path = None
for path in [file_record.original_file_path, file_record.local_filename]:
if path and os.path.exists(path):
file_path = path
break
if not file_path:
raise HTTPException(status_code=404, detail="Original file not found on disk") raise HTTPException(status_code=404, detail="Original file not found on disk")
file_path = file_record.local_filename
elif version == "processed": elif version == "processed":
# Look for processed file in /workdir/processed/ # Use the stored processed_file_path first, then fall back to guessing
workdir = settings.workdir workdir = settings.workdir
processed_dir = os.path.join(workdir, "processed") processed_dir = os.path.join(workdir, "processed")
# Try to find the processed file (same hash or UUID-based naming) # Try to find the processed file (stored path first, then hash or UUID-based naming)
base_filename = os.path.splitext(file_record.original_filename)[0] base_filename = os.path.splitext(file_record.original_filename)[0]
potential_paths = [ potential_paths = [
file_record.processed_file_path,
os.path.join(processed_dir, f"{file_record.filehash}.pdf"), os.path.join(processed_dir, f"{file_record.filehash}.pdf"),
os.path.join(processed_dir, f"{base_filename}_processed.pdf"), os.path.join(processed_dir, f"{base_filename}_processed.pdf"),
os.path.join(processed_dir, file_record.original_filename), os.path.join(processed_dir, file_record.original_filename),
@@ -863,7 +868,7 @@ def get_file_preview(
file_path = None file_path = None
for path in potential_paths: for path in potential_paths:
if os.path.exists(path): if path and os.path.exists(path):
file_path = path file_path = path
break break
@@ -917,20 +922,25 @@ def download_file(
raise HTTPException(status_code=404, detail=f"File with ID {file_id} not found") raise HTTPException(status_code=404, detail=f"File with ID {file_id} not found")
if version == "original": if version == "original":
# Return the original file from tmp # Use the stored original_file_path first (persisted copy), then fall back to local_filename (tmp)
if not file_record.local_filename or not os.path.exists(file_record.local_filename): file_path = None
for path in [file_record.original_file_path, file_record.local_filename]:
if path and os.path.exists(path):
file_path = path
break
if not file_path:
raise HTTPException(status_code=404, detail="Original file not found on disk") raise HTTPException(status_code=404, detail="Original file not found on disk")
file_path = file_record.local_filename
elif version == "processed": elif version == "processed":
# Look for processed file in /workdir/processed/ # Use the stored processed_file_path first, then fall back to guessing
workdir = settings.workdir workdir = settings.workdir
processed_dir = os.path.join(workdir, "processed") processed_dir = os.path.join(workdir, "processed")
# Try to find the processed file (same hash or UUID-based naming) # Try to find the processed file (stored path first, then hash or UUID-based naming)
base_filename = os.path.splitext(file_record.original_filename)[0] base_filename = os.path.splitext(file_record.original_filename)[0]
potential_paths = [ potential_paths = [
file_record.processed_file_path,
os.path.join(processed_dir, f"{file_record.filehash}.pdf"), os.path.join(processed_dir, f"{file_record.filehash}.pdf"),
os.path.join(processed_dir, f"{base_filename}_processed.pdf"), os.path.join(processed_dir, f"{base_filename}_processed.pdf"),
os.path.join(processed_dir, file_record.original_filename), os.path.join(processed_dir, file_record.original_filename),
@@ -938,7 +948,7 @@ def download_file(
file_path = None file_path = None
for path in potential_paths: for path in potential_paths:
if os.path.exists(path): if path and os.path.exists(path):
file_path = path file_path = path
break break