helma.org/wiki/Helma 1.5 skin rendering is broken

Version 7

hannes — Sat, 31 Mar 2007 09:04:32 +0200

18
19	[Update 2007/03/31] Actually, I'm not really sure about the first item (the $handler syntax). But since I have reframed the problem (from dealing with yet another conflict case to simply avoiding it by sprinkling some after-processing magic dust) I have stopped worrying. I think it's just a matter of hours/days until some really right solution pops up, maybe involving JavaScript instead of Java.

Version 6

hannes — Fri, 30 Mar 2007 17:46:57 +0200

* Add a simple syntax that allows to reference parameter handlers in macro tags: If a parameter value (or macro name?) starts with a "$", Helma will try to convert it to the value of the corresponding res.handlers property before passing it to the macro. For example, <code><% list page=$page %></code> will result in the list macro being invoked with res.handlers.page as argument, rather than the string "$page". With this we can disambiguify global macros from handler references.

Version 5

hannes — Fri, 30 Mar 2007 17:46:19 +0200

9	What all this boils down to is to simplify macro handler lookup in Helma 1.6 a fair bit,=== dropping the path lookup, dropping maybe the global lookup but in any case allowing objects in res.handlers to be acessed directly in macros, and providing a switch for Helma 1.5-compatible skins to disable all this and doing it the old way. What do you think?Proposed Changes ===
10
11	[Update 2007/03/30] Here are the measures I came up with after a few days of rumination:
12
13	* Add a simple syntax that allows to reference parameter handlers in macro tags: If a parameter value starts with a "$", Helma will try to convert it to the value of the corresponding res.handlers property before passing it to the macro. For example, <code><% list page=$page %></code> will result in the list macro being invoked with res.handlers.page as argument, rather than the string "$page". With this we can disambiguify global macros from handler references.
14	* Introduce an app property to disable handler lookup in the parent path of the this-object. The goal is to enable a setup where res.handlers is the only place for macro handlers.
15	* Introduce an app property that allows to define namespaces for global macros and filters. The default will be null, i.e. global macros will be looked up in the global scope, while tidiness-concious apps will put global macros and filters into namespaces like helma.macros and helma.filters. It may be possible/disirable to define multiple namespaces to form a search path form global macros.
16
17	These changes would solve all problems above, except the param->res.handlers feature, which anyway is easier done in JavaScript when implementing a renderPart()/render() as proposed by Tobi\|Handler for rendered skins.

Version 4

hannes — Sun, 25 Mar 2007 14:04:14 +0200

What all this boils down to is to simplify macro handler lookup in Helma 1.6 a fair bit, dropping the path lookup, dropping maybe the global lookup but allowing but in any case allowing objects in res.handlers to be acessed directly in macros, and providing a switch for Helma 1.5-compatible skins to disable all this and doing it the old way. What do you think?

Version 3

hannes — Sun, 25 Mar 2007 14:02:40 +0200

* Macro handler lookup: handlers are looked for in the path of the HopObject the skin is being rendered on first, and only then in res.handlers. The reason for this is that at one time, it was impossible to register handlers at all (before res.handlers, the request path was used for handler lookup), so it would have been impossibleimpossible to address anything outside/beyond the request path.

Version 2

hannes — Sun, 25 Mar 2007 14:01:44 +0200

* Finally, the i think the param argument to the renderSkin* methods had been deprived of its usefulness as describe in *this comment|<a href="http://dev.helma.org/wiki/Handler+for+rendered+skins/#$cmt1669*">this comment</a>. I think the param argument would be most useful if it simply was another way of directly registering stuff in res.handlers.

Version 1

hannes — Sun, 25 Mar 2007 14:00:52 +0200

Set tags to helma 1.6, Skin Rendering, templates

1	The more I think about it, the more I find that skin rendering in Helma 1.5 is quite broken, and we need to make a (minor) step in Helma 1.6 to get rid of the worst of it.
3	Here's what I found so far:
4
5	* Macro handler lookup: handlers are looked for in the path of the HopObject the skin is being rendered on first, and only then in res.handlers. The reason for this is that at one time, it was impossible to register handlers at all (before res.handlers, the request path was used for handler lookup), so it would have been impossible.
6	* Global and non-global macros: In the old days, global and non-global macros could simply be kept apart by looking at whether the macro name contained a dot character. This makes less and less sense to me, since global macros may be packed into namespaces as well, and with filters and nested macros, it is quite conceivable that people might want to address handlers directly as macro return values (e.g. to send it down a filter chain, or to send it as an argument to another macro).
7	* Finally, the i think the param argument to the renderSkin* methods had been deprived of its usefulness as describe in this comment\|http://dev.helma.org/wiki/Handler+for+rendered+skins/#$cmt1669. I think the param argument would be most useful if it simply was another way of directly registering stuff in res.handlers.
8
9	What all this boils down to is to simplify macro handler lookup in Helma 1.6 a fair bit, dropping the path lookup, dropping maybe the global lookup but allowing but in any case allowing objects in res.handlers to be acessed directly in macros, and providing a switch for Helma 1.5-compatible skins to disable all this and doing it the old way. What do you think?