solomax commented on pull request #136:
URL: https://github.com/apache/openmeetings/pull/136#issuecomment-791866695


   > They are a mix of "behaviour" and "storage" which is the nature of the 
problem why you need some strange constructs in order to load dependencies into 
them.
   > 
   > Behaviour should be in Controllers. Holding information should be DTOs. 
Mixing them leads to issues. Which is one of the reasons why it would be good 
to refactor this part of the code.
   
   In such case we can
   
   - directly get Bean via Spring's magic 
https://github.com/apache/openmeetings/blob/master/openmeetings-db/src/main/java/org/apache/openmeetings/IApplication.java#L34
   - manually perform injections like 
https://www.baeldung.com/spring-inject-bean-into-unmanaged-objects
   
   I can do some investigation on this
   should not take too much time :)))
   
   will write back here :)
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to