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

sloccount trend report only works up to last failed build

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • sloccount-plugin
    • None

      The rend report generated by the sloccount plugin is destroyed by failing builds. The trend report only displays the results up to the last failing build. This reduces the usefulness of the report as no general view over the whole project life time is possible.

        1. sloccount_build_page.png
          sloccount_build_page.png
          25 kB
        2. sloccount.log
          45 kB

          [JENKINS-9309] sloccount trend report only works up to last failed build

          Nils Caspar added a comment -

          True story... It's very annoying.

          Nils Caspar added a comment - True story... It's very annoying.

          sogabe added a comment -

          @kbrandt

          I tried your pull request.
          It seemed OK but clicking a failed build cause exception.
          See attached log and screenshot.

          sogabe added a comment - @kbrandt I tried your pull request. It seemed OK but clicking a failed build cause exception. See attached log and screenshot.

          Karsten Brandt added a comment - - edited

          @sogabe:Hello Seiji!

          Ups, this use case was not tested by me.
          I have changed the behaviour of the perform function again.
          For more details look into my new commits:

          https://github.com/kbrandt/sloccount-plugin/commit/92f5ca9855f4bbf65f95c61b4ab9ca1e606c6f37
          https://github.com/kbrandt/sloccount-plugin/commit/2d0f2d6358cad6ae144abef5e6f304c4e3e5bb94

          Please, try again!

          Many thanks in advance,
          Karsten

          Karsten Brandt added a comment - - edited @sogabe:Hello Seiji! Ups, this use case was not tested by me. I have changed the behaviour of the perform function again. For more details look into my new commits: https://github.com/kbrandt/sloccount-plugin/commit/92f5ca9855f4bbf65f95c61b4ab9ca1e606c6f37 https://github.com/kbrandt/sloccount-plugin/commit/2d0f2d6358cad6ae144abef5e6f304c4e3e5bb94 Please, try again! Many thanks in advance, Karsten

          sogabe added a comment -

          @kbrandt
          It looks good!
          I'll merge and release in a short time.

          sogabe added a comment - @kbrandt It looks good! I'll merge and release in a short time.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Karsten Brandt
          Path:
          src/main/java/hudson/plugins/sloccount/SloccountProjectAction.java
          src/main/java/hudson/plugins/sloccount/SloccountPublisher.java
          http://jenkins-ci.org/commit/sloccount-plugin/ba86876f210ffc7e210d740fd938cef7b9cf04d3
          Log:
          [FIXED JENKINS-9309]

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Karsten Brandt Path: src/main/java/hudson/plugins/sloccount/SloccountProjectAction.java src/main/java/hudson/plugins/sloccount/SloccountPublisher.java http://jenkins-ci.org/commit/sloccount-plugin/ba86876f210ffc7e210d740fd938cef7b9cf04d3 Log: [FIXED JENKINS-9309]

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Karsten Brandt
          Path:
          src/main/java/hudson/plugins/sloccount/ReportSummary.java
          src/main/java/hudson/plugins/sloccount/SloccountBuildAction.java
          src/main/java/hudson/plugins/sloccount/SloccountProjectAction.java
          src/main/java/hudson/plugins/sloccount/SloccountPublisher.java
          http://jenkins-ci.org/commit/sloccount-plugin/92f5ca9855f4bbf65f95c61b4ab9ca1e606c6f37
          Log:
          [FIXED JENKINS-9309] sloccount trend report only works up to last failed build (second version)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Karsten Brandt Path: src/main/java/hudson/plugins/sloccount/ReportSummary.java src/main/java/hudson/plugins/sloccount/SloccountBuildAction.java src/main/java/hudson/plugins/sloccount/SloccountProjectAction.java src/main/java/hudson/plugins/sloccount/SloccountPublisher.java http://jenkins-ci.org/commit/sloccount-plugin/92f5ca9855f4bbf65f95c61b4ab9ca1e606c6f37 Log: [FIXED JENKINS-9309] sloccount trend report only works up to last failed build (second version)

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Karsten Brandt
          Path:
          src/main/java/hudson/plugins/sloccount/SloccountPublisher.java
          src/main/java/hudson/plugins/sloccount/model/SloccountParser.java
          http://jenkins-ci.org/commit/sloccount-plugin/2d0f2d6358cad6ae144abef5e6f304c4e3e5bb94
          Log:
          [FIXED JENKINS-9309] sloccount trend report only works up to last failed build (3th version)

          Compare: https://github.com/jenkinsci/sloccount-plugin/compare/2a42be3148e1...2d0f2d6358ca

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Karsten Brandt Path: src/main/java/hudson/plugins/sloccount/SloccountPublisher.java src/main/java/hudson/plugins/sloccount/model/SloccountParser.java http://jenkins-ci.org/commit/sloccount-plugin/2d0f2d6358cad6ae144abef5e6f304c4e3e5bb94 Log: [FIXED JENKINS-9309] sloccount trend report only works up to last failed build (3th version) Compare: https://github.com/jenkinsci/sloccount-plugin/compare/2a42be3148e1...2d0f2d6358ca

          Michal Turek added a comment -

          Closing old non-closed but resolved tasks. Fixed in version 1.8 according to wiki, this type of issue fixed in version 1.11 too.

          Michal Turek added a comment - Closing old non-closed but resolved tasks. Fixed in version 1.8 according to wiki, this type of issue fixed in version 1.11 too.

            ben25 Karsten Brandt
            languitar Johannes Wienke
            Votes:
            5 Vote for this issue
            Watchers:
            10 Start watching this issue

              Created:
              Updated:
              Resolved: