<br><br><div class="gmail_quote">2010/8/25 Ed Burns <span dir="ltr"><<a href="mailto:edward.burns@oracle.com">edward.burns@oracle.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">>>>>> On Wed, 25 Aug 2010 13:52:46 -0700, Ed Burns <<a href="mailto:edward.burns@oracle.com">edward.burns@oracle.com</a>> said:<br>
<br>
>>>>> On Tue, 17 Aug 2010 16:02:01 -0500, Leonardo Uribe <<a href="mailto:lu4242@gmail.com">lu4242@gmail.com</a>> said:<br>
LU> Hi<br>
LU> 2010/8/13 Ed Burns <<a href="mailto:edward.burns@oracle.com">edward.burns@oracle.com</a>><br>
<br>
EB> Here is my response.<br>
<br>
LU> Note by default this feature is not active, so I think to test it fully it<br>
LU> is necessary to activate it setting<br>
LU> preserveRowComponentState to true.<br>
<br>
</div>EB> I have executed your very excellent testcase app. When I traverse the<br>
EB> app with the markInitialState variant of your fix in place, the fields<br>
EB> are red that should be red. When I remove the fix and re-traverse the<br>
EB> app, all of the fields are red.<br>
<br>
EB> I'm experimenting with having preserveRowComponentState to true by<br>
EB> default. What do you think of that?<br>
<br>
Well, that didn't work so well. As you'd expect, this failed pretty<br>
quickly because we had an existing test that exercised the existing<br>
behavior.<br>
<br>
Is everyone OK with having this be configured as a UIData property, or<br>
would it be better done using a context-param?<br>
<div><div></div><div class="h5"><br></div></div></blockquote><div><br>The problem with allow it with a context-param is there is no way to "reset" the deltas<br>or remove rows to synchronize the model with the component state.<br>
<br>Suppose a simple use case, where there is a datatable and an option that removes one<br>row. Since we don't have a way to notify the component that the row has been removed<br>the state of the deleted row will be applied on the next one.<br>
<br>It is worth to mention that tomahawk HtmlDataTableHack has a method to deal with this<br>specific use case called:<br><br> public void deleteRowStateForRow(int deletedIndex)<br><br>Maybe it is not the best solution but works. In theory, a publisher-suscriber pattern could<br>
do the job better in this case.<br><br></div></div>Leonardo<br>