diff mbox

[Branch,~linaro-validation/lava-dashboard/trunk] Rev 409: Fix broken target goal validation on image reports page. Reviewed by stylesen.

Message ID 20130619083316.6638.26034.launchpad@ackee.canonical.com
State Accepted
Headers show

Commit Message

Stevan Radakovic June 19, 2013, 8:33 a.m. UTC 
Merge authors:
  Stevan Radaković (stevanr)
Related merge proposals:
  https://code.launchpad.net/~stevanr/lava-dashboard/image-reports-fix3/+merge/170263
  proposed by: Stevan Radaković (stevanr)
  review: Approve - Senthil Kumaran S (stylesen)
------------------------------------------------------------
revno: 409 [merge]
committer: Stevan Radakovic <stevan.radakovic@linaro.org>
branch nick: trunk
timestamp: Wed 2013-06-19 10:32:36 +0200
message:
  Fix broken target goal validation on image reports page. Reviewed by stylesen.
modified:
  dashboard_app/static/dashboard_app/js/image-report.js


--
lp:lava-dashboard
https://code.launchpad.net/~linaro-validation/lava-dashboard/trunk

You are subscribed to branch lp:lava-dashboard.
To unsubscribe from this branch go to https://code.launchpad.net/~linaro-validation/lava-dashboard/trunk/+edit-subscription
diff mbox

Patch

=== modified file 'dashboard_app/static/dashboard_app/js/image-report.js'
--- dashboard_app/static/dashboard_app/js/image-report.js	2013-06-18 13:50:02 +0000
+++ dashboard_app/static/dashboard_app/js/image-report.js	2013-06-19 08:20:44 +0000
@@ -81,7 +81,7 @@ 
 	return false;
     }
 
-    if (!isNumeric($("#target_goal").val())) {
+    if ($("#target_goal").val() && !isNumeric($("#target_goal").val())) {
 	alert("Target goal must be a numeric value.");
 	return false;
     }