From f32aee9926b8cd77027b008738744022c09f74d7 Mon Sep 17 00:00:00 2001 From: SaltedFish Date: Wed, 20 Dec 2023 16:16:13 +0800 Subject: [PATCH] chore: rename methodNames to propertyKeys at applyLazyDecorator --- src/auto-aspect-executor.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/auto-aspect-executor.ts b/src/auto-aspect-executor.ts index a8f5100..91c5ad2 100644 --- a/src/auto-aspect-executor.ts +++ b/src/auto-aspect-executor.ts @@ -47,7 +47,7 @@ export class AutoAspectExecutor implements OnModuleInit { : instanceWrapper.metatype.prototype; // Use scanFromPrototype for support nestjs 8 - const methodNames = this.metadataScanner.scanFromPrototype( + const propertyKeys = this.metadataScanner.scanFromPrototype( target, instanceWrapper.isDependencyTreeStatic() ? Object.getPrototypeOf(target) : target, (name) => name, @@ -55,24 +55,24 @@ export class AutoAspectExecutor implements OnModuleInit { const metadataKey = this.reflector.get(ASPECT, lazyDecorator.constructor); // instance에 method names 를 순회하면서 lazyDecorator.wrap을 적용함 - for (const methodName of methodNames) { + for (const propertyKey of propertyKeys) { // the target method is must be object or function // @see: https://github.com/rbuckton/reflect-metadata/blob/9562d6395cc3901eaafaf8a6ed8bc327111853d5/Reflect.ts#L938 - const targetMethod = target[methodName]; - if (!targetMethod || (typeof targetMethod !== "object" && typeof targetMethod !== "function")) { + const targetProperty = target[propertyKey]; + if (!targetProperty || (typeof targetProperty !== "object" && typeof targetProperty !== "function")) { continue; } const metadataList: AopMetadata[] = this.reflector.get( metadataKey, - targetMethod, + targetProperty, ); if (!metadataList) { continue; } for (const aopMetadata of metadataList) { - this.wrapMethod({ lazyDecorator, aopMetadata, methodName, target }); + this.wrapMethod({ lazyDecorator, aopMetadata, methodName: propertyKey, target }); } } }