Github user nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1361#issuecomment-178346694
@rafaelweingartner thanks for your reply!
I'm familiar to Spring, I've worked with it, I know how it works but I
wouldn't say I know it in depth. If you don't mind I would find your detailed
explanation really helpful for this feature and future tasks.
About bean injection in <code>TemplateServiceImpl</code>, I first tryied
<code>@Inject</code> annotation, but ACS couldn't start because it couldn't
find a bean of type <code>ImageStoreDetailsUtil</code>. I had the same issue
with <code>@Autowired</code> annotation. The only solution I could find was
adding bean in xml context file.
I agree with you about not extending <code>Manager</code> interface, I saw
it was always done like that and just followed the pattern. I will remove that
from <code>ImageStoreDetailsUtil</code>
I could start ACS but didn't fully test <code>Vmware*</code> classes. I
would try to get the context and then the bean from it as you suggested.
Thanks a lot for your help!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
