Skip to content

Commit 421d838

Browse files
committed
Merge pull request hudson2-plugins#37 from oldelvet/jenkins-10272
[FIXED JENKINS-10272] Global git config name and email address settings not used.
2 parents 7109f29 + 7424a9a commit 421d838

1 file changed

Lines changed: 10 additions & 16 deletions

File tree

src/main/java/hudson/plugins/git/GitSCM.java

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -425,27 +425,21 @@ public String getGitConfigEmail() {
425425
}
426426

427427
public String getGitConfigNameToUse() {
428-
String confName;
429-
String globalConfigName = ((DescriptorImpl) getDescriptor()).getGlobalConfigName();
430-
if ((fixEmptyAndTrim(globalConfigName) != null) && (gitConfigName == null)) {
431-
confName = globalConfigName;
432-
} else {
433-
confName = gitConfigName;
428+
String confName = fixEmptyAndTrim(gitConfigName);
429+
if (confName == null) {
430+
String globalConfigName = ((DescriptorImpl) getDescriptor()).getGlobalConfigName();
431+
confName = fixEmptyAndTrim(globalConfigName);
434432
}
435-
436-
return fixEmptyAndTrim(confName);
433+
return confName;
437434
}
438435

439436
public String getGitConfigEmailToUse() {
440-
String confEmail;
441-
String globalConfigEmail = ((DescriptorImpl) getDescriptor()).getGlobalConfigEmail();
442-
if ((fixEmptyAndTrim(globalConfigEmail) != null) && (gitConfigEmail == null)) {
443-
confEmail = globalConfigEmail;
444-
} else {
445-
confEmail = gitConfigEmail;
437+
String confEmail = fixEmptyAndTrim(gitConfigEmail);
438+
if (confEmail == null) {
439+
String globalConfigEmail = ((DescriptorImpl) getDescriptor()).getGlobalConfigEmail();
440+
confEmail = fixEmptyAndTrim(globalConfigEmail);
446441
}
447-
448-
return fixEmptyAndTrim(confEmail);
442+
return confEmail;
449443
}
450444

451445
public boolean getSkipTag() {

0 commit comments

Comments
 (0)