Merge pull request #10970 from nestjs/fix/inject-decorator-signature

fix(common): inject decorator should allow key to be undefined
nestjs · Feb 1, 2023 · 268d930 · 268d930
2 parents b4d6c13 + 24a07d2
commit 268d930
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/common/decorators/core/inject.decorator.ts b/packages/common/decorators/core/inject.decorator.ts
@@ -34,7 +34,7 @@ import { isUndefined } from '../../utils/shared.utils';
  * @publicApi
  */
 export function Inject<T = any>(token?: T) {
-  return (target: object, key: string | symbol, index?: number) => {
+  return (target: object, key: string | symbol | undefined, index?: number) => {
     const type = token || Reflect.getMetadata('design:type', target, key);
 
     if (!isUndefined(index)) {

diff --git a/packages/common/decorators/core/optional.decorator.ts b/packages/common/decorators/core/optional.decorator.ts
@@ -18,7 +18,7 @@ import { isUndefined } from '../../utils/shared.utils';
  * @publicApi
  */
 export function Optional() {
-  return (target: object, key: string | symbol, index?: number) => {
+  return (target: object, key: string | symbol | undefined, index?: number) => {
     if (!isUndefined(index)) {
       const args = Reflect.getMetadata(OPTIONAL_DEPS_METADATA, target) || [];
       Reflect.defineMetadata(OPTIONAL_DEPS_METADATA, [...args, index], target);