Trunk 4988

[Sanskritii09]

Title:
Fix MessageService Injection Issue in Context Class

Description:
This PR resolves an issue with the MessageService dependency injection in the Context class. The following changes were made:

Ensured proper assignment of messageService using @Autowired.

Refactored setter method to maintain consistency.

Improved code readability and maintainability.

Testing:
Verified that the MessageService is correctly injected and accessible.

Checked for potential issues related to static fields and Spring’s dependency injection.

Please review and let me know if any further modifications are required. 🚀

[OmarTarekAli]

PR2

[OmarTarekAli]

public static MessageService getMessageService() {
MessageService ms = getServiceContext().getMessageService();
try {
if (ms.getMessagePreparator() == null) {
ms.setMessagePreparator(getMessagePreparator());
}

    if (ms.getMessageSender() == null) {
        ms.setMessageSender(getMessageSender());
    }

    validateMessageService(ms);

    return ms;
} catch (Exception e) {
    Logger.getLogger(YourClassName.class.getName()).log(Level.SEVERE, "Error initializing message service", e);
    throw new RuntimeException("Error initializing message service", e);
}

}

private static MessagePreparator getMessagePreparator() {
return new MessagePreparator();
}

private static MessageSender getMessageSender() {
return new MessageSender();
}

private static void validateMessageService(MessageService ms) {
if (ms == null) {
throw new IllegalArgumentException("MessageService cannot be null.");
}
}

[OmarTarekAli]

public static MessageService getMessageService() {
MessageService ms = getServiceContext().getMessageService();
try {
if (ms.getMessagePreparator() == null) {
ms.setMessagePreparator(getMessagePreparator());
}

    if (ms.getMessageSender() == null) {
        ms.setMessageSender(getMessageSender());
    }

    validateMessageService(ms);

    return ms;
} catch (Exception e) {
    Logger.getLogger(YourClassName.class.getName()).log(Level.SEVERE, "Error initializing message service", e);
    throw new RuntimeException("Error initializing message service", e);
}

}

private static MessagePreparator getMessagePreparator() {
return new MessagePreparator();
}

private static MessageSender getMessageSender() {
return new MessageSender();
}

private static void validateMessageService(MessageService ms) {
if (ms == null) {
throw new IllegalArgumentException("MessageService cannot be null.");
}
}

[atulyadav745]

Hey @Sanskritii09 , I would suggest you to please go through OpenMRS contribution Guidelines once, to have a better understanding on how a PR should look like.
Thanks!