Skip to content

Commit 2db7500

Browse files
committed
monir refactor
1 parent 54a6d22 commit 2db7500

File tree

3 files changed

+40
-39
lines changed

3 files changed

+40
-39
lines changed

packages/abstractions/src/serialization/kiotaJsonSerializer.ts

+7-6
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import type { ParsableFactory } from "./parsableFactory";
1010
import type { ModelSerializerFunction } from "./serializationFunctionTypes";
1111
import { SerializationWriterFactoryRegistry } from "./serializationWriterFactoryRegistry";
1212
import { ParseNodeFactoryRegistry } from "./parseNodeFactoryRegistry";
13+
import type { ParseNodeFactory } from "./parseNodeFactory";
1314

1415
const jsonContentType = "application/json";
1516
/**
@@ -58,22 +59,22 @@ export function serializeCollectionToJsonAsString<T extends Parsable>(serializat
5859

5960
/**
6061
* Deserializes a buffer into a parsable object
61-
* @param parseNodeFactoryRegistry the parse node factory registry
62+
* @param parseNodeFactory the parse node factory for the content type
6263
* @param bufferOrString the value to serialize
6364
* @param factory the factory for the model type
6465
* @returns the deserialized parsable object
6566
*/
66-
export function deserializeFromJson<T extends Parsable>(parseNodeFactoryRegistry: ParseNodeFactoryRegistry, bufferOrString: ArrayBuffer | string, factory: ParsableFactory<T>): Parsable {
67-
return deserialize(parseNodeFactoryRegistry, jsonContentType, bufferOrString, factory);
67+
export function deserializeFromJson<T extends Parsable>(parseNodeFactory: ParseNodeFactory, bufferOrString: ArrayBuffer | string, factory: ParsableFactory<T>): Parsable {
68+
return deserialize(parseNodeFactory, jsonContentType, bufferOrString, factory);
6869
}
6970

7071
/**
7172
* Deserializes a buffer into a collection of parsable object
72-
* @param parseNodeFactoryRegistry the parse node factory registry
73+
* @param parseNodeFactory the parse node factory for the content type
7374
* @param bufferOrString the value to serialize
7475
* @param factory the factory for the model type
7576
* @returns the deserialized collection of parsable objects
7677
*/
77-
export function deserializeCollectionFromJson<T extends Parsable>(parseNodeFactoryRegistry: ParseNodeFactoryRegistry, bufferOrString: ArrayBuffer | string, factory: ParsableFactory<T>): T[] | undefined {
78-
return deserializeCollection(parseNodeFactoryRegistry, jsonContentType, bufferOrString, factory);
78+
export function deserializeCollectionFromJson<T extends Parsable>(parseNodeFactory: ParseNodeFactory, bufferOrString: ArrayBuffer | string, factory: ParsableFactory<T>): T[] | undefined {
79+
return deserializeCollection(parseNodeFactory, jsonContentType, bufferOrString, factory);
7980
}

packages/abstractions/src/serialization/kiotaSerializer.ts

+26-26
Original file line numberDiff line numberDiff line change
@@ -7,72 +7,72 @@
77
import type { Parsable } from "./parsable";
88
import type { ParsableFactory } from "./parsableFactory";
99
import type { ParseNode } from "./parseNode";
10-
import { ParseNodeFactoryRegistry } from "./parseNodeFactoryRegistry";
1110
import type { ModelSerializerFunction } from "./serializationFunctionTypes";
1211
import type { SerializationWriter } from "./serializationWriter";
13-
import { SerializationWriterFactoryRegistry } from "./serializationWriterFactoryRegistry";
12+
import { SerializationWriterFactory } from "./serializationWriterFactory";
13+
import { ParseNodeFactory } from "./parseNodeFactory";
1414

1515
/**
1616
* Serializes a parsable object into a buffer
17-
* @param serializationWriterFactoryRegistry the serialization writer factory registry
17+
* @param serializationWriterFactory the serialization writer factory for the content type
1818
* @param contentType the content type to serialize to
1919
* @param value the value to serialize
2020
* @param serializationFunction the serialization function for the model type
2121
* @returns a buffer containing the serialized value
2222
*/
23-
export function serialize<T extends Parsable>(serializationWriterFactoryRegistry: SerializationWriterFactoryRegistry, contentType: string, value: T, serializationFunction: ModelSerializerFunction<T>): ArrayBuffer {
24-
const writer = getSerializationWriter(serializationWriterFactoryRegistry, contentType, value, serializationFunction);
23+
export function serialize<T extends Parsable>(serializationWriterFactory: SerializationWriterFactory, contentType: string, value: T, serializationFunction: ModelSerializerFunction<T>): ArrayBuffer {
24+
const writer = getSerializationWriter(serializationWriterFactory, contentType, value, serializationFunction);
2525
writer.writeObjectValue(undefined, value, serializationFunction);
2626
return writer.getSerializedContent();
2727
}
2828
/**
2929
* Serializes a parsable object into a string representation
30-
* @param serializationWriterFactoryRegistry the serialization writer factory registry
30+
* @param serializationWriterFactory the serialization writer factory for the content type
3131
* @param contentType the content type to serialize to
3232
* @param value the value to serialize
3333
* @param serializationFunction the serialization function for the model type
3434
* @returns a string representing the serialized value
3535
*/
36-
export function serializeToString<T extends Parsable>(serializationWriterFactoryRegistry: SerializationWriterFactoryRegistry, contentType: string, value: T, serializationFunction: ModelSerializerFunction<T>): string {
37-
const buffer = serialize(serializationWriterFactoryRegistry, contentType, value, serializationFunction);
36+
export function serializeToString<T extends Parsable>(serializationWriterFactory: SerializationWriterFactory, contentType: string, value: T, serializationFunction: ModelSerializerFunction<T>): string {
37+
const buffer = serialize(serializationWriterFactory, contentType, value, serializationFunction);
3838
return getStringValueFromBuffer(buffer);
3939
}
4040
/**
4141
* Serializes a collection of parsable objects into a buffer
42-
* @param serializationWriterFactoryRegistry the serialization writer factory registry
42+
* @param serializationWriterFactory the serialization writer factory for the content type
4343
* @param contentType the content type to serialize to
4444
* @param values the value to serialize
4545
* @param serializationFunction the serialization function for the model type
4646
* @returns a string representing the serialized value
4747
*/
48-
export function serializeCollection<T extends Parsable>(serializationWriterFactoryRegistry: SerializationWriterFactoryRegistry, contentType: string, values: T[], serializationFunction: ModelSerializerFunction<T>): ArrayBuffer {
49-
const writer = getSerializationWriter(serializationWriterFactoryRegistry, contentType, values, serializationFunction);
48+
export function serializeCollection<T extends Parsable>(serializationWriterFactory: SerializationWriterFactory, contentType: string, values: T[], serializationFunction: ModelSerializerFunction<T>): ArrayBuffer {
49+
const writer = getSerializationWriter(serializationWriterFactory, contentType, values, serializationFunction);
5050
writer.writeCollectionOfObjectValues(undefined, values, serializationFunction);
5151
return writer.getSerializedContent();
5252
}
5353

5454
/**
5555
* Serializes a collection of parsable objects into a string representation
56-
* @param serializationWriterFactoryRegistry the serialization writer factory registry
56+
* @param serializationWriterFactory the serialization writer factory for the content type
5757
* @param contentType the content type to serialize to
5858
* @param values the value to serialize
5959
* @param serializationFunction the serialization function for the model type
6060
* @returns a string representing the serialized value
6161
*/
62-
export function serializeCollectionToString<T extends Parsable>(serializationWriterFactoryRegistry: SerializationWriterFactoryRegistry, contentType: string, values: T[], serializationFunction: ModelSerializerFunction<T>): string {
63-
const buffer = serializeCollection(serializationWriterFactoryRegistry, contentType, values, serializationFunction);
62+
export function serializeCollectionToString<T extends Parsable>(serializationWriterFactory: SerializationWriterFactory, contentType: string, values: T[], serializationFunction: ModelSerializerFunction<T>): string {
63+
const buffer = serializeCollection(serializationWriterFactory, contentType, values, serializationFunction);
6464
return getStringValueFromBuffer(buffer);
6565
}
6666

6767
/**
6868
* Gets a serialization writer for a given content type
69-
* @param serializationWriterFactoryRegistry the serialization writer factory registry
69+
* @param serializationWriterFactory the serialization writer factory for the content type
7070
* @param contentType the content type to serialize to
7171
* @param value the value to serialize
7272
* @param serializationFunction the serialization function for the model type
7373
* @returns the serialization writer for the given content type
7474
*/
75-
function getSerializationWriter(serializationWriterFactoryRegistry: SerializationWriterFactoryRegistry, contentType: string, value: unknown, serializationFunction: unknown): SerializationWriter {
75+
function getSerializationWriter(serializationWriterFactory: SerializationWriterFactory, contentType: string, value: unknown, serializationFunction: unknown): SerializationWriter {
7676
if (!contentType) {
7777
throw new Error("content type cannot be undefined or empty");
7878
}
@@ -82,7 +82,7 @@ function getSerializationWriter(serializationWriterFactoryRegistry: Serializatio
8282
if (!serializationFunction) {
8383
throw new Error("serializationFunction cannot be undefined");
8484
}
85-
return serializationWriterFactoryRegistry.getSerializationWriter(contentType);
85+
return serializationWriterFactory.getSerializationWriter(contentType);
8686
}
8787

8888
/**
@@ -97,28 +97,28 @@ function getStringValueFromBuffer(buffer: ArrayBuffer): string {
9797

9898
/**
9999
* Deserializes a buffer into a parsable object
100-
* @param parseNodeFactoryRegistry the parse node factory registry
100+
* @param parseNodeFactory the parse node factory for the content type
101101
* @param contentType the content type to serialize to
102102
* @param bufferOrString the value to serialize
103103
* @param factory the factory for the model type
104104
* @returns the deserialized parsable object
105105
*/
106-
export function deserialize<T extends Parsable>(parseNodeFactoryRegistry: ParseNodeFactoryRegistry, contentType: string, bufferOrString: ArrayBuffer | string, factory: ParsableFactory<T>): Parsable {
106+
export function deserialize<T extends Parsable>(parseNodeFactory: ParseNodeFactory, contentType: string, bufferOrString: ArrayBuffer | string, factory: ParsableFactory<T>): Parsable {
107107
if (typeof bufferOrString === "string") {
108108
bufferOrString = getBufferFromString(bufferOrString);
109109
}
110-
const reader = getParseNode(parseNodeFactoryRegistry, contentType, bufferOrString, factory);
110+
const reader = getParseNode(parseNodeFactory, contentType, bufferOrString, factory);
111111
return reader.getObjectValue(factory);
112112
}
113113
/**
114114
* Deserializes a buffer into a parsable object
115-
* @param parseNodeFactoryRegistry the parse node factory registry
115+
* @param parseNodeFactory the parse node factory for the content type
116116
* @param contentType the content type to serialize to
117117
* @param buffer the value to deserialize
118118
* @param factory the factory for the model type
119119
* @returns the deserialized parsable object
120120
*/
121-
function getParseNode(parseNodeFactoryRegistry: ParseNodeFactoryRegistry, contentType: string, buffer: ArrayBuffer, factory: unknown): ParseNode {
121+
function getParseNode(parseNodeFactory: ParseNodeFactory, contentType: string, buffer: ArrayBuffer, factory: unknown): ParseNode {
122122
if (!contentType) {
123123
throw new Error("content type cannot be undefined or empty");
124124
}
@@ -128,21 +128,21 @@ function getParseNode(parseNodeFactoryRegistry: ParseNodeFactoryRegistry, conten
128128
if (!factory) {
129129
throw new Error("factory cannot be undefined");
130130
}
131-
return parseNodeFactoryRegistry.getRootParseNode(contentType, buffer);
131+
return parseNodeFactory.getRootParseNode(contentType, buffer);
132132
}
133133
/**
134134
* Deserializes a buffer into a collection of parsable object
135-
* @param parseNodeFactoryRegistry the parse node factory registry
135+
* @param parseNodeFactory the parse node factory for the content type
136136
* @param contentType the content type to serialize to
137137
* @param bufferOrString the value to serialize
138138
* @param factory the factory for the model type
139139
* @returns the deserialized collection of parsable objects
140140
*/
141-
export function deserializeCollection<T extends Parsable>(parseNodeFactoryRegistry: ParseNodeFactoryRegistry, contentType: string, bufferOrString: ArrayBuffer | string, factory: ParsableFactory<T>): T[] | undefined {
141+
export function deserializeCollection<T extends Parsable>(parseNodeFactory: ParseNodeFactory, contentType: string, bufferOrString: ArrayBuffer | string, factory: ParsableFactory<T>): T[] | undefined {
142142
if (typeof bufferOrString === "string") {
143143
bufferOrString = getBufferFromString(bufferOrString);
144144
}
145-
const reader = getParseNode(parseNodeFactoryRegistry, contentType, bufferOrString, factory);
145+
const reader = getParseNode(parseNodeFactory, contentType, bufferOrString, factory);
146146
return reader.getCollectionOfObjectValues(factory);
147147
}
148148

packages/bundle/src/defaultRequestAdapter.ts

+7-7
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@ export class DefaultRequestAdapter extends FetchRequestAdapter {
3030
}
3131

3232
private setupDefaults() {
33-
registerDefaultSerializer(super.getSerializationWriterFactory(), JsonSerializationWriterFactory);
34-
registerDefaultSerializer(super.getSerializationWriterFactory(), TextSerializationWriterFactory);
35-
registerDefaultSerializer(super.getSerializationWriterFactory(), FormSerializationWriterFactory);
36-
registerDefaultSerializer(super.getSerializationWriterFactory(), MultipartSerializationWriterFactory);
37-
registerDefaultDeserializer(super.getSerializationWriterFactory(), JsonParseNodeFactory);
38-
registerDefaultDeserializer(super.getSerializationWriterFactory(), TextParseNodeFactory);
39-
registerDefaultDeserializer(super.getSerializationWriterFactory(), FormParseNodeFactory);
33+
registerDefaultSerializer(super.getSerializationWriterFactory() as SerializationWriterFactoryRegistry, JsonSerializationWriterFactory);
34+
registerDefaultSerializer(super.getSerializationWriterFactory() as SerializationWriterFactoryRegistry, TextSerializationWriterFactory);
35+
registerDefaultSerializer(super.getSerializationWriterFactory() as SerializationWriterFactoryRegistry, FormSerializationWriterFactory);
36+
registerDefaultSerializer(super.getSerializationWriterFactory() as SerializationWriterFactoryRegistry, MultipartSerializationWriterFactory);
37+
registerDefaultDeserializer(super.getParseNodeFactory() as ParseNodeFactoryRegistry, JsonParseNodeFactory);
38+
registerDefaultDeserializer(super.getParseNodeFactory() as ParseNodeFactoryRegistry, TextParseNodeFactory);
39+
registerDefaultDeserializer(super.getParseNodeFactory() as ParseNodeFactoryRegistry, FormParseNodeFactory);
4040
}
4141
}

0 commit comments

Comments
 (0)