feat(di): Added ability to override injectable property getter (#1381)

tsedio · May 5, 2021 · ef16124 · ef16124
1 parent 664970d
commit ef16124
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -7,7 +7,7 @@ on:
   push:
   pull_request:
     branches:
-      - production
+      - 5.x
 
 jobs:
   test:

diff --git a/packages/di/src/interfaces/IInjectableProperties.ts b/packages/di/src/interfaces/IInjectableProperties.ts
@@ -9,6 +9,7 @@ export interface IInjectablePropertyService extends IInjectableProperty {
   bindingType: InjectablePropertyType.METHOD | InjectablePropertyType.PROPERTY | InjectablePropertyType.INTERCEPTOR;
   propertyType: string;
   useType: TokenProvider;
+  onGet?: (bean: any) => any;
   options?: any;
 }
 

diff --git a/packages/di/src/services/InjectorService.ts b/packages/di/src/services/InjectorService.ts
@@ -398,7 +398,7 @@ export class InjectorService extends Container {
    */
   public bindProperty(
     instance: any,
-    {propertyKey, useType, options}: IInjectablePropertyService,
+    {propertyKey, useType, onGet = (f: any) => f, options}: IInjectablePropertyService,
     locals: Map<TokenProvider, any>,
     invokeOptions: Partial<IInvokeOptions>
   ) {
@@ -410,7 +410,7 @@ export class InjectorService extends Container {
     locals.delete(DI_PARAM_OPTIONS);
 
     Object.defineProperty(instance, propertyKey, {
-      get: () => bean
+      get: () => onGet(bean)
     });
   }
-Original file line number
+Diff line change
@@ Expand Up / @@ -7,7 +7,7 @@ on: @@
       push:
       pull_request:
         branches:
-          - production
+          - 5.x
     jobs:
       test:
@@ Expand Down @@