diff options
| author | Jacek Centkowski <geminica.programs@gmail.com> | 2024-03-27 07:53:43 +0100 |
|---|---|---|
| committer | Jacek Centkowski <geminica.programs@gmail.com> | 2024-03-27 07:58:40 +0100 |
| commit | 18fef7ea6d64d1b4be766eab5aecde2ef3e4fd84 (patch) | |
| tree | 9ca16552592e2be9410e2eaaf119ae7713ed89f1 | |
| parent | e02812b521d09a314af95f2804c69d2d1cef67df (diff) | |
Use virtual change id for account patchset reviewdb clear action
So that reviewed flags are removed from the correct change.
Release-Notes: Ensure that clearing reviewed flags is performed on correct change when it is imported from another instance
Forward-Compatible: checked
Change-Id: I27899f76f804d41eacdf9a0f81fc74c012ff2c03
| -rw-r--r-- | java/com/google/gerrit/server/change/DeleteChangeOp.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/java/com/google/gerrit/server/change/DeleteChangeOp.java b/java/com/google/gerrit/server/change/DeleteChangeOp.java index ac751655eb..4ac27c1376 100644 --- a/java/com/google/gerrit/server/change/DeleteChangeOp.java +++ b/java/com/google/gerrit/server/change/DeleteChangeOp.java @@ -125,7 +125,7 @@ public class DeleteChangeOp implements BatchUpdateOp { } private void cleanUpReferences(ChangeData cd) throws IOException { - accountPatchReviewStore.run(s -> s.clearReviewed(cd.getId())); + accountPatchReviewStore.run(s -> s.clearReviewed(cd.virtualId())); // Non-atomic operation on All-Users refs; not much we can do to make it atomic. starredChangesUtil.unstarAllForChangeDeletion(cd.virtualId()); |