Decorators not called about di HOT 6 CLOSED

Hm, I've tried it also in fresh nette project and the decorator was called as expected. So it will be probably in some other place, sorry for the issue.

from di.

It may be related to presenters being automatically registered as services (optionally). I guess your presenter is created in runtime and so does not exist in generated DIC and DecoratorExtension can't apply its setup.

Simplest way to fix it is register presenter as a service manually. Otherwise you will have to debug, why your presenter is not found by ApplicationExtension

from di.

Thanks for your answer! Yes, you are right, if I register e.g. HomepagePresenter, the decorator is finally called. But I don't understand why it worked before the update. 🤷‍♂️

Also, not sure what to do when I need to decorate abstract presenter. Do I have to list all of the child presenters? Maybe some GTMConfig service holding the boolean and passed to abstract presenter would be better then I guess..

from di.

Did you also update nette/application? Or do you have application: options like scanComposer, scanFilter, scanDir set?

Do I have to list all of the child presenters?

Yes, decorator works with service definitions, each presenter must be available as a service.

from di.

Did you also update nette/application?

No

Or do you have application: options like scanComposer, scanFilter, scanDir set?

Yes, scanDirs: false

from di.

There was probably some perf improvement in nette/di which caused that decorating unscanned presenters stopped working. It makes sense actually when I chose not to scan for presenters. 🙂 Did forget about that setting.

from di.