Moved to lodash 3.10.1 (to align with Kibana) (elastic#359)

- created a `predicate` module that exports all predicate function (e.g. `isString`, `isFunction`,...)
- created `isNil` predicate
- `sortable_properties` no longer uses lodash `sortBy` and uses our comparators instaed

this change should enable using tables in Kibana.

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 No public interface changes since `0.0.16`.
 
+**Bug fixes**
+
+- downgraded `lodash` version to `3.10.0` to align it with Kibana  
+
 # [`0.0.16`](https://github.com/elastic/eui/tree/v0.0.16)
 
 - `EuiRadio` now supports the `input` tag's `name` attribute. `EuiRadioGroup` accepts a `name` prop that will propagate to its `EuiRadio`s. ([#348](https://github.com/elastic/eui/pull/348))

package.json

@@ -30,7 +30,7 @@
     "html": "^1.0.0",
     "jquery": "^3.2.1",
     "keymirror": "^0.1.1",
-    "lodash": "^4.17.4",
+    "lodash": "^3.10.1",
     "numeral": "^2.0.6",
     "prop-types": "^15.6.0",
     "react-ace": "^5.5.0",
@@ -75,7 +75,6 @@
     "html-webpack-plugin": "^2.30.1",
     "jest": "^22.0.6",
     "jest-cli": "^22.0.6",
-    "lodash": "^4.17.4",
     "moment": "2.13.0",
     "node-sass": "^4.5.3",
     "npm-run": "^4.1.2",

src/components/table_of_records/default_record_action.js

@@ -1,5 +1,5 @@
 import React from 'react';
-import { isString } from 'lodash';
+import { isString } from '../../services/predicate';
 import { EuiButton, EuiButtonIcon } from '../button';
 
 const defaults = {

src/components/table_of_records/table_of_records.js

@@ -1,5 +1,6 @@
 import React from 'react';
 import _ from 'lodash';
+import { isString } from '../../services/predicate';
 import classNames from 'classnames';
 import PropTypes from 'prop-types';
 import {
@@ -162,7 +163,7 @@ export class EuiTableOfRecords extends React.Component {
 
   recordId(record) {
     const id = this.props.config.recordId;
-    return _.isString(id) ? record[id] : id(record);
+    return isString(id) ? record[id] : id(record);
   }
 
   changeSelection(selection) {

src/services/format/format_auto.js

@@ -1,4 +1,4 @@
-import { isArray, isBoolean, isDate, isNaN, isNil, isNumber, isString } from 'lodash';
+import { isNil, isArray, isBoolean, isDate, isNaN, isNumber, isString } from '../predicate';
 import { formatBoolean } from './format_boolean';
 import { formatDate } from './format_date';
 import { formatNumber } from './format_number';

src/services/format/format_boolean.js

@@ -1,4 +1,4 @@
-import { isNil } from 'lodash';
+import { isNil } from '../predicate';
 
 export const formatBoolean = (value, { yes = 'Yes', no = 'No', nil = '' } = {}) => {
   if (isNil(value)) {

src/services/format/format_date.js

@@ -1,4 +1,4 @@
-import { isNil, isFunction, isString } from 'lodash';
+import { isNil, isFunction, isString } from '../predicate';
 import moment from 'moment';
 
 const calendar = (value, options = {}) => {

src/services/format/format_number.js

@@ -1,5 +1,5 @@
 import numeral from 'numeral';
-import { isNil, isString } from 'lodash';
+import { isNil, isString } from '../predicate';
 
 const numberFormatAliases = {
   decimal1: '0,0.0',

src/services/format/format_text.js

@@ -1,4 +1,4 @@
-import { isNil } from 'lodash';
+import { isNil } from '../predicate';
 
 export const formatText = (value, { nil = '' } = {}) => {
   return isNil(value) ? nil : value.toString();

src/services/predicate/common_predicates.js

@@ -0,0 +1,15 @@
+export const always = () => true;
+
+export const never = () => false;
+
+export const isUndefined = (value) => {
+  return value === undefined;
+};
+
+export const isNull = (value) => {
+  return value === null;
+};
+
+export const isNil = (value) => {
+  return isUndefined(value) || isNull(value);
+};

src/services/predicate/common_predicates.test.js

@@ -0,0 +1,44 @@
+import { always, never, isNil, isUndefined, isNull } from './common_predicates';
+
+describe('common predicates', () => {
+
+  test('always', () => {
+    [ undefined, null, 'a', 1, true, false, Date.now(), {}, [], /.*/ ].forEach(value => {
+      expect(always(value)).toBe(true);
+    });
+  });
+
+  test('never', () => {
+    [ undefined, null, 'a', 1, true, false, Date.now(), {}, [], /.*/ ].forEach(value => {
+      expect(never(value)).toBe(false);
+    });
+  });
+
+  test('isUndefined', () => {
+    [ undefined ].forEach(value => {
+      expect(isUndefined(value)).toBe(true);
+    });
+    [ null, 'a', 1, true, false, Date.now(), {}, [], /.*/ ].forEach(value => {
+      expect(isUndefined(value)).toBe(false);
+    });
+  });
+
+  test('isNull', () => {
+    [ null ].forEach(value => {
+      expect(isNull(value)).toBe(true);
+    });
+    [ undefined, 'a', 1, true, false, Date.now(), {}, [], /.*/ ].forEach(value => {
+      expect(isNull(value)).toBe(false);
+    });
+  });
+
+  test('isNil', () => {
+    [ undefined, null ].forEach(value => {
+      expect(isNil(value)).toBe(true);
+    });
+    [ 'a', 1, true, false, Date.now(), {}, [], /.*/ ].forEach(value => {
+      expect(isNil(value)).toBe(false);
+    });
+  });
+
+});

src/services/predicate/index.js

@@ -0,0 +1,2 @@
+export * from './common_predicates';
+export * from './lodash_predicates';

src/services/predicate/lodash_predicates.js

@@ -0,0 +1,9 @@
+export {
+  isFunction,
+  isArray,
+  isString,
+  isBoolean,
+  isDate,
+  isNumber,
+  isNaN,
+} from 'lodash';

src/services/random.js

@@ -1,4 +1,4 @@
-import { isNil } from 'lodash';
+import { isNil } from './predicate';
 
 const defaultRand = Math.random;
 

src/services/sort/comparators.js

@@ -16,13 +16,17 @@ export const Comparators = Object.freeze({
     return (v1, v2) => comparator(v2, v1);
   },
 
-  property(prop, comparator = undefined) {
+  value(valueCallback, comparator = undefined) {
     if (!comparator) {
       comparator = this.default(SortDirection.ASC);
     }
     return (o1, o2) => {
-      return comparator(o1[prop], o2[prop]);
+      return comparator(valueCallback(o1), valueCallback(o2));
     };
-  }
+  },
+
+  property(prop, comparator = undefined) {
+    return this.value(value => value[prop], comparator);
+  },
 
 });

src/services/sort/sortable_properties.js

@@ -1,4 +1,4 @@
-import sortBy from 'lodash/sortBy';
+import { Comparators } from './comparators';
 
 /**
  * @typedef {Object} SortableProperty
@@ -43,9 +43,13 @@ export class SortableProperties {
    * @returns {Array.<Object>} sorted array of items, based off the sort properties.
    */
   sortItems(items) {
-    return this.isCurrentSortAscending()
-      ? sortBy(items, this.getSortedProperty().getValue)
-      : sortBy(items, this.getSortedProperty().getValue).reverse();
+    const copy = [...items];
+    let comparator = Comparators.value(this.getSortedProperty().getValue);
+    if (!this.isCurrentSortAscending()) {
+      comparator = Comparators.reverse(comparator);
+    }
+    copy.sort(comparator);
+    return copy;
   }
 
   /**

src/utils/prop_types/is.js

@@ -1,4 +1,4 @@
-import { isNil } from 'lodash';
+import { isNil } from '../../services/predicate';
 
 export const is = (expectedValue) => {
 

yarn.lock

@@ -5299,7 +5299,7 @@ lodash.uniq@^4.5.0:
   version "4.5.0"
   resolved "https://registry.yarnpkg.com/lodash.uniq/-/lodash.uniq-4.5.0.tgz#d0225373aeb652adc1bc82e4945339a842754773"
 
-lodash@^3.3.1:
+lodash@^3.10.1, lodash@^3.3.1:
   version "3.10.1"
   resolved "https://registry.yarnpkg.com/lodash/-/lodash-3.10.1.tgz#5bf45e8e49ba4189e17d482789dfd15bd140b7b6"