toss / nestjs-aop

A way to gracefully apply AOP to nestjs
MIT License
213 stars 24 forks source link

fix: Fix applyLazyDecorator return when metadataList is empty #30

Closed smilecc closed 8 months ago

smilecc commented 8 months ago

Overview

I noticed that due to this commit of f319395f1d0b8431b8aee70f1d1306f2811eeef9, when the applyLazyDecorator method checks that methodNames does not meet the conditions, it will return the current check function, even if other methods meet the conditions.

This may be because the return code was copied from forEach.

PR Checklist

  1. I have read the Contributing Guide
  2. I have written documents and tests, if needed.
WhiteKiwi commented 8 months ago

Thank you for your contribution