Merge pull request michaelbromley#282 from axelduch/issue_#281_Pass_oldPageNumber_to_onPageChange_callback

Issue michaelbromley#281 pass old page number to on page change callback. Closes michaelbromley#281

Author: michaelbromley

src/directives/pagination/dirPagination.js

@@ -309,13 +309,19 @@
 
             function goToPage(num) {
                 if (paginationService.isRegistered(paginationId) && isValidPageNumber(num)) {
+                    var oldPageNumber = scope.pagination.current;
+
                     scope.pages = generatePagesArray(num, paginationService.getCollectionLength(paginationId), paginationService.getItemsPerPage(paginationId), paginationRange);
                     scope.pagination.current = num;
                     updateRangeValues();
 
-                    // if a callback has been set, then call it with the page number as an argument
+                    // if a callback has been set, then call it with the page number as the first argument
+                    // and the previous page number as a second argument
                     if (scope.onPageChange) {
-                        scope.onPageChange({ newPageNumber : num });
+                        scope.onPageChange({
+                            newPageNumber : num,
+                            oldPageNumber : oldPageNumber
+                        });
                     }
                 }
             }

src/directives/pagination/dirPagination.spec.js

@@ -644,6 +644,15 @@ describe('dirPagination directive', function() {
                     $scope.$apply();
                     expect($scope.myCallback).toHaveBeenCalledWith(2);
                 });
+
+                it('should pass the previous page number to the callback', function() {
+                    compileWithAttributes(' on-page-change="myCallback(oldPageNumber)" ');
+                    var pagination = containingElement.find('ul.pagination');
+
+                    pagination.children().eq(3).find('a').triggerHandler('click');
+                    $scope.$apply();
+                    expect($scope.myCallback).toHaveBeenCalledWith(1);
+                });
             });
 
             describe('total-items attribute', function() {