Commander1024/lostplaces-backend
1
0
Fork 0
Code Issues 14 Pull Requests Releases Wiki Activity

Pictures are in the wrong directory #59

New Issue
Open
opened 2022-01-01 21:19:59 +01:00 by reverend · 1 comment
reverend commented 2022-01-01 21:19:59 +01:00
Collaborator
Copy Link
Copy Source

On instances with DEBUG=False pictures (cropped by easy_thumbnails) are not displayed. This is because the thumbnail_cleanup command in manage.py does not work properly:

django.core.exceptions.SuspiciousFileOperation: The joined path (/home/leo/Git/lostplaces-backend/django_lostplaces/uploads/places/1-placename.jpg) is located outside of the base path component (/home/leo/Git/lostplaces-backend/django_lostplaces/uploads/thumbs)

On an already existing instance the cropped are there, but are in the wrong folder. The settings for THUMBNAIL_MEDIA_ROOT, THUMBNAIL_MEDIA_URL and the upload of place images need to be refactored. Also already existing images might have to be moved to the correct folder within the MEDIA_ROOT, this might be a thing for migrations.

On instances with `DEBUG=False` pictures (cropped by easy_thumbnails) are not displayed. This is because the *thumbnail_cleanup* command in manage.py does not work properly: ``` django.core.exceptions.SuspiciousFileOperation: The joined path (/home/leo/Git/lostplaces-backend/django_lostplaces/uploads/places/1-placename.jpg) is located outside of the base path component (/home/leo/Git/lostplaces-backend/django_lostplaces/uploads/thumbs) ``` On an already existing instance the cropped are there, but are in the wrong folder. The settings for `THUMBNAIL_MEDIA_ROOT`, `THUMBNAIL_MEDIA_URL` and the upload of place images need to be refactored. Also already existing images might have to be moved to the correct folder within the `MEDIA_ROOT`, this might be a thing for migrations.
reverend added the ChangeHotFix RequiredIssueMajor labels 2022-01-01 21:20:00 +01:00
reverend self-assigned this 2022-01-01 21:20:00 +01:00
Commander1024 was assigned by reverend 2022-01-01 21:20:01 +01:00
reverend commented 2022-01-13 17:25:24 +01:00
Author
Collaborator
Copy Link
Copy Source

Attempted Fix in 830120a929, still needs to be tested under different cricumstances.

Attempted Fix in 830120a929, still needs to be tested under different cricumstances.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Change
Feature
General Task
HotFix Required
Idea
Issue
Major
Minor
Question
No labels Change HotFix Required Issue Major
Milestone
No items
No Milestone
Assignees
Clear assignees
reverend Commander1024 (Marcus Scholz)
No Assignees reverend Commander1024
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Commander1024/lostplaces-backend#59