- Issue created by @dieterholvoet
- First commit to issue fork.
- 🇫🇷France mably
Really nice feature to have, thanks!
Pushed a commit to add a missing import.
- Status changed to Needs review
10 days ago 12:48pm 18 July 2025 - 🇷🇸Serbia gnikolovski Subotica, Serbia
Can you add a test for this? It seems pretty straightforward.
- 🇫🇷France mably
Actually I cannot reproduce the problem on Drupal 11.2.
Calling
$entity->save()
doesn't seem to update thechanged
timestamp.@dieterholvoet could you provide a detailed scenario to reproduce the problem on a fresh Drupal 11 instance?
- 🇫🇷France mably
EDIT: Ok, it looks like it's because I'm creating and updating the node in the same request, so the
\Drupal::time()->getRequestTime()
value is the same when updating the node... - 🇫🇷France mably
Tried to implement some basic changed timestamp preservation tests.
Added a basic test time service to allow changing the request time during test.
I'm not a test expert, so any feedback will be greatly appreciated.