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

Summary is missing for the first build

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • Minor
    • Resolution: Fixed
    • sloccount-plugin
    • None
    • All platforms, bug in code logic.

    Description

      If SLOCCount plugin is newly enabled for a project and a first build is processed, the summary of the first build will contain only SLOCCount icon but no text on its right.

      Attachments

        Activity

          Code changed in jenkins
          User: Michal Turek
          Path:
          src/main/java/hudson/plugins/sloccount/ReportSummary.java
          src/main/resources/hudson/plugins/sloccount/Messages.properties
          http://jenkins-ci.org/commit/sloccount-plugin/5493d931ac74f6330a13f9533671f46d6efd848c
          Log:
          JENKINS-20646 Summary is missing for the first build

          • The build summary was displayed only if both current and previous reports were not null. Only current report should be tested at top-level, test of previous report is relevant only while processing the differences.
          • Hardcoded "in" string on summary page added to localication Messages (English only).
          • Create a new project.
          • Process a first build.
          • Show its summary.
          • Verify the summary contains valid text.
          • Process a second build.
          • Show its summary.
          • Verify the summary contains valid text.
          • All tests passed.
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Michal Turek Path: src/main/java/hudson/plugins/sloccount/ReportSummary.java src/main/resources/hudson/plugins/sloccount/Messages.properties http://jenkins-ci.org/commit/sloccount-plugin/5493d931ac74f6330a13f9533671f46d6efd848c Log: JENKINS-20646 Summary is missing for the first build The build summary was displayed only if both current and previous reports were not null. Only current report should be tested at top-level, test of previous report is relevant only while processing the differences. Hardcoded "in" string on summary page added to localication Messages (English only). Create a new project. Process a first build. Show its summary. Verify the summary contains valid text. Process a second build. Show its summary. Verify the summary contains valid text. All tests passed.
          mixalturek Michal Turek added a comment -

          Please merge-in the patch.

          mixalturek Michal Turek added a comment - Please merge-in the patch.
          mixalturek Michal Turek added a comment - Fixed in https://github.com/mixalturek/sloccount-plugin/commit/5493d931ac74f6330a13f9533671f46d6efd848c . Pull request sent on GitHub.

          People

            mixalturek Michal Turek
            mixalturek Michal Turek
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: