vaadin / cdi

CDI Integration for Vaadin
Apache License 2.0
41 stars 56 forks source link

Lack of @Root #17

Open AdamBien opened 11 years ago

AdamBien commented 11 years ago

The @Root annotation was removed. It means: any @CDIUI annotation with an empty value marks a root. With that you can very efficiently declare a single root (it actually happens only once), but we are loosing Convention over Configuration capabiltities: previously the URI was automatically derived from the CDIUI#getClass().getSimpleName().

Now you will have to declare the URI for all UIs. The behavior is actually not consistent with the JavaEE 6 API (EJBs, CDI, JPA).

Proposal: revival of the @Root annotation.

ManuelB commented 11 years ago

I agree to this opinion. Without the @Root annotation every other UI will have to specify an own URL. It is a lot more convenient to just use the lowercase of the class name as default URL.