Skip to content

Commit 3858026

Browse files
authored
Merge branch 'master' into fix/missing-defined-variables
2 parents 947445a + dd23f4c commit 3858026

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/main/java/com/dabsquared/gitlabjenkins/util/CommitStatusUpdater.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,12 @@ public static void updateCommitStatus(
5959
return;
6060
}
6161

62+
EnvVars environment = null;
6263
if (gitLabBranchBuilds == null || gitLabBranchBuilds.isEmpty()) {
6364
try {
64-
if (!build.getEnvironment(listener).isEmpty()) {
65-
gitLabBranchBuilds = retrieveGitlabProjectIds(build, build.getEnvironment(listener));
65+
environment = build.getEnvironment(listener);
66+
if (!environment.isEmpty()) {
67+
gitLabBranchBuilds = retrieveGitlabProjectIds(build, environment);
6668
}
6769
} catch (IOException | InterruptedException e) {
6870
printf(listener, "Failed to get Gitlab Build list to update status: %s%n", e.getMessage());
@@ -96,7 +98,7 @@ public static void updateCommitStatus(
9698
gitLabBranchBuild.getProjectId(),
9799
gitLabBranchBuild.getRevisionHash(),
98100
state,
99-
getBuildBranchOrTag(build),
101+
getBuildBranchOrTag(build, environment),
100102
current_build_name,
101103
buildUrl,
102104
state.name());
@@ -155,10 +157,10 @@ private static boolean existsCommit(GitLabClient client, String gitlabProjectId,
155157
}
156158
}
157159

158-
private static String getBuildBranchOrTag(Run<?, ?> build) {
160+
private static String getBuildBranchOrTag(Run<?, ?> build, EnvVars environment) {
159161
GitLabWebHookCause cause = build.getCause(GitLabWebHookCause.class);
160162
if (cause == null) {
161-
return null;
163+
return environment == null ? null : environment.get("BRANCH_NAME", null);
162164
}
163165
if (cause.getData().getActionType() == CauseData.ActionType.TAG_PUSH) {
164166
return StringUtils.removeStart(cause.getData().getSourceBranch(), "refs/tags/");

0 commit comments

Comments
 (0)