build completed for RefUpdated events with REST

If REST API enabled, sending of build completed notifications for RefUpdated events are incorrectly attempted. [FIXED JENKINS-31781]
jenkinsci · Nov 27, 2015 · b64d4d7 · b64d4d7
1 parent 3bfe912
commit b64d4d7
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...va/com/sonyericsson/hudson/plugins/gerrit/trigger/gerritnotifier/NotificationFactory.java b/...va/com/sonyericsson/hudson/plugins/gerrit/trigger/gerritnotifier/NotificationFactory.java
@@ -120,7 +120,8 @@ public void queueBuildCompleted(BuildMemory.MemoryImprint memoryImprint, TaskLis
         if (serverName != null) {
             IGerritHudsonTriggerConfig config = getConfig(serverName);
             if (config != null) {
-                if (config.isUseRestApi()) {
+                if (config.isUseRestApi()
+                        && memoryImprint.getEvent() instanceof ChangeBasedEvent) {
                     GerritSendCommandQueue.queue(new BuildCompletedRestCommandJob(config, memoryImprint, listener));
                 } else {
                     GerritSendCommandQueue.queue(new BuildCompletedCommandJob(config, memoryImprint, listener));