Problem/Motivation

It would be helpful to see which log caused an asset to be archived in the asset's revision log messages.

Proposed resolution

Set a revision log message on asset(s) that are terminated by a termination log.

Remaining tasks

Open MR.

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork farm_asset_termination-3573442

Command icon Show commands

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

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

About issue forks

Comments

m.stenta created an issue. See original summary.

m.stenta opened merge request !6

m.stenta’s picture

m.stenta
he/him
Primary language English
commented
Status: Active » Needs review

Opened a merge request. It is based on #3569829: Support farmOS 4 because it touches some of the same code so includes commits from that MR.

I had to split the current logic so that the asset archive logic happens in log insert/update, instead of presave, because that enables us to link to the log in the revision log message (we need the log ID to be created first, and that is not available in presave).

wotnak made their first commit to this issue’s fork.

wotnak’s picture

wotnak
he/him
commented
Status: Needs review » Reviewed & tested by the community

  • wotnak committed 51b68652 on 2.x authored by m.stenta 
    feat: #3573442 Create a revision log message when asset archived by...
wotnak’s picture

wotnak
he/him
commented
Status: Reviewed & tested by the community » Fixed

Released in v2.0.0.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • wotnak committed 8d0f3045 on 1.x 
    feat: #3573442 Create a revision log message when asset archived by...
wotnak’s picture

wotnak
he/him
commented

Also backported to v1 in 1.3.0.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.