Skip to content

Commit f921132

Browse files
authored
refactor: have FindOneOperation extend CommandOperation (#2604)
This refactor changes FindOneOperation so it extends CommandOperation instead of OperationBase, as is already the case with FindOperation. NODE-2853
1 parent 4018a1e commit f921132

File tree

1 file changed

+4
-7
lines changed

1 file changed

+4
-7
lines changed

src/operations/find_one.ts

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,21 @@
1-
import { OperationBase } from './operation';
21
import type { Callback } from '../utils';
3-
import { Document, resolveBSONOptions } from '../bson';
2+
import type { Document } from '../bson';
43
import type { Collection } from '../collection';
54
import type { FindOptions } from './find';
65
import { MongoError } from '../error';
76
import type { Server } from '../sdam/server';
7+
import { CommandOperation } from './command';
88

99
/** @internal */
10-
export class FindOneOperation extends OperationBase<FindOptions, Document> {
10+
export class FindOneOperation extends CommandOperation<FindOptions, Document> {
1111
collection: Collection;
1212
query: Document;
1313

1414
constructor(collection: Collection, query: Document, options: FindOptions) {
15-
super(options);
15+
super(collection, options);
1616

1717
this.collection = collection;
1818
this.query = query;
19-
20-
// Assign BSON serialize options to OperationBase, preferring options over collection options
21-
this.bsonOptions = resolveBSONOptions(options, collection);
2219
}
2320

2421
execute(server: Server, callback: Callback<Document>): void {

0 commit comments

Comments
 (0)