Re: [jsr-314-open-mirror] [jsr-314-open] Fwd: Fix UIData state saving model
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
Attachments:
- attachment.html (text/html — 932 bytes)