Wrong revision timestamp [#3346689]

Problem/Motivation

The revision timestamp of a UserpointsEntity always contains the timestamp of the creation date of the entity. From my understanding, it should rather contain the timestamp of the latest change.

Steps to reproduce

- Create UserpointsEntity
- Add points
- The timestamp will not change

Proposed resolution

- Update the timestamp when points are added or deduced

Remaining tasks

- Implementation

Issue fork userpoints-3346689

Show commands

Start within a Git clone of the project using the version control instructions.

Add & fetch this issue fork’s repository

Or, if you do not have SSH keys set up on git.drupalcode.org:

Add & fetch this issue fork’s repository

3346689-wrong-revision-timestamp changes, plain diff MR !3
Check out this branch

About issue forks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

8 March 2023 at 11:42

nico.b created an issue. See original summary.

Comment #2

8 March 2023 at 11:50

nico.b opened merge request !3

Comment #3

nico.b CreditAttribution: nico.b as a volunteer commented 8 March 2023 at 11:52

Status:

Active

» Needs review

I opened a merge request addressing the issue. I propose to just update the revision time every time a new revision is set since I cannot think of a scenario where you would not want to have this executed automatically.