commit f246801bf4c7b76f7a60149b5ea3a0ba8efa834c
author Dave Borowitz <dborowitz@google.com> Tue Aug 30 10:58:49 2016 -0400
committer David Pursehouse <dpursehouse@collab.net> Wed Oct 12 23:30:38 2016 +0000
tree 632a8bf05cd92a727c25cbdccfa9ea63a8adbe3b
parent c09142a2e562da9ad94cd2206b2ef343ef6b9513

EventUtil: Don't include CHECK option to ChangeJson

This option has the special behavior of suppressing some exceptions on
corrupt changes, which is not appropriate for passing through the
event system.

Change-Id: If2316b64fce5e24b43e74b3de32efa43da676865

gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/EventUtil.java

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/EventUtil.java b/gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/EventUtil.java
index 51ee3ac..e519410 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/EventUtil.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/EventUtil.java
@@ -55,8 +55,9 @@
       Provider<ReviewDb> db) {
     this.changeDataFactory = changeDataFactory;
     this.db = db;
-    this.changeJson = changeJsonFactory.create(
-        EnumSet.allOf(ListChangesOption.class));
+    EnumSet<ListChangesOption> opts = EnumSet.allOf(ListChangesOption.class);
+    opts.remove(ListChangesOption.CHECK);
+    this.changeJson = changeJsonFactory.create(opts);
   }
 
   public ChangeInfo changeInfo(Change change) throws OrmException {