Fix issue where a query may not stop polling after unmount when in Strict mode with `cache-and-network` fetch policy

[jerelmiller]

Fixes #9431 Fixes #11750

When using useQuery in React's strict mode, 2 ObservableQuery instances are created due to how React's strict mode works with refs and rendering the component twice. The 2 ObservableQuery objects are created because the useInternalState hook runs twice in strict mode and the ref.current property is not set until after the component renders the second time. Because of this, a cache broadcast would happen on teardown of the 2nd observable query (the one actually used by useQuery) and this would cause the 1st observable query to start polling itself because it was never torn down.

This change updates the updatePolling function to only run polling if there are active subscribers to the ObservableQuery.

[changeset-bot[bot]]

🦋 Changeset detected

Latest commit: 75bce0d23520330f5e182e4da455ca5f932568c7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

| Name | Type | | -------------- | ----- | | @apollo/client | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[jerelmiller]

/release:pr

[github-actions[bot]]

size-limit report 📦

Path	Size
dist/apollo-client.min.cjs	38.65 KB (+0.02% 🔺)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/main.cjs"	47.06 KB (+0.02% 🔺)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/main.cjs" (production)	44.69 KB (+0.02% 🔺)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/index.js"	34.18 KB (+0.02% 🔺)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/index.js" (production)	32.06 KB (+0.02% 🔺)
import { ApolloProvider } from "dist/react/index.js"	1.23 KB (0%)
import { ApolloProvider } from "dist/react/index.js" (production)	1.22 KB (0%)
import { useQuery } from "dist/react/index.js"	5.28 KB (0%)
import { useQuery } from "dist/react/index.js" (production)	4.36 KB (0%)
import { useLazyQuery } from "dist/react/index.js"	5.52 KB (0%)
import { useLazyQuery } from "dist/react/index.js" (production)	4.59 KB (0%)
import { useMutation } from "dist/react/index.js"	3.52 KB (0%)
import { useMutation } from "dist/react/index.js" (production)	2.74 KB (0%)
import { useSubscription } from "dist/react/index.js"	3.21 KB (0%)
import { useSubscription } from "dist/react/index.js" (production)	2.4 KB (0%)
import { useSuspenseQuery } from "dist/react/index.js"	5.44 KB (0%)
import { useSuspenseQuery } from "dist/react/index.js" (production)	4.1 KB (0%)
import { useBackgroundQuery } from "dist/react/index.js"	4.96 KB (0%)
import { useBackgroundQuery } from "dist/react/index.js" (production)	3.61 KB (0%)
import { useLoadableQuery } from "dist/react/index.js"	5.07 KB (0%)
import { useLoadableQuery } from "dist/react/index.js" (production)	3.72 KB (0%)
import { useReadQuery } from "dist/react/index.js"	3.33 KB (0%)
import { useReadQuery } from "dist/react/index.js" (production)	3.27 KB (0%)
import { useFragment } from "dist/react/index.js"	2.29 KB (0%)
import { useFragment } from "dist/react/index.js" (production)	2.23 KB (0%)

[github-actions[bot]]

A new release has been made for this PR. You can install it with:

npm i @apollo/client@0.0.0-pr-11837-20240509172621

[netlify[bot]]

✅ Deploy Preview for apollo-client-docs ready!

Name	Link
🔨 Latest commit	57515f3737087e9a3bc63cd25dc4ae5548fa504b
🔍 Latest deploy log	https://app.netlify.com/sites/apollo-client-docs/deploys/663d06e1a76bd60008bb7a25
😎 Deploy Preview	https://deploy-preview-11837--apollo-client-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[netlify[bot]]

✅ Deploy Preview for apollo-client-docs ready!

Name	Link
🔨 Latest commit	75bce0d23520330f5e182e4da455ca5f932568c7
🔍 Latest deploy log	https://app.netlify.com/sites/apollo-client-docs/deploys/66423c5c4fc98a0008007abb
😎 Deploy Preview	https://deploy-preview-11837--apollo-client-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.