Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

decorator not working with autoDI #3

Closed
dakorpar opened this issue Nov 9, 2017 · 2 comments
Closed

decorator not working with autoDI #3

dakorpar opened this issue Nov 9, 2017 · 2 comments

Comments

@dakorpar
Copy link

dakorpar commented Nov 9, 2017

Consider this:

decorator:
    App\Repositories\BaseRepository:
        inject: on
autoDI:
    services:
        - class: App\Repositories\*Repository

inject functions are not called when using autoDI but works perfectly when registered manually.

@dakorpar
Copy link
Author

dakorpar commented Nov 9, 2017

This works so it's kinda ok, although decorator should work with this, otherwise sometimes you'll have to repeat the same through multiple folders

autoDI:
    services:
        - class: App\Repositories\*Repository
          inject: on

@fmasa fmasa added bug and removed bug labels Nov 14, 2017
@fmasa
Copy link
Owner

fmasa commented Nov 15, 2017

Hi, thanks for reporting.
I'm working on service registration in on configuration time (instead of compile time), which should solve the issue. I expect it to land later this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants