What do you think about changing the implementation of
SymbolBindingFactory to let it check if the resulted symbol value is a
"binding expression" and if true let it resolve the binding and return
the actual binding instead of the symbol value ?
This will let you declare a symbol which is a binding expression and
have the binding resolved when the symbol is evaluated, for ex this
will let you specify the Block for the empty Grid in a Symbol like
"block:empty" and the Block binding will be resolved at runtime.
Is this something wanted ? ... I think it could be useful at least for
components writers.
Let me know what you think, here is the diff:
diff --git
a/tapestry-core/src/main/java/org/apache/tapestry5/internal/bindings/SymbolBindingFactory.java
b/tapestry-core/src/main/java/org/apache/tapestry5/internal/bindings/SymbolBindingFactory.java
index dce5a6b..b30786c 100644
---
a/tapestry-core/src/main/java/org/apache/tapestry5/internal/bindings/SymbolBindingFactory.java
+++
b/tapestry-core/src/main/java/org/apache/tapestry5/internal/bindings/SymbolBindingFactory.java
@@ -18,22 +18,32 @@ import org.apache.tapestry5.ComponentResources;
import org.apache.tapestry5.ioc.Location;
import org.apache.tapestry5.ioc.services.SymbolSource;
import org.apache.tapestry5.services.BindingFactory;
+import org.apache.tapestry5.services.BindingSource;
public class SymbolBindingFactory implements BindingFactory
{
+ private BindingSource bindingSource;
private SymbolSource symbolSource;
- public SymbolBindingFactory(SymbolSource symbolSource)
+ public SymbolBindingFactory(SymbolSource symbolSource,
BindingSource bindingSource)
{
+ this.bindingSource = bindingSource;
this.symbolSource = symbolSource;
}
public Binding newBinding(String description, ComponentResources container,
ComponentResources component, String expression, Location location)
{
-
String value = symbolSource.valueForSymbol(expression);
-
+
+ int colon = value.indexOf(":");
+
+ if (colon > 0)
+ {
+ return bindingSource.newBinding(description, container, component,
+ value.substring(0, colon), value.substring(colon
+ 1), location);
+ }
+
return new LiteralBinding(location, description, value);
}
Cheers
--
Massimo
http://meridio.blogspot.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
