Merge pull request #276 from elderhsouza/patch-9

fixed an issue with default parameters
This commit is contained in:
Angelos Chalaris
2017-12-20 16:10:12 +02:00
committed by GitHub

View File

@ -10,7 +10,7 @@ const orderBy = (arr, props, orders) =>
arr.sort((a, b) =>
props.reduce((acc, prop, i) => {
if (acc === 0) {
const [p1, p2] = orders[i] === 'asc' ? [a[prop], b[prop]] : [b[prop], a[prop]];
const [p1, p2] = orders && orders[i] === 'desc' ? [b[prop], a[prop]] : [a[prop], b[prop]];
acc = p1 > p2 ? 1 : p1 < p2 ? -1 : 0;
}
return acc;