Skip to content

Bloodhound: Replace getIntersection method. - #121

Closed
kevinkneifel wants to merge 1 commit into
corejavascript:masterfrom
kevinkneifel:master
Closed

kevinkneifel wants to merge 1 commit into
corejavascript:masterfrom
kevinkneifel:master

Conversation

@kevinkneifel

Copy link
Copy Markdown

Fixes functional error when matches array only has one key, and that key value is less than the first key value in the ids array, which results in an empty intersection array. Uses the same logic as the _.intersection(array) method from underscore.js to safely find the intersection of the two arrays.

…s only have one key, and that key value is less than the first key value in the ids array, which results in an empty intersection array.
@jlbooker jlbooker added this to the v1.2.0 milestone Feb 4, 2017
@jlbooker

jlbooker commented Feb 4, 2017

Copy link
Copy Markdown
Contributor

Refs #105

Darnokk added a commit to Darnokk/typeahead.js that referenced this pull request Mar 5, 2018
Darnokk added a commit to Darnokk/typeahead.js that referenced this pull request Mar 5, 2018
@jlbooker jlbooker closed this Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants