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.

Sign up for GitHub

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

Jump to bottom

Descriptors configuration #2

Closed
ndeloof opened this issue Nov 10, 2017 · 1 comment
Closed

Descriptors configuration #2

ndeloof opened this issue Nov 10, 2017 · 1 comment

Comments

@ndeloof
Copy link
Contributor

ndeloof commented Nov 10, 2017 •
edited
Loading

as discussed on mailing list
We have no way to guess Descriptors data model, until they move to DataBoundSetters to "document" their UI databinding.
But this is not enough : Descriptor are mutable objects, and DataBound mechanism don't reset object so some optional attribute becomes null if not-set on UI. So it might be necessary to ask plugin developers to implement the configure(StaplerRequest, JSONObject) method as:

  • reset everything to default values
  • invoke data binding req.bind(json)
  • save descriptor save()

Maybe there's some way to make this the default mechanism ?
@ndeloof
Copy link
Contributor Author

ndeloof commented Nov 15, 2017

see https://issues.jenkins-ci.org/browse/JENKINS-48018

This was referenced Nov 23, 2017
Merged
Closed
@ndeloof ndeloof closed this as completed Jan 8, 2018
LinuxSuRen pushed a commit to LinuxSuRen/configuration-as-code-plugin that referenced this issue Dec 24, 2019
Merge pull request jenkinsci#2 from LinuxSuRen/auto-load
76cf8e5 
Add support to backup and restore automatically
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ndeloof