OK, saw your changes.

Looks like you've found a way to detect that someone set sort=null without
calling refresh, which is good, but it seems like the ListCollectionView
is in an unsupported state at this point.

I'm just worried that there are other errors waiting to be found when in
this state.  ListCollectionView is used as both an Ilist and an
ICollectionView and there are cases where the underlying list/source
changes, etc.  Seems like the safer route would be to encourage the
developer to call refresh(). Did you consider throwing an error saying you
should call refresh() instead?


-Alex


On 5/15/13 10:24 PM, "Justin Mclean" <jus...@classsoftware.com> wrote:

>Hi,
>
>> So is your goal to restore Flex 2 behavior?
>No of course not.
>
>I was just pointing it out why some people did expect that sort = null
>should behave this way. It's not entirely unreasonable to expect that
>setting sort to null without a refresh would remove the sort but let keep
>it so a refresh is needed and see if we can stop the RTEs as well without
>any performance or other side effects. Almost have a solution that I
>believe works in all cases.
>
>> I'd rather just make sure that there is no undesirable behavior if you
>>do
>> call refresh() after setting sort=null.
>Yes I'm checking that as well.
>
>Thanks,
>Justin

Reply via email to