<%namespace:defname> converts the method name to lowercase

[sqlalchemy-bot]

Migrated issue, originally created by Anonymous

This raises a problem when the defname contains uppercase letters as the closing tag won't match the opening tag meaning you have to fall back to using call to call the method.

=== Example ===

<%def name="renderPanel(panel)">
    <div class="x-panel">
        <h1 class="x-panel-title">${panel.title}</h1>
        <div class="x-panel-content">
        </div>
    </div>
</%def>

<%def name="renderTablePanel(panel)">
    <%self:renderPanel panel="${panel}">
        ${self.renderTable(panel.table)}
    </%self>
</%def>

=== Error ===
SyntaxException: Closing tag does not match tag: <%self:renderpanel> in file 'panels.mao' at line: 46 char: 2

---

Attachments: fix-namespacedef.patch

[sqlalchemy-bot]

Anonymous wrote:

Missed out the caller.body in the example:

<%def name="renderPanel(panel)">
    <div class="x-panel">
        <h1 class="x-panel-title">${panel.title}</h1>
        <div class="x-panel-content">
            ${caller.body()}
        </div>
    </div>
</%def>

<%def name="renderTablePanel(panel)">
    <%self:renderPanel panel="${panel}">
        ${self.renderTable(panel.table)}
    </%self>
</%def>

[sqlalchemy-bot]

Anonymous wrote:

The attached patch appears to fix the error, I've no idea if the .lower() was only required just makes things case sensitive?

[sqlalchemy-bot]

Michael Bayer (@zzzeek) wrote:

weird, that lower() must have been there since the beginning. thanks for the patch, b06cd0f.

[sqlalchemy-bot]

Changes by Michael Bayer (@zzzeek):

• changed status to closed