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

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
Change
HotFix Required
Issue
Major
 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

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 Label
Change
HotFix Required
Issue
Major
Milestone
No items
No Milestone
Assignees
Clear assignees
reverend Commander1024
No Assignees Commander1024 reverend
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Commander1024/lostplaces-backend#59
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?