account for current page and page size when hiding schedules (fix #183)

MoralCode · MoralCode · commit ecfa2d279374 · 2019-11-26T09:34:11.000-05:00
has not been tested
fix build
diff --git a/assets/src/modules/sm/Schedule/directives/scheduleActionsDirective.js b/assets/src/modules/sm/Schedule/directives/scheduleActionsDirective.js
@@ -142,7 +142,10 @@ angular.module('sm').directive('scheduleActions', function ($http, $q, shareServ
 
       hide: function () {
         ga('send', 'event', 'schedule', 'hide')
-        scope.$parent.$parent.state.schedules.splice(scope.$index, 1)
+        const appstate = scope.$parent.$parent.state;
+        const pageStartIndex = appstate.displayOptions.currentPage * appstate.displayOptions.pageSize;
+
+        appstate.schedules.splice(pageStartIndex + scope.$index, 1)
       }
     }
   };

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,10 @@ angular.module('sm').directive('scheduleActions', function ($http, $q, shareServ`
`142`	`142`
`143`	`143`	`hide: function () {`
`144`	`144`	`ga('send', 'event', 'schedule', 'hide')`
`145`		`- scope.$parent.$parent.state.schedules.splice(scope.$index, 1)`
	`145`	`+ const appstate = scope.$parent.$parent.state;`
	`146`	`+ const pageStartIndex = appstate.displayOptions.currentPage * appstate.displayOptions.pageSize;`
	`147`	`+`
	`148`	`+ appstate.schedules.splice(pageStartIndex + scope.$index, 1)`
`146`	`149`	`}`
`147`	`150`	`}`
`148`	`151`	`};`