commit 67f566c63559ac9de93063296436f9f65fa4f8e5
author David Pursehouse <dpursehouse@collab.net> Wed Jun 22 16:15:40 2016 +0900
committer Hugo Arès <hugo.ares@ericsson.com> Wed Jun 22 12:30:33 2016 +0000
tree 35a99bbd79a2b8ee90079ceb4318b9b0714054e7
parent 463072f0d9afbd558c2506ade7ec5c6125faf8b4

Adapt to changed ChangeIndexListener interface

Change-Id: I51bf0cf4659e2981db5fc4c2e163786ab5e75c7f

src/main/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandler.java

diff --git a/src/main/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandler.java b/src/main/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandler.java
index f8e2f31..1027565 100644
--- a/src/main/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandler.java
+++ b/src/main/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandler.java
@@ -16,9 +16,7 @@
 
 import com.google.common.base.Objects;
 import com.google.gerrit.extensions.annotations.PluginName;
-import com.google.gerrit.reviewdb.client.Change;
-import com.google.gerrit.server.extensions.events.ChangeIndexedListener;
-import com.google.gerrit.server.query.change.ChangeData;
+import com.google.gerrit.extensions.events.ChangeIndexedListener;
 import com.google.inject.Inject;
 
 import java.util.Collections;
@@ -43,18 +41,18 @@
   }
 
   @Override
-  public void onChangeIndexed(ChangeData cd) {
-    executeIndexTask(cd.getId(), false);
+  public void onChangeIndexed(int id) {
+    executeIndexTask(id, false);
   }
 
   @Override
-  public void onChangeDeleted(Change.Id id) {
+  public void onChangeDeleted(int id) {
     executeIndexTask(id, true);
   }
 
-  private void executeIndexTask(Change.Id id, boolean deleted) {
+  private void executeIndexTask(int id, boolean deleted) {
     if (!Context.isForwardedEvent()) {
-      SyncIndexTask syncIndexTask = new SyncIndexTask(id.get(), deleted);
+      SyncIndexTask syncIndexTask = new SyncIndexTask(id, deleted);
       if (queuedTasks.add(syncIndexTask)) {
         executor.execute(syncIndexTask);
       }

src/main/java/com/ericsson/gerrit/plugins/syncindex/Module.java

diff --git a/src/main/java/com/ericsson/gerrit/plugins/syncindex/Module.java b/src/main/java/com/ericsson/gerrit/plugins/syncindex/Module.java
index 5b278c3..314d119 100644
--- a/src/main/java/com/ericsson/gerrit/plugins/syncindex/Module.java
+++ b/src/main/java/com/ericsson/gerrit/plugins/syncindex/Module.java
@@ -14,9 +14,9 @@
 
 package com.ericsson.gerrit.plugins.syncindex;
 
+import com.google.gerrit.extensions.events.ChangeIndexedListener;
 import com.google.gerrit.extensions.registration.DynamicSet;
 import com.google.gerrit.lifecycle.LifecycleModule;
-import com.google.gerrit.server.extensions.events.ChangeIndexedListener;
 import com.google.inject.Provides;
 import com.google.inject.Scopes;
 

src/test/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandlerTest.java

diff --git a/src/test/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandlerTest.java b/src/test/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandlerTest.java
index 614d30f..6489602 100644
--- a/src/test/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandlerTest.java
+++ b/src/test/java/com/ericsson/gerrit/plugins/syncindex/IndexEventHandlerTest.java
@@ -67,7 +67,7 @@
   public void shouldIndexInRemoteOnChangeIndexedEvent() throws Exception {
     expect(restClientMock.index(CHANGE_ID)).andReturn(true);
     replayAll();
-    indexEventHandler.onChangeIndexed(cd);
+    indexEventHandler.onChangeIndexed(id.get());
     verifyAll();
   }
 
@@ -77,7 +77,7 @@
     reset(cd);
     expect(restClientMock.deleteFromIndex(CHANGE_ID)).andReturn(true);
     replayAll();
-    indexEventHandler.onChangeDeleted(id);
+    indexEventHandler.onChangeDeleted(id.get());
     verifyAll();
   }
 
@@ -86,8 +86,8 @@
     reset(poolMock);
     replayAll();
     Context.setForwardedEvent(true);
-    indexEventHandler.onChangeIndexed(cd);
-    indexEventHandler.onChangeDeleted(id);
+    indexEventHandler.onChangeIndexed(id.get());
+    indexEventHandler.onChangeDeleted(id.get());
     Context.unsetForwardedEvent();
     verifyAll();
   }
@@ -98,8 +98,8 @@
     poolMock.execute(indexEventHandler.new SyncIndexTask(CHANGE_ID, false));
     expectLastCall().once();
     replayAll();
-    indexEventHandler.onChangeIndexed(cd);
-    indexEventHandler.onChangeIndexed(cd);
+    indexEventHandler.onChangeIndexed(id.get());
+    indexEventHandler.onChangeIndexed(id.get());
     verifyAll();
   }