Personally, I expected the Compound/BoundCompoundPropertyModels to behave excactly the same, as they solve the same problem/ do the same magic (operate with Ognl on a 'target' object). Conceptually, the only difference between those two classes and PropertyModel is that the Compound models share the model.

Furthermore, for components like TextField with a parameter like:
* @param type The type to use when updating the model for this text field


users will expect any model updating to use conversion.

Anyway, I do think converters should at least work on the compound models as they work on property models, and we should consider whether conversion should take place for components that have a 'type' parameter set.

Eelco



Jonathan Locke wrote:


not quite a bug because it was designed this way, but if you were expecting converters to work magically, with compoundpropertymodels, this is a design misfeature.


the easy workaround is to use the bound compound model, which lets you specify propertyType

best,

   jon

Eelco Hillenius wrote:

Not sure yet if it is your bug as well, but at least we're on to something.

Eelco

Gili wrote:


Good catch! I'll wait on the bugfix, hopefully it'll solve the problem.


Gili

Eelco Hillenius wrote:

Yeah, found a problem when I stepped all the way through...

In CompoundPropertyModel:

   protected Class propertyType(final Component component)
   {
       return null;
   }

This can't be right...

Unfortunately, I'm out of time here. But I'll file a bug, and check in my examples code.

To be continued...

Eelco






-------------------------------------------------------
This SF.Net email is sponsored by: New Crystal Reports XI.
Version 11 adds new functionality designed to reduce time involved in
creating, integrating, and deploying reporting solutions. Free runtime info,
new features, or free trial, at: http://www.businessobjects.com/devxi/728
_______________________________________________
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user




-------------------------------------------------------
This SF.Net email is sponsored by: New Crystal Reports XI.
Version 11 adds new functionality designed to reduce time involved in
creating, integrating, and deploying reporting solutions. Free runtime info,
new features, or free trial, at: http://www.businessobjects.com/devxi/728
_______________________________________________
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user




-------------------------------------------------------
This SF.Net email is sponsored by: New Crystal Reports XI.
Version 11 adds new functionality designed to reduce time involved in
creating, integrating, and deploying reporting solutions. Free runtime info,
new features, or free trial, at: http://www.businessobjects.com/devxi/728
_______________________________________________
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

Reply via email to