Fix bug where an equals sign in a filter argument confuses the key logic

lib/compile-test.js

@@ -4,6 +4,7 @@ import append from './filters/append'
 import upper from './filters/upper'
 import wrap from './filters/wrap'
 import max from './filters/max'
+import map from './filters/map'
 
 import compile from './compile'
 
@@ -228,3 +229,15 @@ test('should compile prefix modifier before filter functions', (t) => {
 
   t.deepEqual(ret, expected)
 })
+
+test('should compile filter function with equal sign in argument', (t) => {
+  const template = 'http://example.com/{type|map(entry=entries)}'
+  const expected = [
+    'http://example.com/',
+    {param: 'type', filters: [{function: map, args: ['entry=entries']}]}
+  ]
+
+  const ret = compile(template)
+
+  t.deepEqual(ret, expected)
+})

lib/compile.js

@@ -38,13 +38,6 @@ const compileParam = (explode) => (segment) => {
     }
     : {param: segment}
 
-  // Specified key
-  const keyParts = obj.param.split('=')
-  if (keyParts.length > 1) {
-    obj.key = keyParts[0]
-    obj.param = keyParts[1]
-  }
-
   // Filter functions
   const filterParts = obj.param.split('|')
   if (filterParts.length > 1) {
@@ -67,6 +60,13 @@ const compileParam = (explode) => (segment) => {
     obj.filters = [{function: filters.max, args: [prefixParts[1]]}].concat(obj.filters || [])
   }
 
+  // Specified key
+  const keyParts = obj.param.split('=')
+  if (keyParts.length > 1) {
+    obj.key = keyParts[0]
+    obj.param = keyParts[1]
+  }
+
   // Explode
   if (explode) {
     obj.explode = true