Skip to content

Commit 69e9868

Browse files
committed
use provider.initialize in firestore
1 parent 14260d8 commit 69e9868

File tree

4 files changed

+16
-16
lines changed

4 files changed

+16
-16
lines changed

packages/firestore/exp/register.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ import { Component, ComponentType } from '@firebase/component';
2020

2121
import { version } from '../package.json';
2222
import { FirebaseFirestore } from '../src/exp/database';
23+
import { Settings } from '../src/exp/settings';
2324

2425
declare module '@firebase/component' {
2526
interface NameServiceMapping {
@@ -31,12 +32,16 @@ export function registerFirestore(): void {
3132
_registerComponent(
3233
new Component(
3334
'firestore-exp',
34-
container => {
35+
(container, { options: settings }: { options?: Settings }) => {
3536
const app = container.getProvider('app-exp').getImmediate()!;
36-
return ((app, auth) => new FirebaseFirestore(app, auth))(
37+
const firestoreInstance = new FirebaseFirestore(
3738
app,
3839
container.getProvider('auth-internal')
3940
);
41+
if (settings) {
42+
firestoreInstance._setSettings(settings);
43+
}
44+
return firestoreInstance;
4045
},
4146
ComponentType.PUBLIC
4247
)

packages/firestore/lite/register.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ import { Component, ComponentType } from '@firebase/component';
2020

2121
import { version } from '../package.json';
2222
import { FirebaseFirestore } from '../src/lite/database';
23+
import { Settings } from '../src/lite/settings';
2324

2425
declare module '@firebase/component' {
2526
interface NameServiceMapping {
@@ -31,12 +32,16 @@ export function registerFirestore(): void {
3132
_registerComponent(
3233
new Component(
3334
'firestore/lite',
34-
container => {
35+
(container, { options: settings }: { options?: Settings }) => {
3536
const app = container.getProvider('app-exp').getImmediate()!;
36-
return ((app, auth) => new FirebaseFirestore(app, auth))(
37+
const firestoreInstance = new FirebaseFirestore(
3738
app,
3839
container.getProvider('auth-internal')
3940
);
41+
if (settings) {
42+
firestoreInstance._setSettings(settings);
43+
}
44+
return firestoreInstance;
4045
},
4146
ComponentType.PUBLIC
4247
)

packages/firestore/src/exp/database.ts

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -128,10 +128,6 @@ export function initializeFirestore(
128128
);
129129
}
130130

131-
/**
132-
* create a firestore instance with the default settings
133-
*/
134-
const firestore = provider.getImmediate() as FirebaseFirestore;
135131
if (
136132
settings.cacheSizeBytes !== undefined &&
137133
settings.cacheSizeBytes !== CACHE_SIZE_UNLIMITED &&
@@ -143,11 +139,7 @@ export function initializeFirestore(
143139
);
144140
}
145141

146-
/**
147-
* update settings with the user supplied values on the firestore instance
148-
*/
149-
firestore._setSettings(settings);
150-
return firestore;
142+
return provider.initialize({ options: settings });
151143
}
152144

153145
/**

packages/firestore/src/lite/database.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -193,9 +193,7 @@ export function initializeFirestore(
193193
);
194194
}
195195

196-
const firestore = provider.getImmediate() as FirebaseFirestore;
197-
firestore._setSettings(settings);
198-
return firestore;
196+
return provider.initialize({ options: settings });
199197
}
200198

201199
/**

0 commit comments

Comments
 (0)