diff --git a/src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTrigger.java b/src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTrigger.java index 09af51dff..d7d841584 100644 --- a/src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTrigger.java +++ b/src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTrigger.java @@ -46,6 +46,7 @@ import com.sonyericsson.hudson.plugins.gerrit.trigger.version.GerritVersionChecker; import com.sonymobile.tools.gerrit.gerritevents.GerritHandler; import com.sonymobile.tools.gerrit.gerritevents.GerritQueryHandler; +import com.sonymobile.tools.gerrit.gerritevents.dto.GerritChangeStatus; import com.sonymobile.tools.gerrit.gerritevents.dto.attr.Approval; import com.sonymobile.tools.gerrit.gerritevents.dto.attr.Change; import com.sonymobile.tools.gerrit.gerritevents.dto.attr.PatchSet; @@ -925,7 +926,13 @@ public boolean equals(Object obj) { * @return true if we should. */ private boolean isChangeInteresting(Change change, GerritProject project, GerritQueryHandler gerritQueryHandler) { + boolean shouldTrigger = false; + + if (change.getStatus() != GerritChangeStatus.NEW ) { + return shouldTrigger; // return false immediately if it's a non-NEW status change + } + boolean containsFilePathsOrForbiddenFilePaths = ((project.getFilePaths() != null && project.getFilePaths().size() > 0) || (project.getForbiddenFilePaths() != null && project.getForbiddenFilePaths().size() > 0));