We've been working around problems with Entity Cache by manually clearing out the entity cache upon editing a Webform's component. Instead of doing this, we should use the typical node_save() operation which will clear out the entity's cache automatically for us. It would also allow other modules to respond to changes in the node via hook_node_update() and _presave(), which should be fired if we're making changes to the node.
Preliminary patch attached.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | form_builder_webform_node_save-d6.patch | 2.93 KB | quicksketch |
| form_builder_webform_node_save.patch | 2.96 KB | quicksketch |
Comments
Comment #1
quicksketchPatch works great and eliminates special casing in our code. Yay. Backport to D6 also solves similar problems with the node_load() cache. This patch committed to D6. Previous patch committed to D7.