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

Pressing "Apply" in a configuration is considered a modification, rather than clearing them (regression in 2.376)

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • core
    • 2.378
    • 2.385

      Steps to reproduce:

      1. (Optionally) perform some changes in a job configuration
      2. Press Apply
      3. Reload the page / navigate away

      Expected result:

      No question whether I'm sure because of unsaved changes

      Actual result:

      Question whether I'm sure because of unsaved changes

      Works in jenkins-war-2.376-rc33039.637d001ce464.war, broken in jenkins-war-2.376-rc33040.5870838f690e.war, making (unsurprisingly) https://github.com/jenkinsci/jenkins/pull/7203 the culprit.

          [JENKINS-70112] Pressing "Apply" in a configuration is considered a modification, rather than clearing them (regression in 2.376)

          Daniel Beck created issue -
          Daniel Beck made changes -
          Description Original: *Steps to reproduce:*
           # (Optionally) perform some changes in a job configuration
           # Press Apply
           # Reload the page

          *Expected result:*

          No question whether I'm sure because of unsaved changes

          *Actual result:*

          Question whether I'm sure because of unsaved changes
          New: *Steps to reproduce:*
           # (Optionally) perform some changes in a job configuration
           # Press Apply
           # Reload the page / navigate away

          *Expected result:*

          No question whether I'm sure because of unsaved changes

          *Actual result:*

          Question whether I'm sure because of unsaved changes
          Daniel Beck made changes -
          Summary Original: Pressing "Apply" in a configuration is considered a modification (regression in 2.376) New: Pressing "Apply" in a configuration is considered a modification, rather than clearing them (regression in 2.376)
          Basil Crow made changes -
          Labels Original: regression ux New: regression ux ux-untriaged

          Basil Crow added a comment -

          Not fully triaged: the commit that introduced the problem has not been identified, nor has the author been pinged.

          Basil Crow added a comment - Not fully triaged: the commit that introduced the problem has not been identified, nor has the author been pinged.
          Daniel Beck made changes -
          Description Original: *Steps to reproduce:*
           # (Optionally) perform some changes in a job configuration
           # Press Apply
           # Reload the page / navigate away

          *Expected result:*

          No question whether I'm sure because of unsaved changes

          *Actual result:*

          Question whether I'm sure because of unsaved changes
          New: *Steps to reproduce:*
           # (Optionally) perform some changes in a job configuration
           # Press Apply
           # Reload the page / navigate away

          *Expected result:*

          No question whether I'm sure because of unsaved changes

          *Actual result:*

          Question whether I'm sure because of unsaved changes

          Works in jenkins-war-2.376-rc33039.637d001ce464.war, broken in jenkins-war-2.376-rc33040.5870838f690e.war, making (unsurprisingly) [https://github.com/jenkinsci/jenkins/pull/7203] the culprit.
          Daniel Beck made changes -
          Labels Original: regression ux ux-untriaged New: regression ux
          Tim Jacomb made changes -
          Assignee New: Tim Jacomb [ timja ]
          Tim Jacomb made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Tim Jacomb made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]

            timja Tim Jacomb
            danielbeck Daniel Beck
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: