fix(parse): changed the functionCall method to parse filters

Added a condition to allow the parser to understand whether the parameter is a filter

Closes angular#4168

Author: esarbanis

src/ng/parse.js

@@ -684,7 +684,7 @@ function parser(text, json, $filter, csp){
     var argsFn = [];
     if (peekToken().text != ')') {
       do {
-        argsFn.push(expression());
+        argsFn.push(filterChain());
       } while (expect(','));
     }
     consume(')');

test/ng/parseSpec.js

@@ -311,6 +311,43 @@ describe('parser', function() {
         expect(scope.$eval("'abcd'|substring:1:3|uppercase")).toEqual("BC");
       });
 
+      it('should parse filters as method parameters', function() {
+        $filterProvider.register('substring', valueFn(function(input, start, end) {
+          return input.substring(start, end);
+        }));
+
+        scope.offset =  3;
+
+        scope.echo1 = function(substring) {
+          return substring;
+        };
+
+        expect(scope.$eval("echo1(('abcd'|substring:1:offset))")).toEqual("bc");
+        expect(scope.$eval("echo1('abcd'|substring:1:offset)")).toEqual("bc");
+        
+        scope.echo2 = function(lowercaseSubstring,uppercaseSubstring) {
+          return lowercaseSubstring+uppercaseSubstring;
+        };
+
+        expect(scope.$eval("echo2(('abcd'|substring:1:offset),('abcd'|substring:1:3|uppercase))")).toEqual("bcBC");
+        expect(scope.$eval("echo2('abcd'|substring:1:offset,'abcd'|substring:1:3|uppercase)")).toEqual("bcBC");
+
+        expect(scope.$eval("echo2('',('abcd'|substring:1:3|uppercase))")).toEqual("BC");
+        expect(scope.$eval("echo2('','abcd'|substring:1:3|uppercase)")).toEqual("BC");
+
+        expect(scope.$eval("echo2(null,('abcd'|substring:1:3|uppercase))")).toEqual("nullBC");
+        expect(scope.$eval("echo2(null,'abcd'|substring:1:3|uppercase)")).toEqual("nullBC");
+
+        expect(scope.$eval("echo2(undefined,('abcd'|substring:1:3|uppercase))")).toEqual("undefinedBC");
+        expect(scope.$eval("echo2(undefined,'abcd'|substring:1:3|uppercase)")).toEqual("undefinedBC");
+
+        expect(scope.$eval("echo2(('abcd'|substring:1:offset),null)")).toEqual("bcnull");
+        expect(scope.$eval("echo2('abcd'|substring:1:offset,null)")).toEqual("bcnull");
+
+        expect(scope.$eval("echo2(('abcd'|substring:1:offset),undefined)")).toEqual("bcundefined");
+        expect(scope.$eval("echo2('abcd'|substring:1:offset,undefined)")).toEqual("bcundefined");
+      });
+
       it('should access scope', function() {
         scope.a =  123;
         scope.b = {c: 456};