Allow use of locationName on non-flattened lists

aws · Feb 15, 2017 · db1e264 · db1e264
1 parent e4d72fc
commit db1e264
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 17 deletions.
diff --git a/lib/query/query_param_serializer.js b/lib/query/query_param_serializer.js
@@ -59,7 +59,7 @@ function serializeList(name, list, rules, fn) {
         name = parts.join('.');
       }
     } else {
-      suffix = '.member' + suffix;
+      suffix = '.' + (memberRules.name ? memberRules.name : 'member') + suffix;
     }
     serializeMember(name + suffix, v, memberRules, fn);
   });

diff --git a/test/query/query_param_serializer.spec.coffee b/test/query/query_param_serializer.spec.coffee
@@ -190,19 +190,6 @@ describe 'QueryParamSerializer', ->
           ['Person.member.3', 'c'],
         ])
 
-      it 'ignores the list-member name', ->
-        rules =
-          Person:
-            type: 'list'
-            member:
-              locationName: 'Name'
-        params = serialize({Person:['a','b','c']}, rules)
-        expect(params).to.eql([
-          ['Person.member.1', 'a'],
-          ['Person.member.2', 'b'],
-          ['Person.member.3', 'c'],
-        ])
-
       it 'observes both list name and list member name', ->
         rules =
           People:
@@ -212,9 +199,9 @@ describe 'QueryParamSerializer', ->
               locationName: 'Name'
         params = serialize({People:['a','b','c']}, rules)
         expect(params).to.eql([
-          ['Person.member.1', 'a'],
-          ['Person.member.2', 'b'],
-          ['Person.member.3', 'c'],
+          ['Person.Name.1', 'a'],
+          ['Person.Name.2', 'b'],
+          ['Person.Name.3', 'c'],
         ])
 
   describe 'maps', -> # maps are hashes with user defined keys