Comments on: WicketWebBeans 1.0-rc2 Released

By: Dan Syrstad

Dan Syrstad — Wed, 05 Dec 2007 02:35:51 +0000

Jon:

Thank you! Both of these are really good suggestions. The latter one would be especially useful if you're using annotations or beanprops. I'll add them to the to-do list!

By: Jonathan Locke

Jonathan Locke — Tue, 04 Dec 2007 22:02:12 +0000

Also, why can’t I say just this:

12 add(new BeanForm(”beanForm”, new Customer()));

instead of this:

12 Customer bean = new Customer();
13 BeanMetaData meta = new BeanMetaData(bean.getClass(), null, this, null, false);
14 add( new BeanForm(”beanForm”, bean, meta) );

assuming I want the default metadata extraction.

By: Jonathan Locke

Jonathan Locke — Tue, 04 Dec 2007 21:57:47 +0000

One suggestion: simplify or refactor BeanMetaData constructions somehow. Why not use factory methods to create BeanMetaData? I think it might be a bit more self-documenting. This is not easily understood at a glance:

new BeanMetaData(bean.getClass(), null, this, null, false);

Something like this is easier to read, although arguably a bit less discoverable.

BeanMetaData.forBean(bean);
BeanMetaData.forChildBean(bean, VIEW_ONLY);