I took a look to see if I could figure out why saving into the file system wasn't working and it appears... there's no code to do it.
The setting is: bfc_download_storage
But that's not used in the \download.php file or templates, etc. When looking at the \download.php file, the only code there is for saving to the database, no checks at all if it should save into the file system and nothing that would do it if there were (that I can find).
However, the \admincp\download.php does appear to have code for the file system and this option. This would explain why myself and others who have gotten the unlink error get that error. We're set to file system, but there's nothing in the file system to remove, because all uploads are going into the database instead.
Since the download record is being deleted, the database record is actually being deleted anyway, so the good news is this isn't leaving unused data in the database.
There's a lot of code in the \admincp\download.php that pertains to the file system saving and having only looked at this code for a short amount of time, I could be wrong on all of this. So... no offense meant towards the author at all, this is good stuff, I'm just trying to debug the issues I'm having. =)
|