@apollo/client 4.3.0-alpha.1 → 4.3.0-alpha.3

package/core/QueryManager.js

@@ -117,7 +117,7 @@ export class QueryManager {
         const mutationStoreValue = this.mutationStore &&
             (this.mutationStore[queryInfo.id] = {
                 mutation,
-                variables,
+                variables: this.cache.serializeVariables(mutation, variables),
                 loading: true,
                 error: null,
             });
@@ -330,10 +330,6 @@ export class QueryManager {
             }),
         }));
     }
-    requestIdCounter = 1;
-    generateRequestId() {
-        return this.requestIdCounter++;
-    }
     clearStore(options = {
         discardWatches: true,
     }) {
@@ -519,6 +515,7 @@ export class QueryManager {
         const executeContext = {
             client: this.client,
         };
+        variables = this.cache.serializeVariables(query, variables);
         if (serverQuery) {
             const { inFlightLinkObservables, link } = this;
             try {
@@ -620,7 +617,6 @@ export class QueryManager {
         };
     }
     getResultsFromLink(options, { queryInfo, cacheWriteBehavior, observableQuery, exposeExtensions, }) {
-        const requestId = (queryInfo.lastRequestId = this.generateRequestId());
         const { errorPolicy } = options;
         // Performing transformForLink here gives this.cache a chance to fill in
         // missing fragment definitions (for example) before sending this document
@@ -630,10 +626,11 @@ export class QueryManager {
             // Use linkDocument rather than queryInfo.document so the
             // operation/fragments used to write the result are the same as the
             // ones used to obtain it from the link.
-            const result = queryInfo.markQueryResult(incoming, {
+            const { dataState, ...result } = queryInfo.markQueryResult(incoming, {
                 ...options,
                 document: linkDocument,
                 cacheWriteBehavior,
+                returnPartialData: options.returnPartialData,
             });
             const hasErrors = graphQLResultHasError(result);
             if (hasErrors && errorPolicy === "none") {
@@ -641,29 +638,26 @@ export class QueryManager {
                 observableQuery?.["resetNotifications"]();
                 throw new CombinedGraphQLErrors(removeStreamDetailsFromExtensions(result));
             }
+            const partial = dataState !== "complete";
             const aqr = {
                 data: result.data,
                 ...(queryInfo.hasNext ?
                     {
                         loading: true,
                         networkStatus: NetworkStatus.streaming,
-                        dataState: "streaming",
-                        partial: true,
+                        dataState,
+                        partial,
                     }
                     : {
-                        dataState: result.data ? "complete" : "empty",
+                        dataState,
                         loading: false,
                         networkStatus: NetworkStatus.ready,
-                        partial: !result.data,
+                        partial,
                     }),
             };
             if (exposeExtensions && "extensions" in result) {
                 aqr[extensionsSymbol] = result.extensions;
             }
-            // In the case we start multiple network requests simultaneously, we
-            // want to ensure we properly set `data` if we're reporting on an old
-            // result which will not be caught by the conditional above that ends up
-            // throwing the markError result.
             if (hasErrors) {
                 if (errorPolicy === "none") {
                     aqr.data = void 0;
@@ -671,15 +665,14 @@ export class QueryManager {
                 }
                 if (errorPolicy !== "ignore") {
                     aqr.error = new CombinedGraphQLErrors(removeStreamDetailsFromExtensions(result));
-                    if (aqr.dataState !== "streaming") {
+                    if (aqr.networkStatus !== NetworkStatus.streaming) {
                         aqr.networkStatus = NetworkStatus.error;
                     }
                 }
             }
             return aqr;
         }), catchError((error) => {
-            // Avoid storing errors from older interrupted queries.
-            if (requestId >= queryInfo.lastRequestId && errorPolicy === "none") {
+            if (errorPolicy === "none") {
                 queryInfo.resetLastWrite();
                 observableQuery?.["resetNotifications"]();
                 throw error;
@@ -1037,6 +1030,7 @@ export class QueryManager {
             context,
             fetchPolicy,
             errorPolicy,
+            returnPartialData,
         }, {
             cacheWriteBehavior,
             queryInfo,