Edgewall Software

Opened 8 years ago

Last modified 5 weeks ago

#321 new enhancement

(Wishlist) Support for outputting comments

Reported by: daniel-genshi@… Owned by: cmlenz
Priority: minor Milestone: 0.9
Component: Template processing Version: 0.5.1
Keywords: comment Cc:


It would be nice to have the ability to output comments in XML templates, much like you can in XSL with <xsl:comment>.

This would allow actual template data to be put in comments at the author's discretion.

Using an element like <py:comment> seems like the right way to approach this.

Attachments (2)

pycomments.patch (4.8 KB) - added by daniel-genshi@… 8 years ago.
patch for py:comment feature
pycomment1.patch (4.9 KB) - added by daniel-genshi@… 7 years ago.
patch refresh

Download all attachments as: .zip

Change History (11)

comment:1 Changed 8 years ago by anonymous

use may use "<--! this is a comment -->" construction. http://genshi.edgewall.org/wiki/Documentation/0.5.x/xml-templates.html#id11

comment:2 Changed 8 years ago by daniel-genshi@…

I know about that, and it's not what I'm describing. Here's a code example:

<py:comment>${1 + 1}</py:comment>

should yield:


Notice that what is inside the comment is actually processed.

Changed 8 years ago by daniel-genshi@…

patch for py:comment feature

comment:3 Changed 8 years ago by daniel-genshi@…

I added a patch that creates a comment directive and should close this ticket. I included a set of tests in the patch as well. Please review at your convenience.

Changed 7 years ago by daniel-genshi@…

patch refresh

comment:4 Changed 7 years ago by cmlenz

  • Milestone changed from 0.6 to 0.7

comment:5 Changed 7 years ago by daniel-genshi@…

I noticed the milestone was changed to 0.7. Are you just paring back features to stabilize the release? Is the change unwelcome? If there is something wrong with the patch, please let me know.

comment:6 Changed 7 years ago by cmlenz

I just need to get a new stable release (0.6) out real soon, so a lot of tickets have been moved to the next major milestone.

The patch looks good.

comment:7 Changed 7 years ago by Carsten Klein <carsten.klein@…>

see also the initiative to support pluggable directives libraries in WorkInProgress/PluggableDirectivesLibraries and ticket #395 thereof.

comment:8 Changed 3 years ago by spaceone

The patch still applies and is working.

But i found a issue where it causes the process to not generate the template anymore (process status is R+, strace shows mremap(0xb6be5000, 1908736, 2146304, MREMAP_MAYMOVE) = 0xb6be5000).

from genshi.template import MarkupTemplate
s = '''<html xmlns="http://www.w3.org/1999/xhtml" xmlns:py="http://genshi.edgewall.org/">
<?python from genshi import HTML ?>

comment:9 Changed 5 weeks ago by hodgestar

  • Milestone changed from 0.7 to 0.9

Moved to milestone 0.9.

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
as new The owner will remain cmlenz.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from cmlenz to the specified user. Next status will be 'new'.
The owner will be changed from cmlenz to anonymous. Next status will be 'assigned'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.