Skip to content

Commit f63e6f4

Browse files
danielcorreia96sergiofbsilva
authored andcommitted
MessagingUtils redirect doesn't use context FMSG-23 #resolve
1 parent 3e584b3 commit f63e6f4

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

messaging-core/src/main/java/org/fenixedu/messaging/core/ui/MessagingUtils.java

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,22 +5,25 @@
55
import java.io.IOException;
66
import java.util.Optional;
77

8-
public class MessagingUtils {
8+
public final class MessagingUtils {
9+
private static final String MESSAGING_MESSAGE_BEAN = "MESSAGING_MESSAGE_BEAN";
910

10-
public static final String MESSAGING_MESSAGE_BEAN = "MESSAGING_MESSAGE_BEAN";
11+
private MessagingUtils(){
12+
// Utility classes should have a private constructor to prevent instantiation.
13+
}
1114

12-
public static <T> T redirectToNewMessage(HttpServletRequest request, HttpServletResponse response, MessageBean messageBean)
15+
public static <T> T redirectToNewMessage(final HttpServletRequest request, final HttpServletResponse response, final MessageBean messageBean)
1316
throws IOException {
1417
request.getSession().setAttribute(MESSAGING_MESSAGE_BEAN, messageBean);
15-
response.sendRedirect("/messaging/message");
18+
response.sendRedirect(request.getContextPath() + "/messaging/message");
1619
return null;
1720
}
1821

19-
public static Optional<MessageBean> getMessageBeanFromSession(HttpServletRequest request) {
22+
public static Optional<MessageBean> getMessageBeanFromSession(final HttpServletRequest request) {
2023
return Optional.ofNullable((MessageBean) request.getSession().getAttribute(MESSAGING_MESSAGE_BEAN));
2124
}
2225

23-
public static void clearMessageBeanFromSession(HttpServletRequest request) {
26+
public static void clearMessageBeanFromSession(final HttpServletRequest request) {
2427
request.getSession().removeAttribute(MessagingUtils.MESSAGING_MESSAGE_BEAN);
2528
}
2629
}

0 commit comments

Comments
 (0)