Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-36502] Fix Pipeline Snippet Generator #880

Closed
wants to merge 1 commit into from

Conversation

bitwiseman
Copy link
Contributor

@daspilker
The snippet generator is apparently rather picky about databound values.
If there are databound setters without getters/public-fields, it just returns a class description.

Fix for issue: https://issues.jenkins-ci.org/browse/JENKINS-36502

The snippet generator has specific requirements for databound values.
If there are databound setters without getters/public-fields, it throws and exeception
to the log and generates only a class description for parameters.

Fix for issue: https://issues.jenkins-ci.org/browse/JENKINS-36502
@jglick jglick changed the title Fix Pipeline Snippet Generator [JENKINS-36502] Fix Pipeline Snippet Generator Jul 28, 2016
@@ -62,6 +62,7 @@ dependencies {
compile project(':job-dsl-core')
compile 'org.jenkins-ci:symbol-annotation:1.1'
jenkinsPlugins 'org.jenkins-ci.plugins:structs:1.2@jar'
jenkinsPlugins 'org.jenkins-ci.plugins.workflow:workflow-step-api:1.15@jar'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why?

@jglick
Copy link
Member

jglick commented Jul 28, 2016

I feel like there is something more deeply broken in the databinding in this plugin that the PR is just papering over. Why does it not just use standard patterns?

@jglick
Copy link
Member

jglick commented Jul 28, 2016

Oh I see. Close this. The problem is ScriptLocation. It needs to be a Describable with its own config.jelly, and the databinding in this builder needs to be overhauled. See ui-samples-plugin for examples.

@jglick
Copy link
Member

jglick commented Jul 28, 2016

e.g. use f:optionalProperty

@daspilker
Copy link
Member

@jglick Thanks for the review. I wanted to get rid of ScriptLocation anyway...

Replaced by #893

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants