Hello Andrew,

One comment regarding your changes in:

+public abstract class UIXDocumentTemplate
+  extends UIXComponentBase

Overall your fixes look fine to me, but I was wondering if rather than adding the override methods to the new class UIXDocument class, if instead this should be moved out of the UIXDocument and into a UIViewRoot subclass. For a different issue (of re-entrant calls to visitTree not properly suspending and restoring context) that I encountered, I had a similar fix in mind but after discussing with Blake he proposed that adding it to UIViewRoot subclass may be a better option.

Thanks
Pavitra

On 9/21/2010 9:43 AM, Andrew Robinson wrote:
Please see JIRA:

https://issues.apache.org/jira/browse/TRINIDAD-1919

I want to propose adding a set of APIs to Trinidad2 that will allow
component authors to suspend changes and resume changes before and
after an invokeOnComponent or visitTree call. Currently, component
context is not reverted during these calls, and bugs may result. The
proposed changes would allow components to tear down and set back up
their context over one of these calls.

I have attached a way to reproduce the issue and my proposed patch to
the bug. I'll wait a few days for feedback before going ahead and
committing it.

Thank you,
Andrew

Reply via email to