Forum OpenACS Development: variables in attributes for adp tags.

Collapse
Posted by Jeff Davis on
Bart Teeuwisse has a patch which replaces @foo@ with the value of $foo in template::get_attrtibute

I like the change but have a few issues with the implementation:

  • it should not walk up the stack looking for a value.
  • it should let you do things like attr="@value1@/@value2@" (it currently only works for attr="@value@")
i.e. I think it should work the same as @var@ etc in the rest of the .adp

otoh, its better than nothing and it has always been annoying that you can't use @vars in adp tag attributes.

I am curious what other people think, also do you think this change should be TIPed.