PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

Use _uniq instead of _sortedUniq for return unique matched results

Browse source 
_sortedUniq claims to be like _uniq but optimised for sorted arrays - https://lodash.com/docs/4.17.4#sortedUniqBy. But in practice this isn't quite true, so stick with the unoptimised version.
This commit is contained in:
Luke Barnard 2017-07-18 16:23:54 +01:00
parent ee5117525c
commit b185f43d3d
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/autocomplete/QueryMatcher.js
View file

@ -18,7 +18,7 @@ limitations under the License.
import _at from 'lodash/at'; import _at from 'lodash/at';
import _flatMap from 'lodash/flatMap'; import _flatMap from 'lodash/flatMap';
import _sortBy from 'lodash/sortBy'; import _sortBy from 'lodash/sortBy';
import _sortedUniq from 'lodash/sortedUniq'; import _uniq from 'lodash/uniq';
import _keys from 'lodash/keys'; import _keys from 'lodash/keys';
class KeyMap { class KeyMap {
@ -101,7 +101,7 @@ export default class QueryMatcher {
} }
}); });
return _sortedUniq(_flatMap(_sortBy(results, (candidate) => { return _uniq(_flatMap(_sortBy(results, (candidate) => {
return candidate.index; return candidate.index;
}).map((candidate) => { }).map((candidate) => {
// return an array of objects (those given to setObjects) that have the given // return an array of objects (those given to setObjects) that have the given