Bug #12985

Error uploading a Finding Aid when the downloads directory doesn't exist

Added by David Juhasz about 3 years ago. Updated about 3 years ago.

Status:VerifiedStart date:04/26/2019
Priority:MediumDue date:
Assignee:Steve Breker% Done:

0%

Category:Finding aids
Target version:Release 2.5.0
Google Code Legacy ID: Tested version:2.5
Sponsored:No Requires documentation:No

Description

Steps to reproduce:
  1. Make sure the AtoM "downloads" directory does not already exist (a fresh install or delete the directory manually)
  2. Login to AtoM with contributor permissions or better
  3. View an archival description
  4. Click the "finding aid > Upload" link in the right side-bar
  5. Click "Choose File" and select a PDF from your computer
  6. Click "Upload"

Error result

An error message is shown in the UI: "Uploaded finding aid could not be moved to the downloads directory"

and the following error is written to the error log:

2019/04/26 15:52:14 [error] 18209#18209: *224 FastCGI sent in stderr: "PHP
message: Empty module and/or action after parsing the URL
"/js/repositoryHoldings.js" (/)" while reading response header from upstream,
client: 172.100.6.XXX, server: _, request: "GET /js/repositoryHoldings.js
HTTP/1.1", upstream: "fastcgi://unix:/run/php7.2-fpm.atom.sock:", host:
"54.39.186.XXX", referrer:
"http://54.39.186.XXX/index.php/test/informationobject/uploadFindingAid" 

Expected result
The downloads directory should be created if it doesn't exist already. The Finding aid should be uploaded and attached to the description.

finding_aid_upload.png (521 KB) David Juhasz, 04/26/2019 04:27 PM

upload_finding_aid_error.png (45.4 KB) David Juhasz, 04/26/2019 04:30 PM

History

#1 Updated by David Juhasz about 3 years ago

  • Description updated (diff)

#2 Updated by David Juhasz about 3 years ago

  • Assignee set to Steve Breker

Steve, can you please fix this bug?

#4 Updated by Mike Cantelon about 3 years ago

  • Status changed from New to Feedback

PR looks good to me!

#5 Updated by Steve Breker about 3 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Steve Breker to Dan Gillean

Merged. Ready for QA.

#6 Updated by Dan Gillean about 3 years ago

  • Status changed from QA/Review to Verified
  • Assignee deleted (Dan Gillean)
  • Requires documentation set to No

#8 Updated by Mike Cantelon about 3 years ago

  • Status changed from Verified to Code Review

New PR... Added logic to create uploads and downloads directories during installation:

https://github.com/artefactual/atom/pull/887

#9 Updated by David Juhasz about 3 years ago

  • Assignee set to Steve Breker

#10 Updated by Dan Gillean about 3 years ago

  • Status changed from Code Review to Verified

Also available in: Atom PDF