Skip to content

Commit 7955ecf

Browse files
clayrisserbajtos
authored andcommitted
refactor(repository): remove underscore from hasOne protected method
1 parent 28185b6 commit 7955ecf

File tree

3 files changed

+30
-3
lines changed

3 files changed

+30
-3
lines changed

examples/todo-list/src/repositories/todo-list.repository.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ export class TodoListRepository extends DefaultCrudRepository<
4040
'todos',
4141
todoRepositoryGetter,
4242
);
43-
this.image = this._createHasOneRepositoryFactoryFor(
43+
this.image = this.createHasOneRepositoryFactoryFor(
4444
'image',
4545
todoListImageRepositoryGetter,
4646
);

packages/repository/src/__tests__/fixtures/repositories/customer.repository.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,11 @@ export class CustomerRepository extends DefaultCrudRepository<
3535
addressRepositoryGetter: Getter<AddressRepository>,
3636
) {
3737
super(Customer, db);
38-
this.orders = this._createHasManyRepositoryFactoryFor(
38+
this.orders = this.createHasManyRepositoryFactoryFor(
3939
'orders',
4040
orderRepositoryGetter,
4141
);
42-
this.address = this._createHasOneRepositoryFactoryFor(
42+
this.address = this.createHasOneRepositoryFactoryFor(
4343
'address',
4444
addressRepositoryGetter,
4545
);

packages/repository/src/repositories/legacy-juggler-bridge.ts

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -244,13 +244,40 @@ export class DefaultCrudRepository<T extends Entity, ID>
244244
);
245245
}
246246

247+
/**
248+
* @deprecated
249+
* Function to create a constrained hasOne relation repository factory
250+
*
251+
* @param relationName Name of the relation defined on the source model
252+
* @param targetRepo Target repository instance
253+
*/
247254
protected _createHasOneRepositoryFactoryFor<
248255
Target extends Entity,
249256
TargetID,
250257
ForeignKeyType
251258
>(
252259
relationName: string,
253260
targetRepoGetter: Getter<EntityCrudRepository<Target, TargetID>>,
261+
): HasOneRepositoryFactory<Target, ForeignKeyType> {
262+
return this.createHasOneRepositoryFactoryFor(
263+
relationName,
264+
targetRepoGetter,
265+
);
266+
}
267+
268+
/**
269+
* Function to create a constrained hasOne relation repository factory
270+
*
271+
* @param relationName Name of the relation defined on the source model
272+
* @param targetRepo Target repository instance
273+
*/
274+
protected createHasOneRepositoryFactoryFor<
275+
Target extends Entity,
276+
TargetID,
277+
ForeignKeyType
278+
>(
279+
relationName: string,
280+
targetRepoGetter: Getter<EntityCrudRepository<Target, TargetID>>,
254281
): HasOneRepositoryFactory<Target, ForeignKeyType> {
255282
const meta = this.entityClass.definition.relations[relationName];
256283
return createHasOneRepositoryFactory<Target, TargetID, ForeignKeyType>(

0 commit comments

Comments
 (0)