Merge pull request #2375 from CarsonF/bugfix/interface-of-interface-r…

…esolver fix(): Merge in parent interface fields to interfaces
nestjs · Sep 1, 2022 · 691e3f3 · 691e3f3
2 parents 43e5420 + 1c7da39
commit 691e3f3
Showing 1 changed file with 16 additions and 1 deletion.
diff --git a/packages/graphql/lib/schema-builder/factories/interface-definition.factory.ts b/packages/graphql/lib/schema-builder/factories/interface-definition.factory.ts
@@ -113,7 +113,22 @@ export class InterfaceDefinitionFactory {
 
     return () => {
       let fields: GraphQLFieldConfigMap<any, any> = {};
-      metadata.properties.forEach((field) => {
+
+      let properties = [];
+      if (metadata.interfaces) {
+        const implementedInterfaces =
+          TypeMetadataStorage.getInterfacesMetadata()
+            .filter((it) =>
+              getInterfacesArray(metadata.interfaces).includes(it.target),
+            )
+            .map((it) => it.properties);
+        implementedInterfaces.forEach((fields) =>
+          properties.push(...(fields || [])),
+        );
+      }
+      properties = properties.concat(metadata.properties);
+
+      properties.forEach((field) => {
         const type = this.outputTypeFactory.create(
           field.name,
           field.typeFn(),