New issue
Advanced search Search tips
Starred by 4 users

Issue metadata

Status: Released
Owner:
Closed: Sep 14



Sign in to add a comment
link

Issue 9711: Add a ChangeDeletedListener to Gerrit. and hooks plugin

Reported by matt...@unsolvable.org, Sep 12

Issue description

Gerrit (and by extension, the hooks core plugin) supports a ChangeAbandonedListener and ChangeRestoredListener.
These were added in https://gerrit-review.googlesource.com/c/gerrit/+/72215.

However, there's no ChangeDeletedListener. Note that the ability to Delete Changes was added in 2.14.

Gerrit should support a ChangeDeletedListener, analogous to ChangeAbandonedListener and ChangeRestoredListener.

Ideally, it should make available, to an onChangeDeleted implementation, all the details of the original change (I am specifically interested in knowing the topic of the deleted change). A ChangeInfo would be the natural way to do this, but I'm not sure whether that is possible, since at the time the listener fires, the change itself is already deleted.
 

Comment 1 by david.pu...@gmail.com, Sep 12

Owner: david.pu...@gmail.com
Status: Accepted (was: New)
I've uploaded a WIP change to implement the event:

https://gerrit-review.googlesource.com/c/gerrit/+/195717

Comment 3 by david.pu...@gmail.com, Sep 14

Status: Started (was: Accepted)

Comment 5 by david.pu...@gmail.com, Sep 14

Labels: FixedIn-2.14.13
Status: Submitted (was: ChangeUnderReview)

Comment 6 by david.pu...@gmail.com, Sep 21

Status: Released (was: Submitted)

Comment 7 by david.pu...@gmail.com, Sep 26

Labels: FixedIn-2.15.4

Sign in to add a comment