Skip to content

Commit 3a039c1

Browse files
authored
Merge pull request #1751 from microsoft/fix/create-client
fix: client creation fails because of instanceof use
2 parents b00d6c2 + 88c0604 commit 3a039c1

File tree

1 file changed

+12
-19
lines changed

1 file changed

+12
-19
lines changed

packages/bundle/src/defaultRequestAdapter.ts

Lines changed: 12 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -30,27 +30,20 @@ export class DefaultRequestAdapter extends FetchRequestAdapter {
3030
}
3131

3232
private setupDefaults() {
33-
let parseNodeFactoryRegistry: ParseNodeFactoryRegistry;
34-
if (super.getParseNodeFactory() instanceof ParseNodeFactoryRegistry) {
35-
parseNodeFactoryRegistry = super.getParseNodeFactory() as ParseNodeFactoryRegistry;
36-
} else {
37-
throw new Error("ParseNodeFactory must be a ParseNodeFactoryRegistry");
33+
const parseNodeFactoryRegistry = super.getParseNodeFactory() as ParseNodeFactoryRegistry;
34+
const serializationWriterFactoryRegistry = super.getSerializationWriterFactory() as SerializationWriterFactoryRegistry;
35+
const backingStoreFactory = super.getBackingStoreFactory();
36+
if (parseNodeFactoryRegistry.registerDefaultDeserializer) {
37+
parseNodeFactoryRegistry.registerDefaultDeserializer(TextParseNodeFactory, backingStoreFactory);
38+
parseNodeFactoryRegistry.registerDefaultDeserializer(JsonParseNodeFactory, backingStoreFactory);
39+
parseNodeFactoryRegistry.registerDefaultDeserializer(FormParseNodeFactory, backingStoreFactory);
3840
}
3941

40-
let serializationWriterFactoryRegistry: SerializationWriterFactoryRegistry;
41-
if (super.getSerializationWriterFactory() instanceof SerializationWriterFactoryRegistry) {
42-
serializationWriterFactoryRegistry = super.getSerializationWriterFactory() as SerializationWriterFactoryRegistry;
43-
} else {
44-
throw new Error("SerializationWriterFactory must be a SerializationWriterFactoryRegistry");
42+
if (serializationWriterFactoryRegistry.registerDefaultSerializer) {
43+
serializationWriterFactoryRegistry.registerDefaultSerializer(JsonSerializationWriterFactory);
44+
serializationWriterFactoryRegistry.registerDefaultSerializer(TextSerializationWriterFactory);
45+
serializationWriterFactoryRegistry.registerDefaultSerializer(FormSerializationWriterFactory);
46+
serializationWriterFactoryRegistry.registerDefaultSerializer(MultipartSerializationWriterFactory);
4547
}
46-
47-
const backingStoreFactory = super.getBackingStoreFactory();
48-
serializationWriterFactoryRegistry.registerDefaultSerializer(JsonSerializationWriterFactory);
49-
serializationWriterFactoryRegistry.registerDefaultSerializer(TextSerializationWriterFactory);
50-
serializationWriterFactoryRegistry.registerDefaultSerializer(FormSerializationWriterFactory);
51-
serializationWriterFactoryRegistry.registerDefaultSerializer(MultipartSerializationWriterFactory);
52-
parseNodeFactoryRegistry.registerDefaultDeserializer(TextParseNodeFactory, backingStoreFactory);
53-
parseNodeFactoryRegistry.registerDefaultDeserializer(JsonParseNodeFactory, backingStoreFactory);
54-
parseNodeFactoryRegistry.registerDefaultDeserializer(FormParseNodeFactory, backingStoreFactory);
5548
}
5649
}

0 commit comments

Comments
 (0)