Skip to content

Commit a03dd67

Browse files
committed
Pass complete queries options to getOptimisticResult.
1 parent 69e7fda commit a03dd67

File tree

7 files changed

+14
-34
lines changed

7 files changed

+14
-34
lines changed

packages/angular-query-experimental/src/inject-queries.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -279,9 +279,7 @@ export function injectQueries<
279279
const optimisticResultSignal = computed(() =>
280280
observerSignal().getOptimisticResult(
281281
defaultedQueries(),
282-
(optionsSignal() as QueriesObserverOptions<TCombinedResult>).combine,
283-
(optionsSignal() as QueriesObserverOptions<TCombinedResult>)
284-
.structuralSharing,
282+
optionsSignal() as QueriesObserverOptions<TCombinedResult>,
285283
),
286284
)
287285

packages/preact-query/src/useQueries.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -270,8 +270,7 @@ export function useQueries<
270270
const [optimisticResult, getCombinedResult, trackResult] =
271271
observer.getOptimisticResult(
272272
defaultedQueries,
273-
(options as QueriesObserverOptions<TCombinedResult>).combine,
274-
options.structuralSharing,
273+
options as QueriesObserverOptions<TCombinedResult>,
275274
)
276275

277276
const shouldSubscribe = !isRestoring && options.subscribed !== false

packages/query-core/src/queriesObserver.ts

Lines changed: 6 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -180,11 +180,7 @@ export class QueriesObserver<
180180

181181
getOptimisticResult(
182182
queries: Array<QueryObserverOptions>,
183-
combine: CombineFn<TCombinedResult> | undefined,
184-
structuralSharing:
185-
| boolean
186-
| ((oldData: unknown | undefined, newData: unknown) => unknown)
187-
| undefined,
183+
options?: QueriesObserverOptions<TCombinedResult>,
188184
): [
189185
rawResult: Array<QueryObserverResult>,
190186
combineResult: (r?: Array<QueryObserverResult>) => TCombinedResult,
@@ -203,8 +199,7 @@ export class QueriesObserver<
203199
(r?: Array<QueryObserverResult>) => {
204200
return this.#combineResult(
205201
r ?? result,
206-
combine,
207-
structuralSharing,
202+
options,
208203
queryHashes,
209204
)
210205
},
@@ -233,13 +228,11 @@ export class QueriesObserver<
233228

234229
#combineResult(
235230
input: Array<QueryObserverResult>,
236-
combine: CombineFn<TCombinedResult> | undefined,
237-
structuralSharing:
238-
| boolean
239-
| ((oldData: unknown | undefined, newData: unknown) => unknown)
240-
| undefined = true,
231+
options?: QueriesObserverOptions<TCombinedResult>,
241232
queryHashes?: Array<string>,
242233
): TCombinedResult {
234+
const combine = options?.combine
235+
const structuralSharing = options?.structuralSharing ?? true
243236
if (combine) {
244237
const lastHashes = this.#lastQueryHashes
245238
const queryHashesChanged =
@@ -325,8 +318,7 @@ export class QueriesObserver<
325318
const newTracked = this.#trackResult(this.#result, this.#observerMatches)
326319
const newResult = this.#combineResult(
327320
newTracked,
328-
this.#options?.combine,
329-
this.#options?.structuralSharing,
321+
this.#options,
330322
)
331323

332324
if (previousResult !== newResult) {

packages/react-query/src/useQueries.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -271,8 +271,7 @@ export function useQueries<
271271
const [optimisticResult, getCombinedResult, trackResult] =
272272
observer.getOptimisticResult(
273273
defaultedQueries,
274-
(options as QueriesObserverOptions<TCombinedResult>).combine,
275-
options.structuralSharing,
274+
options as QueriesObserverOptions<TCombinedResult>,
276275
)
277276

278277
const shouldSubscribe = !isRestoring && options.subscribed !== false

packages/solid-query/src/useQueries.ts

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -235,9 +235,7 @@ export function useQueries<
235235
const [state, setState] = createStore<TCombinedResult>(
236236
observer.getOptimisticResult(
237237
defaultedQueries(),
238-
(queriesOptions() as QueriesObserverOptions<TCombinedResult>).combine,
239-
(queriesOptions() as QueriesObserverOptions<TCombinedResult>)
240-
.structuralSharing,
238+
queriesOptions() as QueriesObserverOptions<TCombinedResult>,
241239
)[1](),
242240
)
243241

@@ -248,10 +246,7 @@ export function useQueries<
248246
setState(
249247
observer.getOptimisticResult(
250248
defaultedQueries(),
251-
(queriesOptions() as QueriesObserverOptions<TCombinedResult>)
252-
.combine,
253-
(queriesOptions() as QueriesObserverOptions<TCombinedResult>)
254-
.structuralSharing,
249+
queriesOptions() as QueriesObserverOptions<TCombinedResult>,
255250
)[1](),
256251
),
257252
),

packages/svelte-query/src/createQueries.svelte.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -237,8 +237,7 @@ export function createQueries<
237237
function createResult() {
238238
const [_, getCombinedResult, trackResult] = observer.getOptimisticResult(
239239
resolvedQueryOptions,
240-
derivedCreateQueriesOptions.combine as QueriesObserverOptions<TCombinedResult>['combine'],
241-
derivedCreateQueriesOptions.structuralSharing,
240+
derivedCreateQueriesOptions as QueriesObserverOptions<TCombinedResult>,
242241
)
243242
return getCombinedResult(trackResult())
244243
}

packages/vue-query/src/useQueries.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -304,8 +304,7 @@ export function useQueries<
304304
const getOptimisticResult = () => {
305305
const [results, getCombinedResult] = observer.getOptimisticResult(
306306
defaultedQueries.value,
307-
(options as QueriesObserverOptions<TCombinedResult>).combine,
308-
options.structuralSharing,
307+
options as QueriesObserverOptions<TCombinedResult>,
309308
)
310309

311310
return getCombinedResult(
@@ -315,8 +314,7 @@ export function useQueries<
315314
refetch: async (...args: Array<any>) => {
316315
const [{ [index]: query }] = observer.getOptimisticResult(
317316
defaultedQueries.value,
318-
(options as QueriesObserverOptions<TCombinedResult>).combine,
319-
options.structuralSharing,
317+
options as QueriesObserverOptions<TCombinedResult>,
320318
)
321319

322320
return query!.refetch(...args)

0 commit comments

Comments
 (0)