Fix michaelbromley#241

Author: michaelbromley

src/directives/pagination/dirPagination.js

@@ -56,7 +56,7 @@
             // regex taken directly from https://github.com/angular/angular.js/blob/v1.4.x/src/ng/directive/ngRepeat.js#L339
             var match = expression.match(/^\s*([\s\S]+?)\s+in\s+([\s\S]+?)(?:\s+as\s+([\s\S]+?))?(?:\s+track\s+by\s+([\s\S]+?))?\s*$/);
 
-            var filterPattern = /\|\s*itemsPerPage\s*:\s*(.*\(\s*\w*\)|([^\)]*?(?=as))|[^\)]*)/;
+            var filterPattern = /\|\s*itemsPerPage\s*:\s*(.*\(\s*\w*\)|([^\)]*?(?=\s+as\s+))|[^\)]*)/;
             if (match[2].match(filterPattern) === null) {
                 throw 'pagination directive: the \'itemsPerPage\' filter must be set.';
             }

src/directives/pagination/dirPagination.spec.js

@@ -287,6 +287,18 @@ describe('dirPagination directive', function() {
             expect(getListItems().length).toEqual(10);
         });
 
+        // see https://github.com/michaelbromley/angularUtils/issues/241
+        // the actual issue was caused by the work "sharedTasksFilters", which was
+        // being matched by the part of the regex that looks for "as" alias syntax.
+        it('should allow deeply nested, long-winded object for itemsPerPage', function() {
+            function compile() {
+                $scope.eventsCtrl = { eventsFilters: { sharedTasksFilters: { eventsPerPage: 10 } } };
+                compileElement(myCollection, 5, 1,  "item in collection | itemsPerPage:eventsCtrl.eventsFilters.sharedTasksFilters.eventsPerPage");
+            }
+            expect(compile).not.toThrow();
+            expect(getListItems().length).toEqual(10);
+        });
+
 
         it('should allow track by syntax 1', function() {
             function compile() {