Bug #12985
Error uploading a Finding Aid when the downloads directory doesn't exist
Status: | Verified | Start date: | 04/26/2019 | |
---|---|---|---|---|
Priority: | Medium | Due 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:
- Make sure the AtoM "downloads" directory does not already exist (a fresh install or delete the directory manually)
- Login to AtoM with contributor permissions or better
- View an archival description
- Click the "finding aid > Upload" link in the right side-bar
- Click "Choose File" and select a PDF from your computer
- 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.
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?
#3 Updated by Steve Breker about 3 years ago
Ready for code review: https://github.com/artefactual/atom/pull/885
#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
#7 Updated by Steve Breker about 3 years ago
Related PR:
#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:
#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