...

docs/development/plugins/data/public/kibana-plugin-plugins-data-public.indexpattern.md

@@ -39,7 +39,6 @@ export declare class IndexPattern implements IIndexPattern
 |  [\_fetchFields()](./kibana-plugin-plugins-data-public.indexpattern._fetchfields.md) |  |  |
 |  [addScriptedField(name, script, fieldType, lang)](./kibana-plugin-plugins-data-public.indexpattern.addscriptedfield.md) |  |  |
 |  [create(allowOverride)](./kibana-plugin-plugins-data-public.indexpattern.create.md) |  |  |
-|  [destroy()](./kibana-plugin-plugins-data-public.indexpattern.destroy.md) |  |  |
 |  [getAggregationRestrictions()](./kibana-plugin-plugins-data-public.indexpattern.getaggregationrestrictions.md) |  |  |
 |  [getComputedFields()](./kibana-plugin-plugins-data-public.indexpattern.getcomputedfields.md) |  |  |
 |  [getFieldByName(name)](./kibana-plugin-plugins-data-public.indexpattern.getfieldbyname.md) |  |  |

src/plugins/data/common/index_patterns/index_patterns/index_patterns.test.ts

@@ -53,6 +53,7 @@ describe('IndexPatterns', () => {
           Array<SavedObject<any>>
         >
     );
+    savedObjectsClient.delete = jest.fn(() => Promise.resolve({}) as Promise<any>);
 
     indexPatterns = new IndexPatternsService({
       uiSettings: ({
@@ -98,4 +99,13 @@ describe('IndexPatterns', () => {
     await indexPatterns.getFields(['id', 'title'], true);
     expect(savedObjectsClient.find).toHaveBeenCalledTimes(3);
   });
+
+  test('deletes the index pattern', async () => {
+    const id = '1';
+    const indexPattern = await indexPatterns.get(id);
+
+    expect(indexPattern).toBeDefined();
+    await indexPatterns.delete(id);
+    expect(indexPattern).not.toBe(await indexPatterns.get(id));
+  });
 });

src/plugins/data/common/index_patterns/index_patterns/index_patterns.ts

@@ -231,13 +231,11 @@ export class IndexPatternsService {
 
   /**
    * Deletes an index pattern from .kibana index
-   * @param indexPattern: IndexPattern instance to delete
+   * @param indexPatternId: Id of kibana Index Pattern to delete
    */
-  async delete(indexPattern: IndexPattern) {
-    if (indexPattern.id) {
-      indexPatternCache.clear(indexPattern.id);
-      return this.savedObjectsClient.delete('index-pattern', indexPattern.id);
-    }
+  async delete(indexPatternId: string) {
+    indexPatternCache.clear(indexPatternId);
+    return this.savedObjectsClient.delete('index-pattern', indexPatternId);
   }
 }
 

src/plugins/data/public/public.api.md

@@ -988,8 +988,6 @@ export class IndexPattern implements IIndexPattern {
     // (undocumented)
     create(allowOverride?: boolean): Promise<string | false>;
     // (undocumented)
-    destroy(): Promise<{}> | undefined;
-    // (undocumented)
     _fetchFields(): Promise<void>;
     // (undocumented)
     fieldFormatMap: any;

src/plugins/index_pattern_management/public/components/edit_index_pattern/edit_index_pattern.tsx

@@ -143,10 +143,11 @@ export const EditIndexPattern = withRouter(
             uiSettings.set('defaultIndex', otherPatterns[0].id);
           }
         }
-
-        Promise.resolve(data.indexPatterns.delete(indexPattern)).then(function () {
-          history.push('');
-        });
+        if (indexPattern.id) {
+          Promise.resolve(data.indexPatterns.delete(indexPattern.id)).then(function () {
+            history.push('');
+          });
+        }
       }
 
       overlays.openConfirm('', confirmModalOptionsDelete).then((isConfirmed) => {

test/plugin_functional/plugins/index_patterns/server/plugin.ts

@@ -96,8 +96,7 @@ export class IndexPatternsTestPlugin
         const [, { data }] = await core.getStartServices();
         const id = (req.params as Record<string, string>).id;
         const service = await data.indexPatterns.indexPatternsServiceFactory(req);
-        const ip = await service.get(id);
-        await ip.destroy();
+        await service.delete(id);
         return res.ok();
       }
     );