-
Notifications
You must be signed in to change notification settings - Fork 71
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.
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
File Delete Module for VM #1554
Comments
@DonRichards i from time to time read these notifications. That module won't do what you expect it to do (at least not by default without some extra work), read: https://www.drupal.org/node/2891902 Basically in D8 files that have no usage are not longer 'deleted' deleted because of many flaws on how that was happening (usage tracking) and people loosing important files If you want to really delete them, you need to code it or play with some settings (which will then trigger the issue of why it was disabled by default) . We do that in archipelago and its quite a burden but needed. |
@DiegoPino I don't completely follow you. If a person adds an unnecessary file to an existing node and needs to remove just the file. Is this the scenario you're suggesting to cover? |
@DonRichards that is why i shared the link but also my fault to chime in, i was sharing my experience. Executive summary:
Read the code here: So having that module enabled right now is exactly as having this global config: Both require you assume Drupal is not (could) not be keeping track correctly of the file usage. That is all i'm saying. Conclusion: that module gives you a certain level of control. Does not deal with the fact that counting can be wrong and/or you still have (in your case) delete the given media element, which is also attached to your node and then if all is OK, and the file is not being used anywhere, you can press that button. Hope that makes things clearer |
I believe the file_delete module should be added to the base Ansible scripts and should be a suggested as a recommended module.
The text was updated successfully, but these errors were encountered: