Francois Gouget : testbot/web: Fix the sort order of a job's tasks.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: b339f509deb5fe6c859da9b27b310a2f202333b6
URL:    http://source.winehq.org/git/tools.git/?a=commit;h=b339f509deb5fe6c859da9b27b310a2f202333b6

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Jan 17 16:04:24 2017 +0100

testbot/web: Fix the sort order of a job's tasks.

Each entry's id is a numeric value that's equal to step.no*100+task.no.
But by default sort performs a string comparison resulting in task
(10,1), aka "1001", coming before (1,1), aka "101", which comes before
(11,1), aka "1101", etc.

Also note that sorting is normally the job of the CollectionBlock but
the JobDetails page is bypassing it.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/web/JobDetails.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testbot/web/JobDetails.pl b/testbot/web/JobDetails.pl
index b43d375..cf3b4a8 100644
--- a/testbot/web/JobDetails.pl
+++ b/testbot/web/JobDetails.pl
@@ -213,7 +213,7 @@ sub SortKeys($$$)
 {
   my ($self, $CollectionBlock, $Keys) = @_;
 
-  my @SortedKeys = sort @$Keys;
+  my @SortedKeys = sort { $a <=> $b } @$Keys;
   return \@SortedKeys;
 }