- Issue created by @roderik
- Assigned to junkuncz
- Status changed to Needs work
4 months ago 1:17pm 16 August 2024 - Issue was unassigned.
- Status changed to Needs review
4 months ago 8:22am 17 August 2024 - 🇳🇱Netherlands roderik Amsterdam,NL / Budapest,HU
Great, thanks for testing / fixing!
I'm also adding it inside the "else" of buildEntityContent(), when the CE display is disabled. Because when it is enabled again, the same cache tags should be cleared.
But I can do that at the same time as a quick test.
- 🇳🇱Netherlands roderik Amsterdam,NL / Budapest,HU
Two things:
1. My previous comment was dumb. In the "else", there IS no CE display, so you also cannot add it.
2. This is incomplete.
- Suppose you build a list of stuff in "teaser" view mode -- and the "teaser" CE display is not enabled, so it's using the default CE display.
- If this default CE display is edited, then caches for this list should be flushed. OK.
- If the "teaser" CE display is created/saved, then caches for this list should also be flushed, because they should be rebuilt with the new display. This did not happen yet.
So, cache tags should be added for both cases.
I figured it would be better to do this instead of setting needs work. Please review current status.
- Status changed to RTBC
4 months ago 2:10pm 18 August 2024 -
roderik →
committed 8f6d8679 on 3.x authored by
junkuncz →
Issue #3468371 by junkuncz, roderik: Old content is cached after...
-
roderik →
committed 8f6d8679 on 3.x authored by
junkuncz →
- Status changed to Fixed
4 months ago 2:32pm 18 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.