fix: move canPopulate method to the base-repository

This commit is contained in:
yassinedorbozgithub 2025-03-22 17:08:06 +01:00
parent e0a77302cc
commit db94b5c4d5
3 changed files with 8 additions and 9 deletions

View File

@ -1,5 +1,5 @@
/*
* Copyright © 2024 Hexastack. All rights reserved.
* Copyright © 2025 Hexastack. All rights reserved.
*
* Licensed under the GNU Affero General Public License v3.0 (AGPLv3) with the following additional terms:
* 1. The name "Hexabot" is a trademark of Hexastack. You may not use this name in derivative works without express written permission.
@ -32,12 +32,7 @@ export abstract class BaseController<
* @return - True if all populate fields are allowed, otherwise false.
*/
protected canPopulate(populate: string[]): boolean {
return populate.some((p) =>
this.service
.getRepository()
.getPopulate()
.includes(p as P),
);
return this.service.canPopulate(populate);
}
/**

View File

@ -89,8 +89,8 @@ export abstract class BaseRepository<
this.registerLifeCycleHooks();
}
getPopulate(): P[] {
return this.populate;
canPopulate(populate: string[]): boolean {
return populate.some((p) => this.populate.includes(p as P));
}
getEventName(suffix: EHook) {

View File

@ -34,6 +34,10 @@ export abstract class BaseService<
return this.repository;
}
canPopulate(populate: string[]): boolean {
return this.repository.canPopulate(populate);
}
async findOne(
criteria: string | TFilterQuery<T>,
options?: ClassTransformOptions,