tal:omit-tag should be processed before tal:attributes

[Wichert Akkerman]

Currently TAL handles tal:attributes before tal:omit-tag. This has two problems: it is a needless performance penalty (omit-tag will cause the work done by attributes to be removed), and it can be problematic in situations where you want to omit the tag if it is not possible to set an attribute on it for some reason. This issue was also noticed by Tobias in 2008 (see the comment at the bottom of http://wiki.zope.org/ZPT/TALSpecification14#oop )

[Wichert Akkerman (wichert)]

For refererence: chameleon.zpt processes tal:omit-tag before tal:attributes, which has not revealed any problems.

[Fred Drake (fdrake)]

It doesn't help that the spec is inconsistent: The paragraph starting "The reasoning behind this ordering" states that attributes is processed last.

A new version of the spec (1.4.1) should be drafted by copying the current spec and clarifying this, and the implementation should be made to conform.

The current specification is in the ZPT wiki at http://wiki.zope.org/ZPT/TALSpecification14.

[Fred Drake (fdrake)]

Tres: Please justify the "Won't Fix".

I'd actually rather see it fixed.

[Tres Seaver (tseaver)]

This is "WONTFIX" for the Zope3 project, because we aren't going to be making any new releases of Zope3. Note that I made the bug relevant for the 'zope.tal' project, which is under the 'zopetoolkit' umbrella. This is the right home for the bug, because any fix for the bug is going to involve releasing a new version of zope.tal.

[Fred Drake (fdrake)]

Gotcha; thanks.

/me is easily confused by Launchpad.

[Colin Watson (cjwatson)]

The zope.tal project on Launchpad has been archived at the request of the Zope developers (see https://answers.launchpad.net/launchpad/+question/683589 and https://answers.launchpad.net/launchpad/+question/685285). If this bug is still relevant, please refile it at https://github.com/zopefoundation/zope.tal.

[Colin Watson (cjwatson)]

The zope2 project on Launchpad has been archived at the request of the Zope developers (see https://answers.launchpad.net/launchpad/+question/683589 and https://answers.launchpad.net/launchpad/+question/685285). If this bug is still relevant, please refile it at https://github.com/zopefoundation/zope2.