PostReviewersOp: Don't pass ChangeResource to factory method

The resource was only used within the op bodies, where it is more
correct to read the latest state from the ChangeContext rather than the
resource which was read outside of the transaction.

Change-Id: I742ae8d53659eab0cec03a1e233655f47c0cc27c
This commit is contained in:
Dave Borowitz
2018-10-02 12:55:28 -07:00
parent 019089eb48
commit 0564215504
2 changed files with 12 additions and 17 deletions

View File

@@ -446,7 +446,7 @@ public class PostReviewers
caller = rsrc.getUser().asIdentifiedUser();
op =
postReviewersOpFactory.create(
rsrc, this.reviewers, this.reviewersByEmail, state, notify, accountsToNotify);
this.reviewers, this.reviewersByEmail, state, notify, accountsToNotify);
this.exactMatchFound = exactMatchFound;
}