On Jan 19, 2008 3:33 AM, Andy Wardley <[EMAIL PROTECTED]> wrote:

> Hi Sean,
>
> > ok( $text =~ /"back is " . '#ffffff'/, 'col.back folded' );
>
> Oops!  Mea Culpa.
>
> > I think prototypes aren't generally a good idea, so I'd go for the
> latter
> > approach.
>
> Sounds good.
>
> > I'd like to
> > propose that generated code declare only lexical variables whose names
> > are prefixed with an underscore, that class of names being henceforth
> > reserved for use by the TT.
>
> Yep, that works for me.  Or perhaps use a _tt_ prefix to be extra clear
> that they're TT's lexical variables.
>

Thanks, Andy!  I've implemented your suggestion and checked in the fix, in
addition to the ok() issue and the few documentation warts I mentioned.

The only other issue I had noticed has to do with the HTML entity escaping
performed by the html filter and the HTML plugin.  Both replace the
double-quote character with the entity &quot;, but neither replaces the
apostrophe character with &apos;.  Perhaps they should?  It seems like a
potentially unpleasant surprise for those whose convention is to delimit
attributes with single quotes, eg:

<img src='...' alt='[% "Don't look!" | html %]'>


--Sean
_______________________________________________
templates mailing list
templates@template-toolkit.org
http://mail.template-toolkit.org/mailman/listinfo/templates

Reply via email to