Deal with collisions generating the headers.

When generating a query header, we can add "object" and "uploadParams" as
arguments, which could conflict with the service definition, change the
variable name when collisions happen.

Fixes #133

Author: thomasvl

Source/GeneratedServices/Storage/GTLRStorageQuery.h

@@ -1653,14 +1653,14 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  *  @param object The @c GTLRStorage_ObjectAccessControl to include in the
  *    query.
  *  @param bucket Name of a bucket.
- *  @param object Name of the object. For information about how to URL encode
- *    object names to be path safe, see Encoding URI Path Parts.
+ *  @param object_param Name of the object. For information about how to URL
+ *    encode object names to be path safe, see Encoding URI Path Parts.
  *
  *  @returns GTLRStorageQuery_ObjectAccessControlsInsert
  */
 + (instancetype)queryWithObject:(GTLRStorage_ObjectAccessControl *)object
                          bucket:(NSString *)bucket
-                         object:(NSString *)object;
+                         object:(NSString *)object_param;
 
 @end
 
@@ -1764,8 +1764,8 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  *  @param object The @c GTLRStorage_ObjectAccessControl to include in the
  *    query.
  *  @param bucket Name of a bucket.
- *  @param object Name of the object. For information about how to URL encode
- *    object names to be path safe, see Encoding URI Path Parts.
+ *  @param object_param Name of the object. For information about how to URL
+ *    encode object names to be path safe, see Encoding URI Path Parts.
  *  @param entity The entity holding the permission. Can be user-userId,
  *    user-emailAddress, group-groupId, group-emailAddress, allUsers, or
  *    allAuthenticatedUsers.
@@ -1774,7 +1774,7 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  */
 + (instancetype)queryWithObject:(GTLRStorage_ObjectAccessControl *)object
                          bucket:(NSString *)bucket
-                         object:(NSString *)object
+                         object:(NSString *)object_param
                          entity:(NSString *)entity;
 
 @end
@@ -1827,8 +1827,8 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  *  @param object The @c GTLRStorage_ObjectAccessControl to include in the
  *    query.
  *  @param bucket Name of a bucket.
- *  @param object Name of the object. For information about how to URL encode
- *    object names to be path safe, see Encoding URI Path Parts.
+ *  @param object_param Name of the object. For information about how to URL
+ *    encode object names to be path safe, see Encoding URI Path Parts.
  *  @param entity The entity holding the permission. Can be user-userId,
  *    user-emailAddress, group-groupId, group-emailAddress, allUsers, or
  *    allAuthenticatedUsers.
@@ -1837,7 +1837,7 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  */
 + (instancetype)queryWithObject:(GTLRStorage_ObjectAccessControl *)object
                          bucket:(NSString *)bucket
-                         object:(NSString *)object
+                         object:(NSString *)object_param
                          entity:(NSString *)entity;
 
 @end
@@ -2682,14 +2682,14 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  *
  *  @param object The @c GTLRStorage_Object to include in the query.
  *  @param bucket Name of the bucket in which the object resides.
- *  @param object Name of the object. For information about how to URL encode
- *    object names to be path safe, see Encoding URI Path Parts.
+ *  @param object_param Name of the object. For information about how to URL
+ *    encode object names to be path safe, see Encoding URI Path Parts.
  *
  *  @returns GTLRStorageQuery_ObjectsPatch
  */
 + (instancetype)queryWithObject:(GTLRStorage_Object *)object
                          bucket:(NSString *)bucket
-                         object:(NSString *)object;
+                         object:(NSString *)object_param;
 
 @end
 
@@ -2914,14 +2914,14 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  *
  *  @param object The @c GTLRStorage_Policy to include in the query.
  *  @param bucket Name of the bucket in which the object resides.
- *  @param object Name of the object. For information about how to URL encode
- *    object names to be path safe, see Encoding URI Path Parts.
+ *  @param object_param Name of the object. For information about how to URL
+ *    encode object names to be path safe, see Encoding URI Path Parts.
  *
  *  @returns GTLRStorageQuery_ObjectsSetIamPolicy
  */
 + (instancetype)queryWithObject:(GTLRStorage_Policy *)object
                          bucket:(NSString *)bucket
-                         object:(NSString *)object;
+                         object:(NSString *)object_param;
 
 @end
 
@@ -3083,14 +3083,14 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  *
  *  @param object The @c GTLRStorage_Object to include in the query.
  *  @param bucket Name of the bucket in which the object resides.
- *  @param object Name of the object. For information about how to URL encode
- *    object names to be path safe, see Encoding URI Path Parts.
+ *  @param object_param Name of the object. For information about how to URL
+ *    encode object names to be path safe, see Encoding URI Path Parts.
  *
  *  @returns GTLRStorageQuery_ObjectsUpdate
  */
 + (instancetype)queryWithObject:(GTLRStorage_Object *)object
                          bucket:(NSString *)bucket
-                         object:(NSString *)object;
+                         object:(NSString *)object_param;
 
 /**
  *  Fetches the requested resource data as a @c GTLRDataObject.
@@ -3099,14 +3099,14 @@ GTLR_EXTERN NSString * const kGTLRStorageProjectionNoAcl;
  *
  *  @param object The @c GTLRStorage_Object to include in the query.
  *  @param bucket Name of the bucket in which the object resides.
- *  @param object Name of the object. For information about how to URL encode
- *    object names to be path safe, see Encoding URI Path Parts.
+ *  @param object_param Name of the object. For information about how to URL
+ *    encode object names to be path safe, see Encoding URI Path Parts.
  *
  *  @returns GTLRStorageQuery_ObjectsUpdate
  */
 + (instancetype)queryForMediaWithObject:(GTLRStorage_Object *)object
                                  bucket:(NSString *)bucket
