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

Apply button behaviour broken

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      What incorrect behavior happened?

      Lost all changes in a job

      Environment
      O.S.: Linux,Mac OSX
      CJE version: Jenkins ver. 2.7.19
      Web browser:
      JDK version: JDK8

      What was the expected outcome?

      Should save the changes

      Step by step to reproduce it

      1. Create a Job
      2. Open a terminal and execute an script to read every 1 s the config.xml of the job
      3. Add one step to the job and check the terminal to see if the step is in the config
      4. Repeat the previous step until all steps was removed

      The script

      while [ true ]; do curl http://cje.example.com:8080/job/fsj/config.xml; sleep 1s;  done
      

      Video of how to do that

      https://drive.google.com/open?id=0Bz36P9uZT4syR0V0RGFNbzJubzQ

        Attachments

          Issue Links

            Activity

            Hide
            ifernandezcalvo Ivan Fernandez Calvo added a comment -

            I trace the issue to this plugin Form element path plugin , if I disabled it the issue disappear

            Show
            ifernandezcalvo Ivan Fernandez Calvo added a comment - I trace the issue to this plugin Form element path plugin , if I disabled it the issue disappear
            Hide
            vlatombe Vincent Latombe added a comment -

            The problem is the following: If you submit a configuration change through 'Apply', then the next time 'Save' Or 'Apply' is clicked, the submitted list of builders is empty.

            Show
            vlatombe Vincent Latombe added a comment - The problem is the following: If you submit a configuration change through 'Apply', then the next time 'Save' Or 'Apply' is clicked, the submitted list of builders is empty.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            Jenkinsfile
            src/main/resources/org/jenkinsci/plugins/form-element-path.js
            http://jenkins-ci.org/commit/form-element-path-plugin/e1b232636c6a45715d1f33150ff5953861ecddd0
            Log:
            JENKINS-38928 Handle radio groups

            This fixes the form handling issue after 'Apply' is clicked.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: Jenkinsfile src/main/resources/org/jenkinsci/plugins/form-element-path.js http://jenkins-ci.org/commit/form-element-path-plugin/e1b232636c6a45715d1f33150ff5953861ecddd0 Log: JENKINS-38928 Handle radio groups This fixes the form handling issue after 'Apply' is clicked.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            Jenkinsfile
            src/main/resources/org/jenkinsci/plugins/form-element-path.js
            http://jenkins-ci.org/commit/form-element-path-plugin/4a72ea5cf83380cd44ac1c348452f165d8bf3f1f
            Log:
            Merge pull request #6 from Vlatombe/JENKINS-38928

            JENKINS-38928 Fix apply behaviour when form contains radio group

            Compare: https://github.com/jenkinsci/form-element-path-plugin/compare/a4518681c91a...4a72ea5cf833

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: Jenkinsfile src/main/resources/org/jenkinsci/plugins/form-element-path.js http://jenkins-ci.org/commit/form-element-path-plugin/4a72ea5cf83380cd44ac1c348452f165d8bf3f1f Log: Merge pull request #6 from Vlatombe/ JENKINS-38928 JENKINS-38928 Fix apply behaviour when form contains radio group Compare: https://github.com/jenkinsci/form-element-path-plugin/compare/a4518681c91a...4a72ea5cf833
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            src/main/java/org/jenkinsci/test/acceptance/po/ConfigurablePageObject.java
            src/test/java/core/FreestyleJobTest.java
            http://jenkins-ci.org/commit/acceptance-test-harness/4721f1d188c6315b60c5ca9fac6e7495abe54fef
            Log:
            JENKINS-38928 Add a test doing apply then save in a job

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: src/main/java/org/jenkinsci/test/acceptance/po/ConfigurablePageObject.java src/test/java/core/FreestyleJobTest.java http://jenkins-ci.org/commit/acceptance-test-harness/4721f1d188c6315b60c5ca9fac6e7495abe54fef Log: JENKINS-38928 Add a test doing apply then save in a job
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            src/main/java/org/jenkinsci/test/acceptance/po/ConfigurablePageObject.java
            http://jenkins-ci.org/commit/acceptance-test-harness/0e6815addd8e22b904f18578d64fb0db003ca505
            Log:
            JENKINS-38928 Wait for splash to appear

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: src/main/java/org/jenkinsci/test/acceptance/po/ConfigurablePageObject.java http://jenkins-ci.org/commit/acceptance-test-harness/0e6815addd8e22b904f18578d64fb0db003ca505 Log: JENKINS-38928 Wait for splash to appear
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            src/main/java/org/jenkinsci/test/acceptance/po/ConfigurablePageObject.java
            http://jenkins-ci.org/commit/acceptance-test-harness/d20f235ad5f18831ac6333155e7b7c48cb718ad2
            Log:
            JENKINS-38928 Simplify assertion

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: src/main/java/org/jenkinsci/test/acceptance/po/ConfigurablePageObject.java http://jenkins-ci.org/commit/acceptance-test-harness/d20f235ad5f18831ac6333155e7b7c48cb718ad2 Log: JENKINS-38928 Simplify assertion
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            src/main/java/org/jenkinsci/test/acceptance/FallbackConfig.java
            http://jenkins-ci.org/commit/acceptance-test-harness/057ae2fcd8e9cf46ddda357f8068addecda10e3e
            Log:
            JENKINS-38928 Pull form-element-path 1.8

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: src/main/java/org/jenkinsci/test/acceptance/FallbackConfig.java http://jenkins-ci.org/commit/acceptance-test-harness/057ae2fcd8e9cf46ddda357f8068addecda10e3e Log: JENKINS-38928 Pull form-element-path 1.8
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Vincent Latombe
            Path:
            pom.xml
            src/main/java/org/jenkinsci/test/acceptance/FallbackConfig.java
            src/main/java/org/jenkinsci/test/acceptance/po/ConfigurablePageObject.java
            src/test/java/core/FreestyleJobTest.java
            http://jenkins-ci.org/commit/acceptance-test-harness/5f4c2b81cea283b5397b0b3bc40120609a0921a1
            Log:
            Merge pull request #210 from Vlatombe/JENKINS-38928

            JENKINS-38928 Add a test doing apply then save in a job

            Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/fdd33e12599c...5f4c2b81cea2

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Vincent Latombe Path: pom.xml src/main/java/org/jenkinsci/test/acceptance/FallbackConfig.java src/main/java/org/jenkinsci/test/acceptance/po/ConfigurablePageObject.java src/test/java/core/FreestyleJobTest.java http://jenkins-ci.org/commit/acceptance-test-harness/5f4c2b81cea283b5397b0b3bc40120609a0921a1 Log: Merge pull request #210 from Vlatombe/ JENKINS-38928 JENKINS-38928 Add a test doing apply then save in a job Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/fdd33e12599c...5f4c2b81cea2
            Hide
            vlatombe Vincent Latombe added a comment -

            Released in 1.8

            Show
            vlatombe Vincent Latombe added a comment - Released in 1.8

              People

              Assignee:
              vlatombe Vincent Latombe
              Reporter:
              recampbell Ryan Campbell
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: