try fix

parse-community · Jun 25, 2020 · 9d38212 · 9d38212
1 parent b2db04c
commit 9d38212
Show file tree

Hide file tree

Showing 4 changed files with 65 additions and 43 deletions.
diff --git a/lib/GraphQL/ParseGraphQLSchema.js b/lib/GraphQL/ParseGraphQLSchema.js
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "parse-server",
-  "version": "4.8.2",
+  "version": "4.8.3",
   "description": "An express module providing a Parse-compatible API server",
   "main": "lib/index.js",
   "repository": {

diff --git a/src/GraphQL/ParseGraphQLSchema.js b/src/GraphQL/ParseGraphQLSchema.js
@@ -200,6 +200,41 @@ class ParseGraphQLSchema {
 
       if (typeof this.graphQLCustomTypeDefs.getTypeMap === 'function') {
         const customGraphQLSchemaTypeMap = this.graphQLCustomTypeDefs.getTypeMap();
+        const findAndReplaceLastType = (parent, key) => {
+          if (parent[key].name) {
+            if (
+              this.graphQLAutoSchema.getType(parent[key].name) &&
+              this.graphQLAutoSchema.getType(parent[key].name) !== parent[key]
+            ) {
+              // To avoid unresolved field on overloaded schema
+              // replace the final type with the auto schema one
+              parent[key] = this.graphQLAutoSchema.getType(parent[key].name);
+            }
+          } else {
+            if (parent[key].ofType) {
+              findAndReplaceLastType(parent[key], 'ofType');
+            }
+          }
+        };
+        Object.values(customGraphQLSchemaTypeMap).forEach(
+          (customGraphQLSchemaType) => {
+            if (
+              !customGraphQLSchemaType ||
+              !customGraphQLSchemaType.name ||
+              customGraphQLSchemaType.name.startsWith('__')
+            ) {
+              return;
+            }
+            const autoGraphQLSchemaType = this.graphQLAutoSchema.getType(
+              customGraphQLSchemaType.name
+            );
+            if (!autoGraphQLSchemaType) {
+              this.graphQLAutoSchema._typeMap[
+                customGraphQLSchemaType.name
+              ] = customGraphQLSchemaType;
+            }
+          }
+        );
         Object.values(customGraphQLSchemaTypeMap).forEach(
           (customGraphQLSchemaType) => {
             if (
@@ -212,30 +247,11 @@ class ParseGraphQLSchema {
             const autoGraphQLSchemaType = this.graphQLAutoSchema.getType(
               customGraphQLSchemaType.name
             );
+
             if (
               autoGraphQLSchemaType &&
               typeof customGraphQLSchemaType.getFields === 'function'
             ) {
-              const findAndReplaceLastType = (parent, key) => {
-                if (parent[key].name) {
-                  if (
-                    this.graphQLAutoSchema.getType(parent[key].name) &&
-                    this.graphQLAutoSchema.getType(parent[key].name) !==
-                      parent[key]
-                  ) {
-                    // To avoid unresolved field on overloaded schema
-                    // replace the final type with the auto schema one
-                    parent[key] = this.graphQLAutoSchema.getType(
-                      parent[key].name
-                    );
-                  }
-                } else {
-                  if (parent[key].ofType) {
-                    findAndReplaceLastType(parent[key], 'ofType');
-                  }
-                }
-              };
-
               Object.values(customGraphQLSchemaType.getFields()).forEach(
                 (field) => {
                   findAndReplaceLastType(field, 'type');
@@ -245,10 +261,6 @@ class ParseGraphQLSchema {
                 ...autoGraphQLSchemaType.getFields(),
                 ...customGraphQLSchemaType.getFields(),
               };
-            } else {
-              this.graphQLAutoSchema._typeMap[
-                customGraphQLSchemaType.name
-              ] = customGraphQLSchemaType;
             }
           }
         );