Uploaded image for project: 'OpenMRS Core'
  1. OpenMRS Core
  2. TRUNK-5306

Use Spring's method injection to set the dependencies for the message service

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • TBD
    • Resolution: Won't Fix
    • None
    • None
    • None
    • Undetermined

    Description

      In Context.java from org.openmrs.api.context on line 569 and 579:

       

      /**
  * Convenience method to allow us to change the configuration more easily. TODO Ideally, we
  * would be using Spring's method injection to set the dependencies for the message service.
  *
  * @return the ServiceContext
  */
private static MessageSender getMessageSender() {
  return new MailMessageSender(getMailSession());
 }

       

      Gliffy Diagrams

        Attachments

          Issue Links

            relates to

            Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. TRUNK-324 Go through the TODO items code and create tickets out of them

            • Should - Issue should be completed for its milestone; however, will not delay the milestone.
            • Code Review (Initial)

            Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. TRUNK-6087 COPY: Go through the TODO items code and create tickets out of them

            • Should - Issue should be completed for its milestone; however, will not delay the milestone.
            • Needs Assessment

            Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. TRUNK-6108 COPY: Go through the TODO items code and create tickets out of them

            • Should - Issue should be completed for its milestone; however, will not delay the milestone.
            • Needs Assessment

            Activity

              People

                Unassigned Unassigned
                sherlocktech Sean Stamm
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: