2017-06-16 12:19 GMT+02:00 Christian McHugh <[email protected]>:

> On Friday, June 16, 2017 at 11:12:27 AM UTC+1, Robert Sandell wrote:
>>
>> The XStream deserialization from config.xml to the object graph don't use
>> the DataboundConstructor et.al. It is only interested in mapping xml
>> data to fields (in most cases).
>>
>> DataboundConstructor is only used for formbinding from the UI and
>> pipeline step definitions etc.
>>
>
> So a ConverterImpl should be able to update the config properly? At the
> end of my example from the first part of the thread I had a
> system.out.println which displays the desired config, but when hitting the
> convert button in Jenkins, those config.xml files are written without a
> build step. Would this mean I just need to make a better ConverterImpl?
>

Maybe, or you might just haven't registered the converter in the correct
places. But I would urge you to first try using plain old readResolve to
handle the conversion from old data before attempting implementing an
XStream converter.

> --
> You received this message because you are subscribed to the Google Groups
> "Jenkins Developers" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> To view this discussion on the web visit https://groups.google.com/d/
> msgid/jenkinsci-dev/3db7ba8c-7f22-4e06-88ba-8bb048e0720d%
> 40googlegroups.com
> <https://groups.google.com/d/msgid/jenkinsci-dev/3db7ba8c-7f22-4e06-88ba-8bb048e0720d%40googlegroups.com?utm_medium=email&utm_source=footer>
> .
>
> For more options, visit https://groups.google.com/d/optout.
>



-- 
Robert Sandell
*Software Engineer*
*CloudBees Inc.*

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-dev/CALzHZS2jd9%3DBv%2BCjpmkXC7nBuMQ-w6eH4erG0m1-SfYyojQsWQ%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to