fixes default sorting direction and change instructions a little

eb620ffc · meus · 30e41d0e · eb620ffc · eb620ffc
Commit eb620ffc authored Oct 14, 2007 by meus
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

Coverage.php tests/DoctrineTest/Coverage.php +2 -2

index.php tests/coverage/index.php +1 -1

No files found.
--- a/tests/DoctrineTest/Coverage.php
+++ b/tests/DoctrineTest/Coverage.php
@@ -84,7 +84,7 @@ class DoctrineTest_Coverage
        uasort($coveredArray, array($this,"sortArray"));

        //and flip if it perhaps?
-        if (isset($_GET["desc"]) && $_GET["desc"] == "true"){
+        if (isset($_GET["flip"]) && $_GET["flip"] == "true"){
            $coveredArray = array_reverse($coveredArray, true);
        }

@@ -332,6 +332,6 @@ class DoctrineTest_Coverage
        if ($a[$this->sortBy] == $b[$this->sortBy]) {
            return 0;
        }
-        return ( $a[$this->sortBy] < $b[$this->sortBy]) ? -1 : 1;
+        return ( $a[$this->sortBy] < $b[$this->sortBy]) ? 1 : -1;
    }
 }
--- a/tests/coverage/index.php
+++ b/tests/coverage/index.php
@@ -39,7 +39,7 @@ $revision = $svn_info[1];
 <body>
    <h1>Coverage report for Doctrine</h1>
    <p>Report generated against revision <?php echo $reporter->getRevision(); ?> current HEAD revision is <?php echo $revision ?>.</p>
-    <p>Default mode shows results sorted by percentage. This can be changed with GET variables:<br /> <ul><li>order = covered|total|maybe|notcovered|percentage</li><li>desc=true</li></ul></p>
+    <p>Default mode shows results sorted by percentage with highest first. Customize the ordering with the following GET parameters:<br /> <ul><li>order = covered|total|maybe|notcovered|percentage</li><li>flip=true</li></ul></p>
    <table>
        <tr>
            <th></th>