Revert "Applying TSVB fixes to 7.6" (#65674)

This reverts commit c685aac.

Author: Maja Grubic

package.json

@@ -138,7 +138,6 @@
     "@types/node-forge": "^0.9.0",
     "@types/react-grid-layout": "^0.16.7",
     "@types/recompose": "^0.30.5",
-    "@types/set-value": "^2.0.0",
     "JSONStream": "1.3.5",
     "abortcontroller-polyfill": "^1.3.0",
     "angular": "^1.7.9",
@@ -254,7 +253,6 @@
     "script-loader": "0.7.2",
     "seedrandom": "^3.0.5",
     "semver": "^5.5.0",
-    "set-value": "^3.0.2",
     "style-loader": "0.23.1",
     "symbol-observable": "^1.2.0",
     "tar": "4.4.13",

src/legacy/core_plugins/vis_type_timeseries/server/lib/vis_data/helpers/index.js

@@ -29,8 +29,6 @@ import { getTimerange } from './get_timerange';
 import { mapBucket } from './map_bucket';
 import { parseSettings } from './parse_settings';
 
-export { overwrite } from './overwrite';
-
 export const helpers = {
   bucketTransform,
   getAggValue,

src/legacy/core_plugins/vis_type_timeseries/server/lib/vis_data/helpers/overwrite.js

@@ -16,10 +16,11 @@
  * specific language governing permissions and limitations
  * under the License.
  */
+
+import _ from 'lodash';
 import { dateHistogramInterval } from '../../../../../../data/server';
 import { getBucketSize } from '../../helpers/get_bucket_size';
 import { getTimerange } from '../../helpers/get_timerange';
-import { overwrite } from '../../helpers';
 
 export function dateHistogram(
   req,
@@ -35,7 +36,7 @@ export function dateHistogram(
     const { from, to } = getTimerange(req);
     const timezone = capabilities.searchTimezone;
 
-    overwrite(doc, `aggs.${annotation.id}.date_histogram`, {
+    _.set(doc, `aggs.${annotation.id}.date_histogram`, {
       field: timeField,
       min_doc_count: 0,
       time_zone: timezone,

src/legacy/core_plugins/vis_type_timeseries/server/lib/vis_data/request_processors/annotations/date_histogram.js

@@ -17,13 +17,13 @@
  * under the License.
  */
 
-import { overwrite } from '../../helpers';
+import _ from 'lodash';
 
 export function topHits(req, panel, annotation) {
   return next => doc => {
     const fields = (annotation.fields && annotation.fields.split(/[,\s]+/)) || [];
     const timeField = annotation.time_field;
-    overwrite(doc, `aggs.${annotation.id}.aggs.hits.top_hits`, {
+    _.set(doc, `aggs.${annotation.id}.aggs.hits.top_hits`, {
       sort: [
         {
           [timeField]: { order: 'desc' },

src/legacy/core_plugins/vis_type_timeseries/server/lib/vis_data/request_processors/annotations/top_hits.js

@@ -17,7 +17,7 @@
  * under the License.
  */
 
-import { overwrite } from '../../helpers';
+import { set } from 'lodash';
 import { dateHistogramInterval } from '../../../../../../data/server';
 import { getBucketSize } from '../../helpers/get_bucket_size';
 import { offsetTime } from '../../offset_time';
@@ -33,7 +33,7 @@ export function dateHistogram(req, panel, series, esQueryConfig, indexPatternObj
       const { from, to } = offsetTime(req, series.offset_time);
       const timezone = capabilities.searchTimezone;
 
-      overwrite(doc, `aggs.${series.id}.aggs.timeseries.date_histogram`, {
+      set(doc, `aggs.${series.id}.aggs.timeseries.date_histogram`, {
         field: timeField,
         min_doc_count: 0,
         time_zone: timezone,
@@ -46,7 +46,7 @@ export function dateHistogram(req, panel, series, esQueryConfig, indexPatternObj
     };
 
     const getDateHistogramForEntireTimerangeMode = () =>
-      overwrite(doc, `aggs.${series.id}.aggs.timeseries.auto_date_histogram`, {
+      set(doc, `aggs.${series.id}.aggs.timeseries.auto_date_histogram`, {
         field: timeField,
         buckets: 1,
       });
@@ -57,7 +57,7 @@ export function dateHistogram(req, panel, series, esQueryConfig, indexPatternObj
 
     // master
 
-    overwrite(doc, `aggs.${series.id}.meta`, {
+    set(doc, `aggs.${series.id}.meta`, {
       timeField,
       intervalString,
       bucketSize,

src/legacy/core_plugins/vis_type_timeseries/server/lib/vis_data/request_processors/series/date_histogram.js

@@ -19,16 +19,16 @@
 
 const filter = metric => metric.type === 'filter_ratio';
 import { bucketTransform } from '../../helpers/bucket_transform';
-import { overwrite } from '../../helpers';
+import _ from 'lodash';
 
 export function ratios(req, panel, series) {
   return next => doc => {
     if (series.metrics.some(filter)) {
       series.metrics.filter(filter).forEach(metric => {
-        overwrite(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}-numerator.filter`, {
+        _.set(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}-numerator.filter`, {
           query_string: { query: metric.numerator || '*', analyze_wildcard: true },
         });
-        overwrite(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}-denominator.filter`, {
+        _.set(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}-denominator.filter`, {
           query_string: { query: metric.denominator || '*', analyze_wildcard: true },
         });
 
@@ -46,12 +46,8 @@ export function ratios(req, panel, series) {
             metricAgg = {};
           }
           const aggBody = { metric: metricAgg };
-          overwrite(
-            doc,
-            `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}-numerator.aggs`,
-            aggBody
-          );
-          overwrite(
+          _.set(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}-numerator.aggs`, aggBody);
+          _.set(
             doc,
             `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}-denominator.aggs`,
             aggBody
@@ -60,7 +56,7 @@ export function ratios(req, panel, series) {
           denominatorPath = `${metric.id}-denominator>metric`;
         }
 
-        overwrite(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}`, {
+        _.set(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}`, {
           bucket_script: {
             buckets_path: {
               numerator: numeratorPath,

src/legacy/core_plugins/vis_type_timeseries/server/lib/vis_data/request_processors/series/filter_ratios.js

@@ -17,7 +17,7 @@
  * under the License.
  */
 
-import { overwrite } from '../../helpers';
+import _ from 'lodash';
 import { getBucketSize } from '../../helpers/get_bucket_size';
 import { bucketTransform } from '../../helpers/bucket_transform';
 import { getIntervalAndTimefield } from '../../get_interval_and_timefield';
@@ -33,7 +33,7 @@ export function metricBuckets(req, panel, series, esQueryConfig, indexPatternObj
         if (fn) {
           try {
             const bucket = fn(metric, series.metrics, intervalString);
-            overwrite(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}`, bucket);
+            _.set(doc, `aggs.${series.id}.aggs.timeseries.aggs.${metric.id}`, bucket);
           } catch (e) {
             // meh
           }

src/legacy/core_plugins/vis_type_timeseries/server/lib/vis_data/request_processors/series/metric_buckets.js

@@ -16,10 +16,9 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-import { overwrite } from '../../helpers';
-import _ from 'lodash';
+const { set, get, isEmpty } = require('lodash');
 
-const isEmptyFilter = (filter = {}) => Boolean(filter.match_all) && _.isEmpty(filter.match_all);
+const isEmptyFilter = (filter = {}) => Boolean(filter.match_all) && isEmpty(filter.match_all);
 const hasSiblingPipelineAggregation = (aggs = {}) => Object.keys(aggs).length > 1;
 
 /* For grouping by the 'Everything', the splitByEverything request processor
@@ -31,12 +30,12 @@ const hasSiblingPipelineAggregation = (aggs = {}) => Object.keys(aggs).length >
  *
  */
 function removeEmptyTopLevelAggregation(doc, series) {
-  const filter = _.get(doc, `aggs.${series.id}.filter`);
+  const filter = get(doc, `aggs.${series.id}.filter`);
 
   if (isEmptyFilter(filter) && !hasSiblingPipelineAggregation(doc.aggs[series.id].aggs)) {
-    const meta = _.get(doc, `aggs.${series.id}.meta`);
-    overwrite(doc, `aggs`, doc.aggs[series.id].aggs);
-    overwrite(doc, `aggs.timeseries.meta`, meta);
+    const meta = get(doc, `aggs.${series.id}.meta`);
+    set(doc, `aggs`, doc.aggs[series.id].aggs);
+    set(doc, `aggs.timeseries.meta`, meta);
   }
 
   return doc;

src/legacy/core_plugins/vis_type_timeseries/server/lib/vis_data/request_processors/series/normalize_query.js

@@ -17,7 +17,7 @@
  * under the License.
  */
 
-import { overwrite } from '../../helpers';
+import _ from 'lodash';
 import { getBucketSize } from '../../helpers/get_bucket_size';
 import { bucketTransform } from '../../helpers/bucket_transform';
 import { getIntervalAndTimefield } from '../../get_interval_and_timefield';
@@ -40,7 +40,7 @@ export function siblingBuckets(
         if (fn) {
           try {
             const bucket = fn(metric, series.metrics, bucketSize);
-            overwrite(doc, `aggs.${series.id}.aggs.${metric.id}`, bucket);
+            _.set(doc, `aggs.${series.id}.aggs.${metric.id}`, bucket);
           } catch (e) {
             // meh
           }