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

Redesign of build summary page

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • cppcheck-plugin
    • None
    • All, plugin version 1.14

    Description

      Redesign Cppcheck summary on the build page to contain a sortable table with the same information that will be better readable.

      The current implementation displays "(nb/new)" legend in the middle of each line, zero new value is displayed as "+0" and resolved issues are not present at all (negative values not implemented). There is also NullPointerException in CppcheckSummary.createReportSummaryDetails() if the first build is displayed (attached).

      Attachments

        1. summary_before.png
          summary_before.png
          24 kB
        2. exception.txt
          10 kB
        3. summary_after.png
          summary_after.png
          25 kB

        Activity

          mixalturek Michal Turek added a comment - Implemented in https://github.com/mixalturek/cppcheck-plugin/commit/8edec09b5c49d22fdb8519e66827fe2abce81b64 .

          Code changed in jenkins
          User: Michal Turek
          Path:
          src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckResult.java
          src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckStatistics.java
          src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckSummary.java
          src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckBuildAction/summary.jelly
          http://jenkins-ci.org/commit/cppcheck-plugin/8edec09b5c49d22fdb8519e66827fe2abce81b64
          Log:
          JENKINS-21927 Redesign of build summary page

          • List in build summary updated to a sortable table that is better readable and more usable. The displayed values were fixed, e.g. negative result in delta can be displayed.
          • The design is fully defined in jelly instead of in Java code.
          • CppcheckSummary class is not needed, removed.
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Michal Turek Path: src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckResult.java src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckStatistics.java src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckSummary.java src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckBuildAction/summary.jelly http://jenkins-ci.org/commit/cppcheck-plugin/8edec09b5c49d22fdb8519e66827fe2abce81b64 Log: JENKINS-21927 Redesign of build summary page List in build summary updated to a sortable table that is better readable and more usable. The displayed values were fixed, e.g. negative result in delta can be displayed. The design is fully defined in jelly instead of in Java code. CppcheckSummary class is not needed, removed.
          mixalturek Michal Turek added a comment -

          Implemented, will be released in version 1.15.

          mixalturek Michal Turek added a comment - Implemented, will be released in version 1.15.

          People

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

            Dates

              Created:
              Updated:
              Resolved: