Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-52906

jira-plugin: FAILED TO EXPORT hudson.plugins.jira.JiraProjectProperty

    XMLWordPrintable

    Details

    • Similar Issues:
    • Released As:
      JIRA Plugin 3.0.7

      Description

      please check the following issue over at github

      https://github.com/jenkinsci/configuration-as-code-plugin/issues/409

      seems like the getter/setters are taking and returning different objects. 

      proposal:

      • "To keep binary compatibility you could introduce a new property to access sites

       

       

        Attachments

          Activity

          Hide
          ndeloof Nicolas De Loof added a comment -

          Incompatibility comes from conflicting accessors :

          public void setSites(JiraSite site)

          vs 
          public JiraSite[] getSites()

           

          The later return an array, while the former takes a single argument (and actually don't set value, but add to internal list.

          CasC can't introspect the data type and fail to read current value.

          one could imagine to introduce DescriptorImpl#setSites(Sites[]), but doing so CasC won't be able to distinguish the two set methods :'(

           

           

           

          Show
          ndeloof Nicolas De Loof added a comment - Incompatibility comes from conflicting accessors : public void setSites(JiraSite site) vs  public JiraSite[] getSites()   The later return an array, while the former takes a single argument (and actually don't set value, but add to internal list. CasC can't introspect the data type and fail to read current value. one could imagine to introduce DescriptorImpl#setSites(Sites[]), but doing so CasC won't be able to distinguish the two set methods :'(      
          Hide
          olamy Olivier Lamy added a comment -

          Yup would good to add such method.

          any way to change CasC to prefer the method with the type returned by the getter?

          Otherwise I'm fine to merge the pr. 

          Show
          olamy Olivier Lamy added a comment - Yup would good to add such method. any way to change CasC to prefer the method with the type returned by the getter? Otherwise I'm fine to merge the pr. 
          Hide
          casz Joseph Petersen (old) added a comment -

          Created https://github.com/jenkinsci/jira-plugin/pull/168 in hopes of getting this resolved

          Show
          casz Joseph Petersen (old) added a comment - Created https://github.com/jenkinsci/jira-plugin/pull/168  in hopes of getting this resolved
          Hide
          casz Joseph Petersen (old) added a comment -

          Reworking the PR after being stale for a while

          Show
          casz Joseph Petersen (old) added a comment - Reworking the PR after being stale for a while
          Hide
          casz Joseph Petersen (old) added a comment -

          Currently awaiting review/merge

          Show
          casz Joseph Petersen (old) added a comment - Currently awaiting review/merge
          Hide
          olamy Olivier Lamy added a comment -

          Joseph Petersen (old) this has probably cause   JENKINS-57899 

          Show
          olamy Olivier Lamy added a comment - Joseph Petersen (old) this has probably cause    JENKINS-57899  
          Hide
          casz Joseph Petersen (old) added a comment -

          easy fix inc Olivier Lamy

          Show
          casz Joseph Petersen (old) added a comment - easy fix inc Olivier Lamy

            People

            Assignee:
            olamy Olivier Lamy
            Reporter:
            herrmannhinz Tobias Herrmann Hinz
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: