Skip to content

Commit 8b035b2

Browse files
committed
[JENKINS-62648] Handle null change query string
When using a legacy SCM definition, the query string is null and therefore needs considered as empty. Change-Id: I6237615bc347265bf037bf7d6d0e81161fac37a3
1 parent b1cf12b commit 8b035b2

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/main/java/jenkins/plugins/gerrit/AbstractGerritSCMSource.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -728,7 +728,12 @@ protected GerritChecksApi createGerritChecksApi(
728728
private Changes.QueryRequest getOpenChanges(
729729
GerritApi gerritApi, String project, String changeQueryFilter)
730730
throws UnsupportedEncodingException {
731-
String query = "p:" + project + " status:open " + OPEN_CHANGES_FILTER + " " + changeQueryFilter;
731+
String query =
732+
"p:"
733+
+ project
734+
+ " status:open "
735+
+ OPEN_CHANGES_FILTER
736+
+ (changeQueryFilter == null ? "" : " " + changeQueryFilter);
732737
return gerritApi
733738
.changes()
734739
.query(URLEncoder.encode(query, StandardCharsets.UTF_8.name()))

0 commit comments

Comments
 (0)