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

Configure trend graph do not save # days setting

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • Minor
    • Resolution: Fixed
    • analysis-core-plugin
    • None
    • Windows 2003 Server
      Java 1.6.0_16
      Hudson 1.356
      Checkstyle Plug-in 3.6
      Static Analysis Utilities 1.8

    Description

      The Checkstyle trend configuration page do not save all values.

      Steps to reproduce:
      1)Click on the Configure link below the Checkstyle Trend graph on the Project page.
      2)Change the value for width from 500 to 400, and set the # days value to 21
      3)Click the save button
      4)Click again on the Configure link below the Checkstyle Trend graph on the Project page

      Expected:
      The width and days field should contain the newly set values.

      Actual result:
      The width field has the correct value. The days field are empty.

      Attachments

        Activity

          Ascending order - Click to sort in descending order
          amh Anders Hansen created issue -
          drulli Ulli Hafner added a comment -

          Actually the values are saved and used during the computation. Only the configuration view does not show the persisted values.

          drulli Ulli Hafner added a comment - Actually the values are saved and used during the computation. Only the configuration view does not show the persisted values.
          drulli Ulli Hafner made changes -
          Field Original Value New Value
          Priority Major [ 3 ] Minor [ 4 ]
          drulli Ulli Hafner made changes -
          Component/s analysis-core [ 15709 ]
          Component/s checkstyle [ 15512 ]
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : drulli
          Path:
          trunk/hudson/plugins/analysis-core/src/main/java/hudson/plugins/analysis/graph/GraphConfigurationView.java
          trunk/hudson/plugins/analysis-core/src/main/java/hudson/plugins/analysis/util/CookieHandler.java
          http://jenkins-ci.org/commit/30818
          Log:
          [FIXED JENKINS-6466] Fixed display of #days and #build values in configuration view.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : drulli Path: trunk/hudson/plugins/analysis-core/src/main/java/hudson/plugins/analysis/graph/GraphConfigurationView.java trunk/hudson/plugins/analysis-core/src/main/java/hudson/plugins/analysis/util/CookieHandler.java http://jenkins-ci.org/commit/30818 Log: [FIXED JENKINS-6466] Fixed display of #days and #build values in configuration view.
          scm_issue_link SCM/JIRA link daemon made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          drulli Ulli Hafner added a comment -

          This still seems to be broken for the default configuration (via .txt file and not via cookie).

          drulli Ulli Hafner added a comment - This still seems to be broken for the default configuration (via .txt file and not via cookie).
          drulli Ulli Hafner made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : drulli
          Path:
          trunk/hudson/plugins/analysis-core/src/main/java/hudson/plugins/analysis/graph/GraphConfiguration.java
          http://jenkins-ci.org/commit/31180
          Log:
          [FIXED JENKINS-6466] Added default value of 30 days and 50 builds when computing the trend graph.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : drulli Path: trunk/hudson/plugins/analysis-core/src/main/java/hudson/plugins/analysis/graph/GraphConfiguration.java http://jenkins-ci.org/commit/31180 Log: [FIXED JENKINS-6466] Added default value of 30 days and 50 builds when computing the trend graph.
          scm_issue_link SCM/JIRA link daemon made changes -
          Resolution Fixed [ 1 ]
          Status Reopened [ 4 ] Resolved [ 5 ]
          amh Anders Hansen made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 136561 ] JNJira + In-Review [ 204104 ]

          People

            drulli Ulli Hafner
            amh Anders Hansen
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: