@lazy not working with @autoinject

I’m submitting a bug report

• Library Version: 1.3.2

Please tell us about your environment:

• Operating System: Windows 10

• Node Version: 6.11.2

• NPM Version: 4.6.1

• JSPM OR Webpack AND Version JSPM 0.16.48 (reproduced with Aurelia CLI v0.31.1)

• Browser: all

• Language: TypeScript 2.4.2

Current behavior: Constructor arguments decorated with @lazy are injected with undefined when class is @autoinjected. Code, live demo.

Expected/desired behavior:

• What is the expected behavior? @autoinject and @lazy should work well together. It probably was after #111 was merged.

• What is the motivation / use case for changing the behavior? I want to be able to do this:

@autoinject
export class Alert {
  constructor(private i18n: I18N, @lazy(TemplatingEngine) private templatingEngine: () => TemplatingEngine) {
  }
}

[doktordirk]

@ccrd yes, for all the resolver decorators, the type of the metadata and the inject property won’t match and hence the current implementation of autoinject+resolver_decorators won’t work.

@EisenbergEffect as such sure, but that was a hack. anyways, i did dig further and found that this bug was introduced when fixing that issue #121 in pr #129. however, that issue highlights, that the design of @lazy et al., didn’t took inheritance into account. i’ll see if i can find some solution

[EisenbergEffect]

Merged the fix 😃