Merge lp:~stevanr/lava-dashboard/fix-1197338 into lp:lava-dashboard

[Stevan Radaković]

Introducing more accurate build number range check.

=== modified file 'dashboard_app/static/dashboard_app/js/image-report.js'
--- dashboard_app/static/dashboard_app/js/image-report.js	2013-07-03 11:42:45 +0000
+++ dashboard_app/static/dashboard_app/js/image-report.js	2013-07-03 14:51:25 +0000
@@ -33,12 +33,13 @@
 function update_filters(column_data, test_run_names) {
     for (iter in column_data) {
 	build_number = column_data[iter]["number"].split('.')[0];
+	build_date = column_data[iter]["date"];
 	$("#build_number_start").append($('<option>', {
-	    value: build_number,
+	    value: build_date,
 	    text: build_number
 	}));
 	$("#build_number_end").append($('<option>', {
-	    value: build_number,
+	    value: build_date,
 	    text: build_number
 	}));
     }
@@ -76,7 +77,7 @@
 	build_number_end = parseInt(build_number_end);
     }
 
-    if (build_number_start >= build_number_end) {
+    if (build_number_start > build_number_end) {
 	alert("End build number must be greater then the start build number.");
 	return false;
     }
@@ -380,19 +381,7 @@
 function test_build_number(column_data, iter) {
     // Test if the build number/date is between specified number/date boundaries.
 
-    build_number = column_data[iter]["number"].split('.')[0];
-    if (isNumeric(build_number)) {
-	build_number = parseInt(build_number);
-    }
-
-    build_number_start = $("#build_number_start").val();
-    if (isNumeric(build_number_start)) {
-	build_number_start = parseInt(build_number_start);
-    }
-    build_number_end = $("#build_number_end").val();
-    if (isNumeric(build_number_end)) {
-	build_number_end = parseInt(build_number_end);
-    }
+    build_number = column_data[iter]["date"];
 
     if (build_number <= $("#build_number_end").val() && build_number >= $("#build_number_start").val()) {
 	return true;