silverstripe-terraformers/keys-for-cache

CacheKeys are not cleared on GridField reorder actions

chrispenny opened this issue · 3 comments

chrispenny commented

This is because GridFieldOrderableRows does not perform a write() on the models, it instead performs SQL Update queries. This bypasses all of the triggers for KFC.

We might need some new logic specifically for this type of action. I'm not sure we would want to call write() on all of the models. Maybe we could call triggerEvent() instead? Or perhaps we need some specific logic for GridFields that is kind of "to the side" of this existing model logic.

chrispenny commented

After some more investigation, it looks like it is only unversioned models that have their sort changed through SQL queries rather than through the ORM.

chrispenny commented

Closed by #46