From dev-return-12738-apmail-activemq-dev-archive=activemq.apache.org@activemq.apache.org Tue Sep 30 10:28:15 2008 Return-Path: Delivered-To: apmail-activemq-dev-archive@www.apache.org Received: (qmail 76691 invoked from network); 30 Sep 2008 10:28:15 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 30 Sep 2008 10:28:15 -0000 Received: (qmail 90760 invoked by uid 500); 30 Sep 2008 10:28:13 -0000 Delivered-To: apmail-activemq-dev-archive@activemq.apache.org Received: (qmail 90558 invoked by uid 500); 30 Sep 2008 10:28:12 -0000 Mailing-List: contact dev-help@activemq.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@activemq.apache.org Delivered-To: mailing list dev@activemq.apache.org Received: (qmail 90544 invoked by uid 99); 30 Sep 2008 10:28:12 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 30 Sep 2008 03:28:12 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 30 Sep 2008 10:27:18 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id A563B234C1F4 for ; Tue, 30 Sep 2008 03:27:52 -0700 (PDT) Message-ID: <1186761059.1222770472676.JavaMail.jira@brutus> Date: Tue, 30 Sep 2008 03:27:52 -0700 (PDT) From: "Dejan Bosanac (JIRA)" To: dev@activemq.apache.org Subject: [jira] Resolved: (AMQ-1326) Move/Copy feature for web console In-Reply-To: <31006157.1184659906189.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/activemq/browse/AMQ-1326?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dejan Bosanac resolved AMQ-1326. -------------------------------- Assignee: Dejan Bosanac Resolution: Fixed Patch applied (with a few more code improvements) in SVN revision 700405 > Move/Copy feature for web console > --------------------------------- > > Key: AMQ-1326 > URL: https://issues.apache.org/activemq/browse/AMQ-1326 > Project: ActiveMQ > Issue Type: Improvement > Affects Versions: 5.0.0 > Reporter: Dejan Bosanac > Assignee: Dejan Bosanac > Fix For: 5.3.0 > > Attachments: activemq-core.patch, activemq-web-console.patch > > > I've added message copy and move funcionality to the web console. For now it is only applied to the message.jsp page, since I think it is not a good fit the browse page interface (we can add some bulk copy/move messages operations later). > I'm not sure what's your opinion on JS confirmation dialogs, but I generally think we should have some kind of user confirmation for operations such as delete, copy, move, etc (queues, topics, messages, ...) . I've added some JS code for these new operations and would provide additional confirmations for existing operations if it is OK with you. I've put all JS code in the common.js as it seemed the most appropriate place for it (maybe we could create a new file for this kind of JS code, but I think it is not necessary). > While working on this, I found a potential problem in activemq-core module. A patch for it is also attached to this issue. The bug scenario is as follows: > 1. create two queues > 2. send text message using the web console > 3. try to copy/move the message > the null pointer exception is thrown. > the same code used to copy the message in a test case works fine. > After investigating a bit, I found that problem is caused by the fact that message sent through web console has responseRequired property set to false and it is tried to be dispatched asynchronously using the context without a valid connection. I saw the code similar to the problematic is used elsewhere in the Queue.java class but only for expired messages, so I've added an additional condition. I'm not sure if it is valid patch, but it solved this particular problem. It would be good if someone could investigate this further. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.