-                                 object:(NSString *)object;
+                                 object:(NSString *)object_param;
 
 @end
 

Source/Tools/ServiceGenerator/SGGenerator.m

@@ -1432,7 +1432,7 @@ - (NSString *)generateQueryMethodParametersBlockForMode:(GeneratorMode)mode {
 
 // Append "_param" to any name used as a local variable when generating the
 // query method implementation to avoid duplication variables.
-static NSString *MappedParamName(NSString *name) {
+static NSString *MappedParamImplName(NSString *name) {
   NSString *result = name;
 
   if ([name isEqual:@"object"] ||
@@ -1446,6 +1446,19 @@ - (NSString *)generateQueryMethodParametersBlockForMode:(GeneratorMode)mode {
   return result;
 }
 
+// Append "_param" to any name used as a parameter when generating the
+// query method interface to avoid duplication variables.
+static NSString *MappedParamInterfaceName(NSString *name, BOOL takesObject, BOOL takesUploadParams) {
+  NSString *result = name;
+
+  if ((takesObject && [name isEqual:@"object"]) ||
+      (takesUploadParams && [name isEqual:@"uploadParameters"])) {
+    result = [name stringByAppendingString:@"_param"];
+  }
+
+  return result;
+}
+
 - (NSString *)generateQueryPropertyBlock:(NSArray *)paramSchema
                                     mode:(GeneratorMode)mode {
   NSMutableArray *parts = [NSMutableArray array];
@@ -1935,10 +1948,14 @@ - (NSString *)generateQueryClassesForMode:(GeneratorMode)mode {
         NSString *capitalizeObjCName = param.sg_capObjCName;
         [methodStr appendFormat:@"With%@:(%@%@)%@",
          capitalizeObjCName, objcType, (asPtr ? @" *" : @""),
-         (mode == kGenerateInterface ? name : MappedParamName(name))];
+         (mode == kGenerateInterface
+          ? MappedParamInterfaceName(name, doesQueryTakeObject, supportsMediaUpload)
+          : MappedParamImplName(name))];
         [downloadMethodStr appendFormat:@"With%@:(%@%@)%@",
          capitalizeObjCName, objcType, (asPtr ? @" *" : @""),
-         (mode == kGenerateInterface ? name : MappedParamName(name))];
+         (mode == kGenerateInterface
+          ? MappedParamInterfaceName(name, doesQueryTakeObject, supportsMediaUpload)
+          : MappedParamImplName(name))];
         nameWidth += 4 + capitalizeObjCName.length; // 'With%@'
         downloadNameWidth += 4 + capitalizeObjCName.length;
         needsWith = NO;
@@ -1949,11 +1966,15 @@ - (NSString *)generateQueryClassesForMode:(GeneratorMode)mode {
         [methodStr appendFormat:@"\n%*s:(%@%@)%@",
          (int)nameWidth, name.UTF8String, objcType,
          (asPtr ? @" *" : @""),
-         (mode == kGenerateInterface ? name : MappedParamName(name))];
+         (mode == kGenerateInterface
+          ? MappedParamInterfaceName(name, doesQueryTakeObject, supportsMediaUpload)
+          : MappedParamImplName(name))];
         [downloadMethodStr appendFormat:@"\n%*s:(%@%@)%@",
          (int)downloadNameWidth, name.UTF8String, objcType,
          (asPtr ? @" *" : @""),
-         (mode == kGenerateInterface ? name : MappedParamName(name))];
+         (mode == kGenerateInterface
+          ? MappedParamInterfaceName(name, doesQueryTakeObject, supportsMediaUpload)
+          : MappedParamImplName(name))];
       }
 
       NSString *paramDesc = param.descriptionProperty;
@@ -1970,7 +1991,8 @@ - (NSString *)generateQueryClassesForMode:(GeneratorMode)mode {
       if (paramDesc.length == 0) {
         paramDesc = objcType;
       }
-      [methodHDoc appendParam:name string:paramDesc];
+      [methodHDoc appendParam:MappedParamInterfaceName(name, doesQueryTakeObject, supportsMediaUpload)
+                       string:paramDesc];
 
     }  // for (param in method.sg_sortedParameters)
 
@@ -2188,7 +2210,7 @@ - (NSString *)generateQueryClassesForMode:(GeneratorMode)mode {
         for (GTLRDiscovery_JsonSchema *param in method.sg_sortedParameters) {
           if (param.required.boolValue) {
             NSString *name = param.sg_objcName;
-            NSString *nameAsValue = MappedParamName(name);
+            NSString *nameAsValue = MappedParamImplName(name);
             [methodStr appendFormat:@"  query.%@ = %@;\n", name, nameAsValue];
           }
         }
@@ -2242,13 +2264,13 @@ - (NSString *)generateQueryClassesForMode:(GeneratorMode)mode {
           if (needsWith) {
             NSString *capitalizeObjCName = param.sg_capObjCName;
             [downloadMethodStr appendFormat:@"With%@:%@",
-             capitalizeObjCName, MappedParamName(param.sg_objcName)];
+             capitalizeObjCName, MappedParamImplName(param.sg_objcName)];
             nameWidth += 4 + capitalizeObjCName.length; // 'With%@'
             needsWith = NO;
           } else {
             NSString *name = param.sg_objcName;
             [downloadMethodStr appendFormat:@"\n%*s:%@",
-             (int)nameWidth, name.UTF8String, MappedParamName(name)];
+             (int)nameWidth, name.UTF8String, MappedParamImplName(name)];
           }
         }
       }