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

[Stevan Radaković]

Fix bug #1198190.

[lp:~stevanr/lava-dashboard/fix-1198190]

414. By Stevan Radaković on 2013-07-25

Scratch the global variable problem.

[Senthil Kumaran S (stylesen)]

Looks good.

review: Approve

=== modified file 'dashboard_app/static/dashboard_app/js/image-report.js'
--- dashboard_app/static/dashboard_app/js/image-report.js	2013-07-03 14:48:22 +0000
+++ dashboard_app/static/dashboard_app/js/image-report.js	2013-07-25 13:59:25 +0000
@@ -104,9 +104,9 @@
     // Create column headlines.
     result_table_head = "<tr>";
     for (iter in column_data) {
-	build_number = column_data[iter]["number"].split('.')[0];
+	if (test_build_number(column_data, iter)) {
 
-	if (test_build_number(column_data, iter)) {
+	    build_number = column_data[iter]["number"].split('.')[0];
 	    if (!isNumeric(build_number)) {
 		build_number = format_date(build_number.split(' ')[0]);
 	    }
@@ -381,7 +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]["date"];
+    var build_number = column_data[iter]["date"];
 
     if (build_number <= $("#build_number_end").val() && build_number >= $("#build_number_start").val()) {
 	return true;