Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FileField ID in pre-delete #96

Closed
NadavK opened this issue Dec 12, 2022 · 1 comment · Fixed by #95
Closed

FileField ID in pre-delete #96

NadavK opened this issue Dec 12, 2022 · 1 comment · Fixed by #95

Comments

@NadavK
Copy link

NadavK commented Dec 12, 2022 •
edited

To delete a file in a non-typical storage location, I required the file ID.
The problem is that cleanup_pre_delete() was being called AFTER the FakeInstance(), which effectively wipes the id from the instance.
My solution is to move cleanup_pre_delete() before calling FakeInstance().
Not sure what other impact this has...
@NadavK NadavK changed the title FileField ID Before Delete FileField ID in pre-delete Dec 12, 2022
@NadavK NadavK mentioned this issue Dec 12, 2022
Closed
@vinnyrose vinnyrose mentioned this issue Jan 8, 2023
Merged
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature/7.0 un1t/django-cleanup
1 participant
@NadavK