[jsr-314-open-mirror] [jsr-314-open] Fwd: Fix UIData state saving model

Leonardo Uribe lu4242 at gmail.com
Fri Aug 27 17:32:21 EDT 2010


Hi

The OCA was filled. I hope it will be no problem with share this
contribution
with myfaces.

LU> - On UIData.markInitialState do not use a facesContext attribute
> LU> "com.sun.faces.MARK_INITIAL_STATE", instead it could be better to use
> LU> a variable on FacesContext to indicate when we are marking the initial
> LU> state,
> LU> like FacesContext.isProcessingEvents() works.
>
> We can do that if you like, but it's just syntatic sugar IMHO.  I prefer
> to just introduce constants rather than methods.  Would you be ok if we
> made it an official "javax.faces.MARK_INITIAL_STATE" attribute?
>
>
Yes, it sounds good.

regards,

Leonardo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jsr-314-open-mirror/attachments/20100827/bf37ff30/attachment-0002.html 


More information about the jsr-314-open-mirror mailing list