yasserzamani commented on PR #573: URL: https://github.com/apache/struts/pull/573#issuecomment-1161673971
Thank you so much @JCgH4164838Gh792C124B5 ! I remember that I also had some same difficulties when I was working on #167 . Finally I simply workaround it by injecting Container itself and getting required bean instead of injecting the bean itself. This isn't a good practice albeit, as Lukasz commented there. But this is what I could and able to do that time to make it working. Even with a lot debugging. I'm trying to say that your case is same but more complex, i.e. with constructor injection with multiple overloaded constructors. But you were able to make it work with no hack. So I think now you know DI better than any of us and your changes are best possible solution :) BTW I found all usages of `@Inject` in constructors parameters by IntelliJ IDEA. There are similar cases already implemented in Struts. I assume they're working. So the problem is not because of multiple overloaded constructors I think. And could you please explain a bit more about: > but the default container instance construction of OgnlUtil still seems to use the default implementations Thanks again! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@struts.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
