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
Update the DBAFS cache when creating files #7029
Update the DBAFS cache when creating files #7029
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In my case, the listContents()
did also not work after writing, which is why I had to change this line (https://github.com/contao/contao/pull/6973/files#diff-29a707fd9a3ce791f30b7313fbf90c20eb457d0383b681eea3d6d3c7fa2f9589R158). But maybe with unsetting the records, this works now. In any case, we should probably add a test for that on top, wdyt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally and fixes the issue 😊 ❤️
Thank you @m-vo. |
I agree with the general idea of the changes in #6973. IMHO we should keep generating cache entries lazy, though + the fix must also be applied to 4.13. This is why I created this PR as an alternative.
/cc @Toflar