- Issue created by @jhuhta
- Issue was unassigned.
- Status changed to Needs review
8 months ago 12:50pm 7 August 2024 - 🇫🇮Finland jhuhta
This patch fixes the issue for us. I can make an MR out of it if the maintainer thinks this makes any sense.
- 🇳🇱Netherlands johnv
Indeed, in the efforts to fix the field cache problems, I only experienced (and only got reports from the audience) for anonymous users (with Internal Page Cache module enabled).
Apparently, your client has a setup that also breaks cache for known users (with Internal Dynamic Page Cache module enabled).I cannot reproduce the error (neither when displaying a node,nor when displaying Views Fields)
Let me check your fix, since it may harm performance on the majority of sites.
- 🇳🇱Netherlands johnv
I am not sure. Your test case is exactly the same as mine.
I prefer leaving the code as-is, until i get some more reports with the same problem.
In the past, many people attributed to the field caching, - 🇳🇱Netherlands johnv
Please check 📌 Eliminate 'status_update', since the Open/Closed status is now Computed, and page cache is no hurdle anymore Active and report back.
- 🇳🇱Netherlands johnv
Is your problem perhaps related to exception days?
@see 🐛 Fix field cache for exception day Active - 🇳🇱Netherlands johnv
Also, please take a look at ✨ Rework displaying dynamic information to JS for persistent caching Active . Perhaps we can take this a step further.