Remove shorthands option from LodashModuleReplacementPlugin

getsentry · Jun 24, 2017 · 1cd2042 · 1cd2042
1 parent 284063d
commit 1cd2042
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/src/sentry/static/sentry/app/components/events/interfaces/keyValueList.jsx b/src/sentry/static/sentry/app/components/events/interfaces/keyValueList.jsx
@@ -30,7 +30,7 @@ const KeyValueList = React.createClass({
       data = Object.keys(data).map(key => [key, data[key]]);
     }
 
-    data = this.props.isSorted ? _.sortBy(data, (key, value) => key) : data;
+    data = this.props.isSorted ? _.sortBy(data, [(key, value) => key]) : data;
 
     const props = this.props.onClick ? {onClick: this.props.onClick} : {};
     return (

diff --git a/src/sentry/static/sentry/app/options.jsx b/src/sentry/static/sentry/app/options.jsx
@@ -124,7 +124,7 @@ const definitions = [
   }
 ];
 
-const definitionsMap = _.keyBy(definitions, 'key');
+const definitionsMap = _.keyBy(definitions, def => def.key);
 
 const disabledReasons = {
   diskPriority: 'This setting is defined in config.yml and may not be changed via the web UI.',

diff --git a/webpack.config.js b/webpack.config.js
@@ -137,7 +137,6 @@ var config = {
   },
   plugins: [
     new LodashModuleReplacementPlugin({
-      shorthands: true,
       collections: true
     }),
     new webpack.optimize.CommonsChunkPlugin({