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

Column sorting is incorrect in Emma coverage report

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: emma-plugin
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:

      Description

      The Emma coverage report is a table that has Emma's four columns of percentages.
      Each column is sortable.

      When I click on any column, it appears to sort alphabetically. This gives
      incorrect results because the values are actually numbers (i.e. percentages) and
      should really be sorted numerically.

      For example, I sorted by the blocks column and then saw rows containing: 0%,
      10%, 100%, 22%. I would expect the order to be 0%, 10%, 22%, 100%.

        Attachments

          Activity

          Hide
          rickb007 rickb007 added a comment -

          improved title for this issue

          Show
          rickb007 rickb007 added a comment - improved title for this issue
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : manuel_carrasco
          Path:
          trunk/hudson/plugins/emma/src/main/java/hudson/plugins/emma/CoverageObject.java
          trunk/hudson/plugins/emma/src/test/java/hudson/plugins/emma/CoverageObjectTest.java
          http://jenkins-ci.org/commit/28452
          Log:
          table sorting was not working, fixes JENKINS-4173

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : manuel_carrasco Path: trunk/hudson/plugins/emma/src/main/java/hudson/plugins/emma/CoverageObject.java trunk/hudson/plugins/emma/src/test/java/hudson/plugins/emma/CoverageObjectTest.java http://jenkins-ci.org/commit/28452 Log: table sorting was not working, fixes JENKINS-4173
          Hide
          manuel_carrasco manuel_carrasco added a comment -

          fixed in trunk, r28452

          Show
          manuel_carrasco manuel_carrasco added a comment - fixed in trunk, r28452

            People

            • Assignee:
              kohsuke Kohsuke Kawaguchi
              Reporter:
              rickb007 rickb007
            • Votes:
              1 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: