jakartaee / faces

Jakarta Faces
Other
109 stars 55 forks source link

@jakarta.faces.annotation.View#value must be @Nonbinding #1865

Closed BalusC closed 1 year ago

BalusC commented 1 year ago

Discovered during https://github.com/eclipse-ee4j/mojarra/issues/5362

The @jakarta.faces.annotation.View#value must be @Nonbinding.

volosied commented 1 year ago

Commented in the Mojarra repo by accident. See my post there: https://github.com/eclipse-ee4j/mojarra/pull/5368#issuecomment-1832305496

The /hello-facelet mapping works, but /hello-facelet.xhtml fails due to ambiguous beans.

Should we make this Nonbinding? The Any Literal Annotation may enough for 4.1 & 5.0 to avoid regressions.

BalusC commented 1 year ago

Reverted: https://github.com/eclipse-ee4j/mojarra/commit/12ca56e2838dece42dfef3bfaa50914a1355ce24