From 645f3c486bb6f8321da9227b0eba217cff598ff0 Mon Sep 17 00:00:00 2001
From: Nathan Reese <reese.nathan@gmail.com>
Date: Wed, 13 Nov 2019 14:19:05 -0700
Subject: [PATCH] add checkbox to join UI

---
 .../components/global_filter_checkbox.js      | 10 ++++----
 .../layer_panel/join_editor/resources/join.js | 23 +++++++++++++++++++
 .../layer_panel/join_editor/view.js           |  3 ++-
 .../maps/public/layers/sources/source.js      |  1 -
 4 files changed, 31 insertions(+), 6 deletions(-)

diff --git a/x-pack/legacy/plugins/maps/public/components/global_filter_checkbox.js b/x-pack/legacy/plugins/maps/public/components/global_filter_checkbox.js
index 8694fd0451238..e841fa573c9a5 100644
--- a/x-pack/legacy/plugins/maps/public/components/global_filter_checkbox.js
+++ b/x-pack/legacy/plugins/maps/public/components/global_filter_checkbox.js
@@ -8,7 +8,11 @@ import React from 'react';
 import { EuiFormRow, EuiSwitch } from '@elastic/eui';
 import { i18n } from '@kbn/i18n';
 
-export function GlobalFilterCheckbox({ applyGlobalQuery, setApplyGlobalQuery }) {
+const label = i18n.translate('xpack.maps.layerPanel.applyGlobalQueryCheckboxLabel', {
+  defaultMessage: `Apply global filter to source`,
+});
+
+export function GlobalFilterCheckbox({ applyGlobalQuery, customLabel, setApplyGlobalQuery }) {
   const onApplyGlobalQueryChange = event => {
     setApplyGlobalQuery(event.target.checked);
   };
@@ -18,9 +22,7 @@ export function GlobalFilterCheckbox({ applyGlobalQuery, setApplyGlobalQuery })
       display="columnCompressedSwitch"
     >
       <EuiSwitch
-        label={i18n.translate('xpack.maps.layerPanel.applyGlobalQueryCheckboxLabel', {
-          defaultMessage: `Apply global filter to source`,
-        })}
+        label={customLabel ? customLabel : label}
         checked={applyGlobalQuery}
         onChange={onApplyGlobalQueryChange}
         data-test-subj="mapLayerPanelApplyGlobalQueryCheckbox"
diff --git a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/join_editor/resources/join.js b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/join_editor/resources/join.js
index 44987e0d241e3..2b7c614f182fd 100644
--- a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/join_editor/resources/join.js
+++ b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/join_editor/resources/join.js
@@ -15,6 +15,7 @@ import { i18n } from '@kbn/i18n';
 import { JoinExpression } from './join_expression';
 import { MetricsExpression } from './metrics_expression';
 import { WhereExpression } from './where_expression';
+import { GlobalFilterCheckbox } from '../../../../components/global_filter_checkbox';
 
 import {
   indexPatternService,
@@ -168,6 +169,16 @@ export class Join extends Component {
     });
   }
 
+  _onApplyGlobalQueryChange = applyGlobalQuery => {
+    this.props.onChange({
+      leftField: this.props.join.leftField,
+      right: {
+        ...this.props.join.right,
+        applyGlobalQuery,
+      },
+    });
+  }
+
   render() {
     const {
       join,
@@ -184,6 +195,7 @@ export class Join extends Component {
     const isJoinConfigComplete = join.leftField && right.indexPatternId && right.term;
 
     let metricsExpression;
+    let globalFilterCheckbox;
     if (isJoinConfigComplete) {
       metricsExpression = (
         <EuiFlexItem grow={false}>
@@ -194,6 +206,15 @@ export class Join extends Component {
           />
         </EuiFlexItem>
       );
+      globalFilterCheckbox = (
+        <GlobalFilterCheckbox
+          applyGlobalQuery={right.applyGlobalQuery}
+          setApplyGlobalQuery={this._onApplyGlobalQueryChange}
+          customLabel={i18n.translate('xpack.maps.layerPanel.join.applyGlobalQueryCheckboxLabel', {
+            defaultMessage: `Apply global filter to join`,
+          })}
+        />
+      );
     }
 
     let whereExpression;
@@ -234,6 +255,8 @@ export class Join extends Component {
 
           {whereExpression}
 
+          {globalFilterCheckbox}
+
           <EuiButtonIcon
             className="mapJoinItem__delete"
             iconType="trash"
diff --git a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/join_editor/view.js b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/join_editor/view.js
index 51092b0abc1a1..b48163816bb52 100644
--- a/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/join_editor/view.js
+++ b/x-pack/legacy/plugins/maps/public/connected_components/layer_panel/join_editor/view.js
@@ -58,7 +58,8 @@ export function JoinEditor({ joins, layer, onChange }) {
       ...joins,
       {
         right: {
-          id: uuid()
+          id: uuid(),
+          applyGlobalQuery: true,
         }
       }
     ]);
diff --git a/x-pack/legacy/plugins/maps/public/layers/sources/source.js b/x-pack/legacy/plugins/maps/public/layers/sources/source.js
index d1b17cb7aec3b..78e57f79bbe56 100644
--- a/x-pack/legacy/plugins/maps/public/layers/sources/source.js
+++ b/x-pack/legacy/plugins/maps/public/layers/sources/source.js
@@ -96,7 +96,6 @@ export class AbstractSource {
   }
 
   getApplyGlobalQuery() {
-    console.log(this._descriptor.applyGlobalQuery);
     return !!this._descriptor.applyGlobalQuery;
   }