Return-Path: <tcurdt@dff.st>
Mailing-List: contact cocoon-dev-help@xml.apache.org; run by ezmlm
Delivered-To: mailing list cocoon-dev@xml.apache.org
Received: (qmail 57912 invoked from network); 27 Dec 2000 10:20:51 -0000
Received: from pns.dff.st (HELO mail.dff.st) (62.153.127.51)
  by h29.sny.collab.net with SMTP; 27 Dec 2000 10:20:51 -0000
Received: from mail.dff.local ([172.16.1.10])
	by mail.dff.st with esmtp (Exim 3.14 #3)
	id 14BEZg-0005R5-00
	for cocoon-dev@xml.apache.org; Wed, 27 Dec 2000 12:16:52 +0100
Received: from shodan.dff.local ([172.16.2.1] helo=shodan)
	by mail.dff.local with smtp (Exim 3.16 #3)
	id 14BDh3-0004KC-00
	for cocoon-dev@xml.apache.org; Wed, 27 Dec 2000 11:20:25 +0100
From: "Torsten Curdt" <tcurdt@dff.st>
To: <cocoon-dev@xml.apache.org>
Subject: RE: xsp attribute bug?
Date: Wed, 27 Dec 2000 11:20:24 +0100
Message-ID: <CHEGLKNOIJHFNDNBCIAJIEDOCAAA.tcurdt@dff.st>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
X-Priority: 3 (Normal)
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook IMO, Build 9.0.2416 (9.0.2910.0)
X-MimeOLE: Produced By Microsoft MimeOLE V5.50.4133.2400
Importance: Normal
In-Reply-To: <005901c06db3$24af0f80$3200a8c0@calvary.org>
X-Spam-Rating: h29.sny.collab.net 1.6.2 0/1000/N

Cool! But there still seems to be a problem. The "addAttribute"
gets now called at the right place but there seems to be a
problem with it's content!

  <test>
    <xsp:attribute name="color">blue</xsp:attribute>
  </test>

gives me...

  xspAttr.addAttribute("", "color", "color", "CDATA", "");

this means: in the xsp:attribute template within the content variable
the for-each selects no text nodes. I can't see why...

  <test>
    <xsp:attribute
name="color"><xsp:expr>testString</xsp:expr></xsp:attribute>
  </test>

If I have xsp:expr inside the xsp:attribute it works fine.

> This should now be fixed.  I ran into that last week, made the fix on
> Thursday or Friday.
> Get new CVS and see if it is still a problem.

I just upgraded my xsp.xsl in my CVS version from about 10 days ago.
--
Torsten