On Mon, 2006-01-23 at 15:33 +0100, Mathias Stein wrote: > What t:dataTable really does at the moment, is: > 1) In rendering phase getAllItems() is called the first time. > 2) If preserveDataModel is enabled, t:dataTable calls getAllItems() > !!!one more time!!! to save the data model. [snip] > But if the dataModel changes between step 1 and 2, the previous > rendering phase still uses the old version, while t:dataTable stores and > restores already the changed one. So preserveDataModel only reduces the > risk. The problem is shifted between step 1 and 2. > > I think step 1 and 2 should call getAllItems() only once per request, > and they should use exactly the same data model for rendering and for > storing. > > Please correct me, if I'm wrong.
No, if things do indeed work as you describe then I agree that would be a bug. Please create a JIRA issue for this. Of course I have no idea how easy it would be to fix that issue... Regards, Simon
