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

Error when selecting when / changeRequest or tag in the Directive Generator

    XMLWordPrintable

Details

    Description

      No idea what's going on here :shrug:

      Attachments

        Issue Links

          Activity

            bitwiseman Liam Newman added a comment -

            Bulk closing resolved issues.

            bitwiseman Liam Newman added a comment - Bulk closing resolved issues.
            abayer Andrew Bayer added a comment -

            Merged, releasing today in Declarative 1.2.9.

            abayer Andrew Bayer added a comment - Merged, releasing today in Declarative 1.2.9.

            Code changed in jenkins
            User: Andrew Bayer
            Path:
            pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/generator/WhenDirective.java
            pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/when/impl/BuildingTagConditional.java
            pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/when/impl/ChangeRequestConditional.java
            pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/when/impl/TagConditional.java
            pipeline-model-definition/src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/generator/WhenDirective/config.jelly
            pipeline-model-definition/src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/when/impl/ChangeRequestConditional/config.jelly
            http://jenkins-ci.org/commit/pipeline-model-definition-plugin/eac90be72124d7ef32d957af898a5eef913f478b
            Log:
            [FIXED JENKINS-50682] Actually pass descriptor to when condition forms

            Also add display name for a few when conditions, move all of
            ChangeRequestConditional's fields into advanced, since they're
            optional and all, and smack things around a bit internally to not
            display when directive parameters when they're not required and
            they've got an empty string value.

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Andrew Bayer Path: pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/generator/WhenDirective.java pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/when/impl/BuildingTagConditional.java pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/when/impl/ChangeRequestConditional.java pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/when/impl/TagConditional.java pipeline-model-definition/src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/generator/WhenDirective/config.jelly pipeline-model-definition/src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/when/impl/ChangeRequestConditional/config.jelly http://jenkins-ci.org/commit/pipeline-model-definition-plugin/eac90be72124d7ef32d957af898a5eef913f478b Log: [FIXED JENKINS-50682] Actually pass descriptor to when condition forms Also add display name for a few when conditions, move all of ChangeRequestConditional's fields into advanced, since they're optional and all, and smack things around a bit internally to not display when directive parameters when they're not required and they've got an empty string value.
            abayer Andrew Bayer added a comment -

            Got it - PR up at https://github.com/jenkinsci/pipeline-model-definition-plugin/pull/261 fixing that and making a few other changes to when condition directive generation.

            abayer Andrew Bayer added a comment - Got it - PR up at https://github.com/jenkinsci/pipeline-model-definition-plugin/pull/261 fixing that and making a few other changes to when condition directive generation.
            abayer Andrew Bayer added a comment -

            tag barfs too, but you have to open the "Advanced" options to get it to do so.

            abayer Andrew Bayer added a comment - tag barfs too, but you have to open the "Advanced" options to get it to do so.

            People

              abayer Andrew Bayer
              rtyler R. Tyler Croy
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: