- 🇺🇸United States smustgrave
Was previously tagged for tests which still need to happen.
- 🇮🇳India rksyravi New Delhi, 🇮🇳
Hi @jernejmramor
Update the changes and create a MR(Merge Request), because Drupal recommends MR over patch file.
- last update
10 months ago 26,067 pass, 1,836 fail - First commit to issue fork.
- Assigned to akshaydalvi212
- 🇮🇳India akshaydalvi212
Hey,
will provide the MR for the updating Entity timestamp. - Issue was unassigned.
- Status changed to Needs review
9 months ago 7:14am 12 February 2024 - Status changed to Needs work
9 months ago 3:38pm 12 February 2024 - 🇺🇸United States smustgrave
Was previously tagged for tests which still need to happen.
- 🇺🇸United States Farnoosh
I performed Novice Triage on this issue. I am leaving the Novice tag on this issue because we still need to test the MR.
- 🇺🇸United States xjm
To be clear, the task is to add test coverage to the MR (a test that fails before the fix and passes with it). Thanks!
- Status changed to Postponed: needs info
3 months ago 9:58am 2 September 2024 - 🇸🇮Slovenia jernejmramor
I've tried replicating this issue again by creating a new custom module which adds an Entity that extended EntityChangedInterface, and exposed a rest endpoint with custom rest resource plugin that extended EntityResource. Using postman API I then created a new entity using POST method, and updated the same entity with PATCH method. Every successful update of an entity is also reflected in updated 'changed' value in the database.
I've also tried replicating this behavior using bundle-able custom entities and even then 'changed' value was always updated in database as expected.If this is still an issue, please provide a more detailed set of replication steps so that we can provide tests and solution for this issue. I'm committing an updated patch form what is currently in the open MR so that plugin uses DI for the time service. I'm also changing the status to Postponed (maintainer needs more info) as the issue is 4 years old, and is no longer replicate-able (at least by me).