Return-Path: Delivered-To: apmail-activemq-dev-archive@www.apache.org Received: (qmail 30463 invoked from network); 13 Jun 2008 14:36:52 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 13 Jun 2008 14:36:52 -0000 Received: (qmail 24954 invoked by uid 500); 13 Jun 2008 14:36:53 -0000 Delivered-To: apmail-activemq-dev-archive@activemq.apache.org Received: (qmail 24932 invoked by uid 500); 13 Jun 2008 14:36:53 -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 24921 invoked by uid 99); 13 Jun 2008 14:36:53 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 13 Jun 2008 07:36:53 -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; Fri, 13 Jun 2008 14:36:12 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 9140F234C140 for ; Fri, 13 Jun 2008 07:36:00 -0700 (PDT) Message-ID: <1017758022.1213367760593.JavaMail.jira@brutus> Date: Fri, 13 Jun 2008 07:36:00 -0700 (PDT) From: "Dan Schooler (JIRA)" To: dev@activemq.apache.org Subject: [jira] Commented: (AMQ-1377) amq.js should not have a dependency on prototype.js In-Reply-To: <19573635.1188914423050.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-1377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=43456#action_43456 ] Dan Schooler commented on AMQ-1377: ----------------------------------- Jim's modified files require either the amq_jquery_adapter or the amq_prototype_adapter, which in turn require jquery or prototype, respectively. The include order of the .js files is also important. You must first include prototype or jquery first, then include the respective adapter, then include the modified amq.js If you eliminate the amq_prototype_adapter.js completely then amq.js will definitely not work because it requires one of the adapters to use the appropriate AJAX library. I would suggest that you install the Firebug plugin in your Firefox web browser, set some breakpoints in the javascript code and step through it to see what is happening. > amq.js should not have a dependency on prototype.js > --------------------------------------------------- > > Key: AMQ-1377 > URL: https://issues.apache.org/activemq/browse/AMQ-1377 > Project: ActiveMQ > Issue Type: Improvement > Components: Broker > Affects Versions: 5.0.0 > Reporter: Jim Cook > Attachments: amq.js, amq.js, amq.js, amq_js.zip > > > amq.js uses a script loading technique to include behavior.js, prototype.js and _amq.js. The Behavior library is included only for the purposes of the chat client and is only GUI in nature, so that should go. Also, Prototype is included for its AJAX functionality, however everyone has their favorite javascript library and although I cut my teeth on prototype, I use jQuery now. I certainly don't want amq.js to include prototype when I have a perfectly good AJAX library already included in jQuery. > IMHO, amq.js should include only the functionality of client-side javascript-enabled JMS. The page author should include her AJAX library of choice (prototype, dojo, jquery, mootools, etc.) and an adapter class that allows amq.js to use the particular ajax library. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.