mirror of https://github.com/apache/activemq.git
[AMQ-4174] Redirect back to queue overview after delete/moving message
Patch from Robert Balent applied
This commit is contained in:
parent
146fb12108
commit
3321f99643
|
@ -49,7 +49,7 @@ public class CopyMessage extends DestinationFacade implements Controller {
|
||||||
log.warn("No queue named: " + getPhysicalDestinationName());
|
log.warn("No queue named: " + getPhysicalDestinationName());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return redirectToBrowseView();
|
return redirectToDestinationView();
|
||||||
}
|
}
|
||||||
|
|
||||||
public String getMessageId() {
|
public String getMessageId() {
|
||||||
|
|
|
@ -48,7 +48,7 @@ public class DeleteMessage extends DestinationFacade implements Controller {
|
||||||
log.warn("No queue named: " + getPhysicalDestinationName());
|
log.warn("No queue named: " + getPhysicalDestinationName());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return redirectToBrowseView();
|
return redirectToDestinationView();
|
||||||
}
|
}
|
||||||
|
|
||||||
public String getMessageId() {
|
public String getMessageId() {
|
||||||
|
|
|
@ -49,7 +49,7 @@ public class MoveMessage extends DestinationFacade implements Controller {
|
||||||
log.warn("No queue named: " + getPhysicalDestinationName());
|
log.warn("No queue named: " + getPhysicalDestinationName());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return redirectToBrowseView();
|
return redirectToDestinationView();
|
||||||
}
|
}
|
||||||
|
|
||||||
public String getMessageId() {
|
public String getMessageId() {
|
||||||
|
|
|
@ -133,10 +133,10 @@ No message could be found for ID <c:out value="${requestContext.messageQuery.id}
|
||||||
</thead>
|
</thead>
|
||||||
<tbody>
|
<tbody>
|
||||||
<tr>
|
<tr>
|
||||||
<td colspan="2"><a href="deleteMessage.action?JMSDestination=<c:out value="${row.JMSDestination}" />&messageId=${row.JMSMessageID}&secret=<c:out value='${sessionScope["secret"]}'/>">Delete</a></td>
|
<td colspan="2"><a href="deleteMessage.action?JMSDestination=<c:out value="${requestContext.messageQuery.JMSDestination}" />&messageId=${row.JMSMessageID}&secret=<c:out value='${sessionScope["secret"]}'/>">Delete</a></td>
|
||||||
</tr>
|
</tr>
|
||||||
<tr class="odd">
|
<tr class="odd">
|
||||||
<td><a href="javascript:confirmAction('queue', 'copyMessage.action?destination=%target%&JMSDestination=<c:out value="${row.JMSDestination}" />&messageId=${row.JMSMessageID}&JMSDestinationType=queue&secret=<c:out value='${sessionScope["secret"]}'/>')">Copy</a></td>
|
<td><a href="javascript:confirmAction('queue', 'copyMessage.action?destination=%target%&JMSDestination=<c:out value="${requestContext.messageQuery.JMSDestination}" />&messageId=${row.JMSMessageID}&JMSDestinationType=queue&secret=<c:out value='${sessionScope["secret"]}'/>')">Copy</a></td>
|
||||||
<td rowspan="2">
|
<td rowspan="2">
|
||||||
<select id="queue">
|
<select id="queue">
|
||||||
<option value=""> -- Please select --</option>
|
<option value=""> -- Please select --</option>
|
||||||
|
@ -150,7 +150,7 @@ No message could be found for ID <c:out value="${requestContext.messageQuery.id}
|
||||||
|
|
||||||
</tr>
|
</tr>
|
||||||
<tr class="odd">
|
<tr class="odd">
|
||||||
<td><a href="javascript:confirmAction('queue', 'moveMessage.action?destination=%target%&JMSDestination=<c:out value="${row.JMSDestination}" />&messageId=${row.JMSMessageID}&JMSDestinationType=queue&secret=<c:out value='${sessionScope["secret"]}'/>')">Move</a></td>
|
<td><a href="javascript:confirmAction('queue', 'moveMessage.action?destination=%target%&JMSDestination=<c:out value="${requestContext.messageQuery.JMSDestination}" />&messageId=${row.JMSMessageID}&JMSDestinationType=queue&secret=<c:out value='${sessionScope["secret"]}'/>')">Move</a></td>
|
||||||
</tr>
|
</tr>
|
||||||
</tbody>
|
</tbody>
|
||||||
</table>
|
</table>
|
||||||
|
|
|
@ -128,6 +128,10 @@ public class DestinationFacade {
|
||||||
return new ModelAndView("redirect:" + (isQueue() ? "queues.jsp" : "topics.jsp"));
|
return new ModelAndView("redirect:" + (isQueue() ? "queues.jsp" : "topics.jsp"));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
protected ModelAndView redirectToDestinationView() {
|
||||||
|
return new ModelAndView("redirect:browse.jsp?JMSDestination=" + jmsDestination);
|
||||||
|
}
|
||||||
|
|
||||||
protected String getPhysicalDestinationName() {
|
protected String getPhysicalDestinationName() {
|
||||||
return createDestination().getPhysicalName();
|
return createDestination().getPhysicalName();
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue