fixed #68

joefutrelle · Sep 18, 2015 · 4ab98c2 · 4ab98c2
1 parent 9875f43
commit 4ab98c2
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/rbac/admin/timeseries/TimeSeries.js b/rbac/admin/timeseries/TimeSeries.js
@@ -1,6 +1,6 @@
 
 
-ifcbAdmin.controller('TimeSeriesCtrl', ['$scope', 'TimeSeriesService', function ($scope, TimeSeriesService) {
+ifcbAdmin.controller('TimeSeriesCtrl', ['$scope', 'Restangular', 'TimeSeriesService', function ($scope, Restangular, TimeSeriesService) {
 
     // initialize local scope
     $scope.alert = null;
@@ -50,14 +50,16 @@ ifcbAdmin.controller('TimeSeriesCtrl', ['$scope', 'TimeSeriesService', function
     $scope.saveTimeSeries = function(ts) {
 	console.log("saving time series "+ts.label);
         // remove blank paths before save
-        for (var i = 0; i < ts.data_dirs.length; i++) {
-            if (ts.data_dirs[i].path.trim() == "") {
-                $scope.removePath(ts, ts.data_dirs[i]);
-            }
-        }
+        $.each(ts.data_dirs, function(ix, dd) {
+			if(dd.path.trim()=="") {
+				$scope.removePath(ts, dd);
+			} else {
+				console.log(ts.label+" has "+dd.product_type+" path "+dd.path);
+			}
+		});
         if(ts.id) {
         // timeseries group already exists on server. update.
-        ts.patch().then(function(serverResponse) {
+        Restangular.copy(ts).patch().then(function(serverResponse) {
                 delete ts.edit;
                 delete restore[ts.id];
                 $scope.alert = null;