Selecting line numbers without text now works, line numbers are sorted correctly

- Fetch line number from parent instead of text
- Extended Array.prototype with min() and max() methods, use these to extract first and last line numbers

1 files changed, 3 insertions, 3 deletions

diff --git a/public/javascripts/diff_browser.js b/public/javascripts/diff_browser.js
index 3b78ad1..9e38e2d 100644
--- a/public/javascripts/diff_browser.js
+++ b/public/javascripts/diff_browser.js
@@ -548,7 +548,8 @@ Gitorious.enableCommenting = function() {
                 $(this).parent().addClass("selected-for-commenting");
             })
                 var allLineNumbers = $(diffTable).find("td.ui-selected").map(function(){
-                    return $(this).text();
+                    var lineNo = $(this).parent().attr("class").replace(/[\sa-z\-]*/g,"");
+                    return lineNo;
                 });
             var path = $(diffTable).parent().prev(".header").children(".title").text();
             var commentForm = new Gitorious.CommentForm(path);
@@ -605,8 +606,7 @@ Gitorious.CommentForm = function(path){
         this.numbers = result;
     }
     this.linesAsString = function() {
-        var sortedLines = this.numbers.sort();
-        return sortedLines[0] + ".." + sortedLines[sortedLines.length - 1];
+        return this.numbers.min() + ".." + this.numbers.max();
     }
     this.hasLines = function() {
         return this.numbers.length > 0;