From ac549ac6f5fe47ec3eacde87a1b1827c68ece119 Mon Sep 17 00:00:00 2001
From: Wylie Conlon <william.conlon@elastic.co>
Date: Wed, 15 Apr 2020 14:57:44 -0400
Subject: [PATCH] [Lens] Only show copy on save for previously saved docs
 (#63535)

* [Lens] Only show copy on save for previously saved docs

* Update app.test.tsx

import after kibana platform changes

Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com>
---
 .../lens/public/app_plugin/app.test.tsx       | 22 +++++++++++++++++++
 x-pack/plugins/lens/public/app_plugin/app.tsx |  2 +-
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/x-pack/plugins/lens/public/app_plugin/app.test.tsx b/x-pack/plugins/lens/public/app_plugin/app.test.tsx
index d49c128dff60..41d0e3a7aa9a 100644
--- a/x-pack/plugins/lens/public/app_plugin/app.test.tsx
+++ b/x-pack/plugins/lens/public/app_plugin/app.test.tsx
@@ -12,6 +12,7 @@ import { EditorFrameInstance } from '../types';
 import { Storage } from '../../../../../src/plugins/kibana_utils/public';
 import { Document, SavedObjectStore } from '../persistence';
 import { mount } from 'enzyme';
+import { SavedObjectSaveModal } from '../../../../../src/plugins/saved_objects/public';
 import {
   esFilters,
   FilterManager,
@@ -650,6 +651,27 @@ describe('Lens App', () => {
           },
         });
       });
+
+      it('does not show the copy button on first save', async () => {
+        const args = defaultArgs;
+        args.editorFrame = frame;
+
+        instance = mount(<App {...args} />);
+
+        const onChange = frame.mount.mock.calls[0][1].onChange;
+        await act(async () =>
+          onChange({
+            filterableIndexPatterns: [],
+            doc: ({ expression: 'valid expression' } as unknown) as Document,
+          })
+        );
+        instance.update();
+
+        await act(async () => getButton(instance).run(instance.getDOMNode()));
+        instance.update();
+
+        expect(instance.find(SavedObjectSaveModal).prop('showCopyOnSave')).toEqual(false);
+      });
     });
   });
 
diff --git a/x-pack/plugins/lens/public/app_plugin/app.tsx b/x-pack/plugins/lens/public/app_plugin/app.tsx
index 2d8f1650e400..28135dd12a72 100644
--- a/x-pack/plugins/lens/public/app_plugin/app.tsx
+++ b/x-pack/plugins/lens/public/app_plugin/app.tsx
@@ -387,7 +387,7 @@ export function App({
             }}
             onClose={() => setState(s => ({ ...s, isSaveModalVisible: false }))}
             title={lastKnownDoc.title || ''}
-            showCopyOnSave={!addToDashboardMode}
+            showCopyOnSave={!!lastKnownDoc.id && !addToDashboardMode}
             objectType={i18n.translate('xpack.lens.app.saveModalType', {
               defaultMessage: 'Lens visualization',
             })}