Return-Path: Delivered-To: apmail-cocoon-dev-archive@cocoon.apache.org Received: (qmail 36476 invoked by uid 500); 28 Jul 2003 17:21:02 -0000 Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: dev@cocoon.apache.org Delivered-To: mailing list dev@cocoon.apache.org Received: (qmail 36381 invoked from network); 28 Jul 2003 17:21:00 -0000 Received: from exchange.sun.com (192.18.33.10) by daedalus.apache.org with SMTP; 28 Jul 2003 17:21:00 -0000 Received: (qmail 1013 invoked by uid 50); 28 Jul 2003 17:23:38 -0000 Date: 28 Jul 2003 17:23:38 -0000 Message-ID: <20030728172338.1012.qmail@nagoya.betaversion.org> From: bugzilla@apache.org To: dev@cocoon.apache.org Cc: Subject: DO NOT REPLY [Bug 21609] - [PATCH] NaN error in XSP version of the calc flow example X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT . ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21609 [PATCH] NaN error in XSP version of the calc flow example ------- Additional Comments From codonovan@virtuallyuseful.org 2003-07-28 17:23 ------- Sorry my mistake. It doesn't look like this is still necessary. I misread the code. I believe cocoon.request.name doesn't work because 'name' is a variable not the name of a request parameter. So the patch should most likely be EITHER - return parseFloat(cocoon.request.name); + return parseFloat(cocoon.request.getParameter(name)); OR - return parseFloat(cocoon.request.name); + return parseFloat(eval("cocoon.request." + name));