Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-43507] Fix the <scm:traits> taglib
  • Loading branch information
stephenc committed Jun 12, 2017
1 parent 01465e3 commit 7f32a8b
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 8 deletions.
6 changes: 3 additions & 3 deletions src/main/java/jenkins/plugins/git/AbstractGitSCMSource.java
Expand Up @@ -132,7 +132,7 @@ public abstract class AbstractGitSCMSource extends SCMSource {
* <p>
* The template uses {@code @{...}} as that is an illegal sequence in a remote name
*
* @see <a href="https://github.com/git/git/blob/027a3b943b444a3e3a76f9a89803fc10245b858f/refs.c#L61-L68>git
* @see <a href="https://github.com/git/git/blob/027a3b943b444a3e3a76f9a89803fc10245b858f/refs.c#L61-L68">git
* source code rules on ref spec names</a>
* @since 3.4.0
*/
Expand Down Expand Up @@ -266,7 +266,7 @@ public List<GitSCMExtension> getExtensions() {
/**
* Returns the {@link SCMSourceTrait} instances for this {@link AbstractGitSCMSource}.
* @return the {@link SCMSourceTrait} instances
* @sinec 3.4.0
* @since 3.4.0
*/
@NonNull
public List<SCMSourceTrait> getTraits() {
Expand Down Expand Up @@ -729,7 +729,7 @@ protected GitSCMBuilder<?> newBuilder(@NonNull SCMHead head, @CheckForNull SCMRe
* {@link #build(SCMHead, SCMRevision)} immediately prior to returning {@link GitSCMBuilder#build()}.
* Subclasses should override this method if they need to overrule builder behaviours defined by traits.
*
* @param builder
* @param builder the builder to decorate.
*/
protected void decorate(GitSCMBuilder<?> builder) {
}
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/jenkins/plugins/git/GitSCMSource.java
Expand Up @@ -501,7 +501,7 @@ public List<SCMSourceTraitDescriptor> getTraitDescriptors() {
return SCMSourceTrait._for(this, GitSCMSourceContext.class, GitSCMBuilder.class);
}

public List<SCMSourceTrait> getDefaultTraits() {
public List<SCMSourceTrait> getTraitDefaults() {
return Collections.emptyList();
}
}
Expand Down
Expand Up @@ -23,16 +23,14 @@
~ THE SOFTWARE.
-->

<j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:c="/lib/credentials">
<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form" xmlns:c="/lib/credentials" xmlns:scm="/jenkins/scm/api/form">
<f:entry title="${%Project Repository}" field="remote">
<f:textbox/>
</f:entry>
<f:entry title="${%Credentials}" field="credentialsId">
<c:select/>
</f:entry>
<f:entry title="${%Behaviours}">
<f:hetero-list name="traits" items="${instance == null ? descriptor.defaultTraits : instance.traits}"
descriptors="${descriptor[attrs.field+'Descriptors'] ?: descriptor.getPropertyType(instance,'traits').getApplicableItemDescriptors()}"
hasHeader="true" honorOrder="true" oneEach="true"/>
<scm:traits field="traits"/>
</f:entry>
</j:jelly>

0 comments on commit 7f32a8b

Please sign in to comment.