Set and Map will also need this change then. On Tue, Jun 9, 2020 at 5:16 PM Jeanette Winzenburg <[email protected]> wrote:
> > Zitat von José Pereda <[email protected]>: > > > Thanks, that's helpful. > > > > If I get it right, the change applied in ListExpressionHelper [1][2] > > removed equals: > > > > - if ((currentValue == null)? (oldValue != null) : > > !currentValue.equals(oldValue)) { > > + if (currentValue != oldValue) { > > > > So that will imply that we should do the same in ListPropertyBase::bind? > > > > exactly :) IMO, at least - otherwise we get all the old problems back: > the binding (just the same as listing) must be to an exact instance, > not to something that's equal but another instance. > >
