can.Observe.List.Sort: fix for case when comparator is not a function #481

Merged
merged 1 commit into from Sep 24, 2013

Conversation

Projects
None yet
2 participants
@daniel-franz
Contributor

daniel-franz commented Sep 18, 2013

The test for can.Observe.List.Sort did not work. Sorting like

    this.comparator = 'name';

did not work as typeof(a===b) === 'boolean'

daffl added a commit that referenced this pull request Sep 24, 2013

Merge pull request #481 from daniel-franz/patch-9
can.Observe.List.Sort: fix for case when comparator is not a function

@daffl daffl merged commit 831383f into canjs:master Sep 24, 2013

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment