Hi, you’re absolutely right that this issue manifests itself in more places, like
- editing dropdowns
- editing layouts
- installing modules (SQL not executed)
Adding the fix the the method you proposed would alleviate some of these issues. However, from an architecture viewpoint, the fix should be implemented in SugarCache::cleanFile (include/SugarCache/SugarCache.php), where there is already a fix for APC caching.
SugarCache however should not be called in file_utils.php, because that’s too low level, instead it should be called in those classes that write vardef data (and subsequently call “write_array_to_file” or “sugar_file_put_contents” )
I haven’t got to this yet.