cocoon-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Donald Ball <ba...@webslingerZ.com>
Subject Re: [C1.8.2] sendmail.xsl update
Date Tue, 03 Apr 2001 18:47:42 GMT
On Tue, 3 Apr 2001, Drasko Kokic wrote:

> here is a proposal for the extended sendmail.xsl
>
> The following elements are being add:
> sendmail:reply-to
> sendmail:to
> sendmail:cc
> sendmail:bcc
> These elements could also apear more than once
>
> For the debuging purposes there is a new
> sendmail:debug element

overall, i like this patch. if i may make a few requests before i import
this:

1. please prefix all variables with _sendmail_. i don't think was done
consistently in the existing cvs source, but it should be.

2. i believe you should use <> instead of the entities in attribute values
like this:

        <xsl:if test="count(sendmail:reply-to) &gt; 0">

3. please add yourself to the author list at the top of the file. :)

4. send a unified context diff

thanks!

- donald


---------------------------------------------------------------------
Please check that your question has not already been answered in the
FAQ before posting. <http://xml.apache.org/cocoon/faqs.html>

To unsubscribe, e-mail: <cocoon-users-unsubscribe@xml.apache.org>
For additional commands, e-mail: <cocoon-users-help@xml.apache.org>


Mime
View raw message