Karnan t c has uploaded a new change for review.

Change subject: gluster-nagios-monitoring: Added date validation error message 
on trends tab
......................................................................

gluster-nagios-monitoring: Added date validation error message on trends tab

Displays a error popup when the user selects To date
greater than From Date to show trends.

Change-Id: Id84d02203cd94a780303c5e802dba82e3db500ce
Bug-Url: https://bugzilla.redhat.com/1112183
Signed-off-by: Karnan TC <[email protected]>
---
M gluster-nagios-monitoring/src/js/trends.js
1 file changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/samples-uiplugins 
refs/changes/04/31004/1

diff --git a/gluster-nagios-monitoring/src/js/trends.js 
b/gluster-nagios-monitoring/src/js/trends.js
index c826b89..b128fbd 100644
--- a/gluster-nagios-monitoring/src/js/trends.js
+++ b/gluster-nagios-monitoring/src/js/trends.js
@@ -485,7 +485,10 @@
             var stopDate = $rootScope.stopDate;
             var stopTime = $rootScope.stopTime;
             var stopFullDate = graphUtils.prepareDate(stopDate , stopTime);
-            if ((($scope.graphs)) && ($scope.graphs.length > 0)) {
+            if (stopFullDate.getTime() < startFullDate.getTime()) {
+               alert("The To date-time must be greater than the From 
date-time");
+            }
+            else if ((($scope.graphs)) && ($scope.graphs.length > 0)) {
                 latestGraphs = dataManager.appendUrlsWithTime($scope.graphs, 
startFullDate, stopFullDate);
                 for(var i = 0 ; i < latestGraphs.length ; i++) {
                     $scope.graphs[i] = latestGraphs[i];


-- 
To view, visit http://gerrit.ovirt.org/31004
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id84d02203cd94a780303c5e802dba82e3db500ce
Gerrit-PatchSet: 1
Gerrit-Project: samples-uiplugins
Gerrit-Branch: master
Gerrit-Owner: Karnan t c <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to