@looker/sdk-codegen 21.6.0 → 21.7.0

package/CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [21.7.0](https://www.github.com/looker-open-source/sdk-codegen/compare/sdk-codegen-v21.6.0...sdk-codegen-v21.7.0) (2022-07-29)
+
+
+### Features
+
+* keep SDK request interfaces distinct ([#1125](https://www.github.com/looker-open-source/sdk-codegen/issues/1125)) ([50e925c](https://www.github.com/looker-open-source/sdk-codegen/commit/50e925c90eb3c5ffefc8e8535131471e2f2d0334))
+
 ## [21.6.0](https://www.github.com/looker-open-source/sdk-codegen/compare/sdk-codegen-v21.5.2...sdk-codegen-v21.6.0) (2022-07-07)
 
 
@@ -344,4 +351,4 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
-- [API spec diff utility](https://github.com/looker-open-source/sdk-codegen/pull/380)
+- [API spec diff utility](https://github.com/looker-open-source/sdk-codegen/pull/380)

package/lib/esm/sdkModels.js

@@ -1275,8 +1275,6 @@ export class ApiModel {
   constructor(spec) {
     this.spec = spec;
 
-    _defineProperty(this, "requestTypes", {});
-
     _defineProperty(this, "enumTypes", {});
 
     _defineProperty(this, "refs", {});
@@ -1507,11 +1505,14 @@ export class ApiModel {
     throw new Error("Schema ".concat(typeof schema === 'string' ? schema : JSON.stringify(schema), " must have a ref or a type"));
   }
 
-  makeRequestType(hash, method) {
-    var name = "".concat(strRequest).concat(camelCase('_' + method.name));
+  requestTypeName(method) {
+    return "".concat(strRequest).concat(camelCase('_' + method.name));
+  }
+
+  makeRequestType(method) {
+    var name = this.requestTypeName(method);
     var request = new RequestType(this, name, method.allParams, "Dynamically generated request type for ".concat(method.name));
     this.types[name] = request;
-    this.requestTypes[hash] = request;
     method.addType(this, request);
     return request;
   }
@@ -1544,11 +1545,8 @@ export class ApiModel {
 
   _getRequestType(method) {
     if (method.optionalParams.length <= 1) return undefined;
-    var paramHash = '';
-    method.allParams.forEach(p => paramHash += p.asHashString());
-    var hash = md5(paramHash);
-    var result = this.requestTypes[hash];
-    if (!result) result = this.makeRequestType(hash, method);
+    var result = this.types[this.requestTypeName(method)];
+    if (!result) result = this.makeRequestType(method);
     return result;
   }
 
@@ -1563,10 +1561,9 @@ export class ApiModel {
     return this.enumTypes;
   }
 
-  makeWriteableType(hash, type) {
+  makeWriteableType(type) {
     var writer = new WriteType(this, type);
     this.types[writer.name] = writer;
-    this.requestTypes[hash] = writer;
     return writer;
   }
 
@@ -1596,10 +1593,7 @@ export class ApiModel {
     }
 
     if (writes.length === props.length && JSON.stringify(writes) === JSON.stringify(props)) return undefined;
-    var hash = md5(type.asHashString());
-    var result = this.requestTypes[hash];
-    if (result) return result;
-    result = this.makeWriteableType(hash, type);
+    var result = this.makeWriteableType(type);
     type.types.add(result.name);
     type.customTypes.add(result.name);
     result.parentTypes.add(type.name);