Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-43507] Found some minor bugs in the form validation action m…
…ethod names
  • Loading branch information
stephenc committed Jun 23, 2017
1 parent e9df86e commit 8c5447b
Showing 1 changed file with 15 additions and 2 deletions.
Expand Up @@ -1346,6 +1346,17 @@ public String getDisplayName() {
return Messages.GitHubSCMSource_DisplayName();
}

public ListBoxModel doFillCredentialsIdItems(@CheckForNull @AncestorInPath Item context,
@QueryParameter String apiUri) {
return Connector.listScanCredentials(context, apiUri);
}

public FormValidation doCheckCredentialsId(@CheckForNull @AncestorInPath Item context,
@QueryParameter String apiUri,
@QueryParameter String value) {
return Connector.checkScanCredentials(context, apiUri, value);
}

@Restricted(NoExternalUse.class)
public FormValidation doCheckIncludes(@QueryParameter String value) {
if (value.isEmpty()) {
Expand All @@ -1358,7 +1369,7 @@ public FormValidation doCheckIncludes(@QueryParameter String value) {
public FormValidation doCheckScanCredentialsId(@CheckForNull @AncestorInPath Item context,
@QueryParameter String apiUri,
@QueryParameter String scanCredentialsId) {
return Connector.checkScanCredentials(context, apiUri, scanCredentialsId);
return doCheckCredentialsId(context, apiUri, scanCredentialsId);
}

@Restricted(NoExternalUse.class)
Expand Down Expand Up @@ -1419,12 +1430,14 @@ public boolean isApiUriSelectable() {
return !GitHubConfiguration.get().getEndpoints().isEmpty();
}

@Restricted(NoExternalUse.class)
public ListBoxModel doFillCheckoutCredentialsIdItems(@CheckForNull @AncestorInPath Item context, @QueryParameter String apiUri) {
return Connector.listCheckoutCredentials(context, apiUri);
}

@Restricted(NoExternalUse.class)
public ListBoxModel doFillScanCredentialsIdItems(@CheckForNull @AncestorInPath Item context, @QueryParameter String apiUri) {
return Connector.listScanCredentials(context, apiUri);
return doFillCredentialsIdItems(context, apiUri);
}

public ListBoxModel doFillRepositoryItems(@CheckForNull @AncestorInPath Item context, @QueryParameter String apiUri,
Expand Down

0 comments on commit 8c5447b

Please sign in to comment.