Initially I was not sure (anymore), but after reading Phil's comment I also think model.equals(model) is better. I also think the identity definition (wheter through equals or string uniquekey) belongs to the data side, because you can only do this with knowledge about the data. And I belief that it can save you some typing if you have it all in one place.

I strongly agree that there should be a property on DataView to turn OIR on or off.

BTW: Does DataView need to call IDataProvider.size(). Wouldn't it be enough if it just called iterator() with viewSize. The iterator just returns than what is there. I have quite some views (unfortunately orderitems) where I do not need paging and can just get all of them with one select .

Christian

On Mon, 15 Aug 2005 01:00:38 -0700, Phil Kulak <[EMAIL PROTECTED]> wrote:

Won't it be comparing with the models from the last request? You know,
those models that were detached and may be discarded, never to be
attached again save that comparison?

Another problem is that I don't want the DataView using value equality
when row equality is available, which it always will be in this case.
I think if you want to make it more general, use the model's equals so
that it can use value, row, or object identity. I don't think that
requiring a proper implementation of equals on the model is any
different from requiring it on the object and it gives you more
options.


        

        
                
___________________________________________________________ Gesendet von Yahoo! Mail - Jetzt mit 1GB Speicher kostenlos - Hier anmelden: http://mail.yahoo.de



-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO
September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf
_______________________________________________
Wicket-user mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-user

Reply via email to