improve block insert method

src/components/modules/api/blocks.ts

@@ -1,5 +1,6 @@
 import { BlockAPI as BlockAPIInterface, Blocks } from '../../../../types/api';
 import { BlockToolData, OutputData, ToolConfig } from '../../../../types';
+import {InsertedBlock} from '../../../../types/api';
 import * as _ from './../../utils';
 import BlockAPI from '../../block/api';
 import Module from '../../__module';
@@ -222,30 +223,10 @@ export default class BlocksAPI extends Module {
 
   /**
    * Insert new Block and returns it's API
-   *
-   * @param {string} type — Tool name
-   * @param {BlockToolData} data — Tool data to insert
-   * @param {ToolConfig} config — Tool config
-   * @param {number?} index — index where to insert new Block
-   * @param {boolean?} needToFocus - flag to focus inserted Block
-   * @param replace - pass true to replace the Block existed under passed index
+   * @param {InsertedBlock} block - The block being inserted
    */
-  public insert = (
-    type: string = this.config.defaultBlock,
-    data: BlockToolData = {},
-    config: ToolConfig = {},
-    index?: number,
-    needToFocus?: boolean,
-    replace?: boolean
-  ): BlockAPIInterface => {
-    const insertedBlock = this.Editor.BlockManager.insert({
-      tool: type,
-      data,
-      index,
-      needToFocus,
-      replace,
-    });
-
+   public insert (block: InsertedBlock): BlockAPIInterface {
+    const insertedBlock = this.Editor.BlockManager.insert(block);
     return new BlockAPI(insertedBlock);
   }
 
@@ -278,7 +259,7 @@ export default class BlocksAPI extends Module {
   public insertNewBlock(): void {
     _.log('Method blocks.insertNewBlock() is deprecated and it will be removed in the next major release. ' +
       'Use blocks.insert() instead.', 'warn');
-    this.insert();
+    this.insert({});
   }
 
   /**

src/components/ui/toolbox.ts

@@ -404,14 +404,12 @@ export default class Toolbox extends EventsDispatcher<ToolboxEvent> {
       blockData = Object.assign(defaultBlockData, blockDataOverrides);
     }
 
-    const newBlock = this.api.blocks.insert(
-      toolName,
-      blockData,
-      undefined,
+    const newBlock = this.api.blocks.insert({
+      type: toolName,
+      data: blockData,
       index,
-      undefined,
-      currentBlock.isEmpty
-    );
+      replace: currentBlock.isEmpty,
+    });
 
     /**
      * Apply callback before inserting html

types/api/block.d.ts

@@ -74,3 +74,39 @@ export interface BlockAPI {
    */
   dispatchChange(): void;
 }
+
+/**
+ * @interface InsertedBLock Describes methods and properties of inserted blocks
+ */
+
+ export interface InsertedBlock{
+  /**
+   * Tool name
+   */
+  type?: string,
+
+  /**
+   * Tool data to insert
+   */
+  data?: BlockToolData,
+
+  /**
+   * Tool config
+   */
+  config?: ToolConfig,
+
+  /**
+   * index where to insert new block
+   */
+  index?: number,
+
+  /**
+   * falg to focus inserted block
+   */
+  needToFocus?: boolean,
+
+  /**
+   * pass true to replace the Block existed under passed index
+   */
+  replace?: boolean
+}

types/api/blocks.d.ts

@@ -1,3 +1,4 @@
+import { InsertedBlock } from '.';
 import {OutputData} from '../data-formats/output-data';
 import {BlockToolData, ToolConfig} from '../tools';
 import {BlockAPI} from './block';
@@ -96,21 +97,9 @@ export interface Blocks {
   /**
    * Insert new Block and return inserted Block API
    *
-   * @param {string} type — Tool name
-   * @param {BlockToolData} data — Tool data to insert
-   * @param {ToolConfig} config — Tool config
-   * @param {number?} index — index where to insert new Block
-   * @param {boolean?} needToFocus - flag to focus inserted Block
-   * @param {boolean?} replace - should the existed Block on that index be replaced or not
-   */
-  insert(
-    type?: string,
-    data?: BlockToolData,
-    config?: ToolConfig,
-    index?: number,
-    needToFocus?: boolean,
-    replace?: boolean,
-  ): BlockAPI;
+   * @param {InsertedBlock} block - new block config
+   */
+  insert(block: InsertedBlock): BlockAPI;
 
 
   /**