- last update
about 1 year ago 16 pass, 3 fail - 🇫🇷France anrikun
Patch at #2 seems to work but I'm pretty sure setting
$node->original
is not wanted in the first place.
Alsonode_load($node->nid)
may get node from cache instead of loading the expected unchanged node.
Patch attached removes the line that sets$node->original
. The last submitted patch, 4: original_node_as_current_published-3246269-4.patch, failed testing. View results →
- Status changed to Needs review
about 1 year ago 9:34am 16 January 2024 - last update
about 1 year ago 16 pass, 3 fail - 🇫🇷France anrikun
Let's see if loading the unchanged node is preferred then.
The last submitted patch, 6: original_node_as_current_published-3246269-6.patch, failed testing. View results →
- last update
about 1 year ago 16 pass, 3 fail - 🇫🇷France anrikun
Ok patch at #2 fails too now. So I guess there's something wrong elsewhere.
For now patch #4 seems the right one.