@kineticdata/react 6.0.0 → 6.0.2

package/lib/components/task/builder/Node.js

@@ -14,19 +14,23 @@ var _react = _interopRequireWildcard(require("react"));
 var _lodashEs = require("lodash-es");
 var _classnames = _interopRequireDefault(require("classnames"));
 var _store = require("../../../store");
-var constants = _interopRequireWildcard(require("./constants"));
+var _constants = _interopRequireWildcard(require("./constants"));
+var constants = _constants;
 var _helpers = require("./helpers");
 var _models = require("./models");
 var _SvgText = require("./SvgText");
 var _plus_small = _interopRequireDefault(require("../../../../assets/task/icons/plus_small.svg"));
 var _dragHandle = _interopRequireDefault(require("../../../../assets/task/icons/drag-handle.svg"));
-var Node = /*#__PURE__*/function (_Component) {
+var Node = exports.Node = /*#__PURE__*/function (_Component) {
   (0, _inherits2["default"])(Node, _Component);
   var _super = (0, _createSuper2["default"])(Node);
   function Node(props) {
     var _this;
     (0, _classCallCheck2["default"])(this, Node);
     _this = _super.call(this, props);
+    /*****************************************************************************
+     * Click handlers                                                            *
+     ****************************************************************************/
     _this.onSelect = function (shiftKey) {
       return function () {
         if ((0, _lodashEs.isFunction)(_this.props.onSelect)) {
@@ -34,6 +38,16 @@ var Node = /*#__PURE__*/function (_Component) {
         }
       };
     };
+    /*****************************************************************************
+     * Drag-and-drop support                                                     *
+     * Leverages the `watchDrag` helper exposed by the `TreeBuilder` instance.   *
+     * On move we need to update any related connectors manually (for maximum    *
+     * performance).                                                             *
+     * On drop we dispatch an action to update the node in redux to persist the  *
+     * change.                                                                   *
+     * We also watch for drag on the plus button, which should leverage the      *
+     * `TreeBuilder` instance to start creating a new connector                  *
+     ****************************************************************************/
     _this.drag = function (event) {
       _this.treeBuilder.watchDrag({
         event: event,
@@ -48,7 +62,7 @@ var Node = /*#__PURE__*/function (_Component) {
         relative: false,
         onMove: _this.treeBuilder.dragNewConnector(_this.props.node),
         onDrop: _this.treeBuilder.dropNewConnector,
-        onClick: _this.treeBuilder.addNewNode(_this.props.node, event.nativeEvent, (0, _helpers.getNodeType)(_this.props.node) === 'start' ? constants.NODE_START_RADIUS * 2 - 18 : constants.NODE_WIDTH - 18, (0, _helpers.getNodeType)(_this.props.node) === 'start' ? (constants.NODE_START_RADIUS * 2 - constants.NODE_HEIGHT) / 2 : 0)
+        onClick: _this.treeBuilder.addNewNode(_this.props.node, event.nativeEvent, (0, _helpers.getNodeType)(_this.props.node) === 'start' ? _constants.NODE_START_RADIUS * 2 - 18 : _constants.NODE_WIDTH - 18, (0, _helpers.getNodeType)(_this.props.node) === 'start' ? (_constants.NODE_START_RADIUS * 2 - _constants.NODE_HEIGHT) / 2 : 0)
       });
     };
     _this.drop = function () {
@@ -84,10 +98,6 @@ var Node = /*#__PURE__*/function (_Component) {
     value: function setTreeBuilder(treeBuilder) {
       this.treeBuilder = treeBuilder;
     }
-
-    /*****************************************************************************
-     * Click handlers                                                            *
-     ****************************************************************************/
   }, {
     key: "shouldComponentUpdate",
     value:
@@ -238,5 +248,4 @@ var Node = /*#__PURE__*/function (_Component) {
     }
   }]);
   return Node;
-}(_react.Component);
-exports.Node = Node;
+}(_react.Component);

package/lib/components/task/builder/NodeForm.js

@@ -212,11 +212,10 @@ var handleSubmit = function handleSubmit(_ref11) {
     return node.merge(values);
   };
 };
-var NodeForm = (0, _Form.generateForm)({
+var NodeForm = exports.NodeForm = (0, _Form.generateForm)({
   formOptions: ['node', 'tasks', 'tree'],
   dataSources: dataSources,
   fields: fields,
   handleSubmit: handleSubmit
 });
-exports.NodeForm = NodeForm;
 NodeForm.displayName = 'NodeForm';

package/lib/components/task/builder/NodeParametersForm.js

@@ -134,11 +134,10 @@ var handleSubmit = function handleSubmit(_ref5) {
     return values;
   };
 };
-var NodeParametersForm = (0, _Form.generateForm)({
+var NodeParametersForm = exports.NodeParametersForm = (0, _Form.generateForm)({
   formOptions: ['node', 'task', 'tasks', 'tree'],
   dataSources: dataSources,
   fields: fields,
   handleSubmit: handleSubmit
 });
-exports.NodeParametersForm = NodeParametersForm;
 NodeParametersForm.displayName = 'NodeParametersForm';

package/lib/components/task/builder/SvgCanvas.js

@@ -15,7 +15,7 @@ var _lodashEs = require("lodash-es");
 var _helpers = require("./helpers");
 var constants = _interopRequireWildcard(require("./constants"));
 var _models = require("./models");
-var SvgCanvas = /*#__PURE__*/function (_Component) {
+var SvgCanvas = exports.SvgCanvas = /*#__PURE__*/function (_Component) {
   (0, _inherits2["default"])(SvgCanvas, _Component);
   var _super = (0, _createSuper2["default"])(SvgCanvas);
   function SvgCanvas(props) {
@@ -267,5 +267,4 @@ var SvgCanvas = /*#__PURE__*/function (_Component) {
     }
   }]);
   return SvgCanvas;
-}(_react.Component);
-exports.SvgCanvas = SvgCanvas;
+}(_react.Component);

package/lib/components/task/builder/SvgText.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.SvgText = void 0;
 var _react = _interopRequireDefault(require("react"));
-var SvgText = function SvgText(props) {
+var SvgText = exports.SvgText = function SvgText(props) {
   return /*#__PURE__*/_react["default"].createElement("foreignObject", {
     className: props.className,
     x: props.x + props.padding,
@@ -16,5 +16,4 @@ var SvgText = function SvgText(props) {
   }, /*#__PURE__*/_react["default"].createElement("div", {
     className: "text-node"
   }, /*#__PURE__*/_react["default"].createElement("span", null, props.children)));
-};
-exports.SvgText = SvgText;
+};

package/lib/components/task/builder/TaskDefinitionConfigForm.js

@@ -10,15 +10,14 @@ var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/esm/
 var _immutable = require("immutable");
 var _Form = require("../../form/Form");
 var _apis = require("../../../apis");
-var checkOmittedParameters = function checkOmittedParameters(node, parameter) {
+var checkOmittedParameters = exports.checkOmittedParameters = function checkOmittedParameters(node, parameter) {
   if (node.definitionId === 'system_submission_create_v1') {
     return !['kappSlug', 'formSlug'].includes(parameter.id);
   } else {
     return true;
   }
 };
-exports.checkOmittedParameters = checkOmittedParameters;
-var generateTaskDefinition = function generateTaskDefinition(taskDefinition) {
+var generateTaskDefinition = exports.generateTaskDefinition = function generateTaskDefinition(taskDefinition) {
   if (taskDefinition && taskDefinition.definitionName === 'system_submission_create') {
     return function (_ref) {
       var form = _ref.form;
@@ -46,7 +45,6 @@ var generateTaskDefinition = function generateTaskDefinition(taskDefinition) {
     return null;
   }
 };
-exports.generateTaskDefinition = generateTaskDefinition;
 var dataSources = function dataSources() {
   return {
     kapps: {
@@ -108,11 +106,10 @@ var handleSubmit = function handleSubmit(_ref6) {
     }).then(generateTaskDefinition(taskDefinition));
   };
 };
-var TaskDefinitionConfigForm = (0, _Form.generateForm)({
+var TaskDefinitionConfigForm = exports.TaskDefinitionConfigForm = (0, _Form.generateForm)({
   formOptions: ['taskDefinition'],
   dataSources: dataSources,
   fields: fields,
   handleSubmit: handleSubmit
 });
-exports.TaskDefinitionConfigForm = TaskDefinitionConfigForm;
 TaskDefinitionConfigForm.displayName = 'TaskDefinitionConfigForm';

package/lib/components/task/builder/TreeBuilder.js

@@ -24,7 +24,7 @@ var _SvgCanvas = require("./SvgCanvas");
 var _Node = require("./Node");
 var _Connector = require("./Connector");
 var _immutable = require("immutable");
-var TreeBuilderComponent = /*#__PURE__*/function (_Component) {
+var TreeBuilderComponent = exports.TreeBuilderComponent = /*#__PURE__*/function (_Component) {
   (0, _inherits2["default"])(TreeBuilderComponent, _Component);
   var _super = (0, _createSuper2["default"])(TreeBuilderComponent);
   function TreeBuilderComponent(props) {
@@ -78,6 +78,12 @@ var TreeBuilderComponent = /*#__PURE__*/function (_Component) {
         return _this.props.tree.nodes.get(connector.headId);
       });
     };
+    // helper provided so that the plus button on a node can start the process of
+    // creating a new connector, since the drop position may not be valid we do
+    // not want to persist anything to redux state until we can confirm the drop
+    // is valid so we store the temp connector in component state so it will be
+    // rendered and then on move we update the Connector instance (by ref) with
+    // the new position
     _this.dragNewConnector = function (node) {
       return function (position) {
         if (!_this.state.newConnector) {
@@ -95,6 +101,8 @@ var TreeBuilderComponent = /*#__PURE__*/function (_Component) {
         }
       };
     };
+    // helper provided so that the plus button on a node can create a new
+    // connector in conjunction with `dragNewConnector` above
     _this.dropNewConnector = function () {
       if (_this.newConnector.current) {
         _this.newConnector.current.dropHead();
@@ -403,12 +411,10 @@ var TreeBuilderComponent = /*#__PURE__*/function (_Component) {
   }]);
   return TreeBuilderComponent;
 }(_react.Component);
-exports.TreeBuilderComponent = TreeBuilderComponent;
 var mapStateToProps = function mapStateToProps(state, props) {
   return {
     treeBuilderState: state.getIn(['trees', props.treeKey]),
     tree: state.getIn(['trees', props.treeKey, 'tree'])
   };
 };
-var TreeBuilder = (0, _store.connect)(mapStateToProps)(TreeBuilderComponent);
-exports.TreeBuilder = TreeBuilder;
+var TreeBuilder = exports.TreeBuilder = (0, _store.connect)(mapStateToProps)(TreeBuilderComponent);

package/lib/components/task/builder/builder.redux.js

@@ -16,25 +16,22 @@ var _store = require("../../../store");
 var _models = require("./models");
 var _apis = require("../../../apis");
 var _helpers = require("./helpers");
-var mountTreeBuilder = function mountTreeBuilder(treeKey) {
+var mountTreeBuilder = exports.mountTreeBuilder = function mountTreeBuilder(treeKey) {
   return (0, _store.dispatch)('TREE_MOUNT', {
     treeKey: treeKey
   });
 };
-exports.mountTreeBuilder = mountTreeBuilder;
-var unmountTreeBuilder = function unmountTreeBuilder(treeKey) {
+var unmountTreeBuilder = exports.unmountTreeBuilder = function unmountTreeBuilder(treeKey) {
   return (0, _store.dispatch)('TREE_UNMOUNT', {
     treeKey: treeKey
   });
 };
-exports.unmountTreeBuilder = unmountTreeBuilder;
-var configureTreeBuilder = function configureTreeBuilder(props) {
+var configureTreeBuilder = exports.configureTreeBuilder = function configureTreeBuilder(props) {
   return (0, _store.dispatch)('TREE_CONFIGURE', props);
 };
 
 // Helper that adds the present state to the past stack and clears the future
 // stack. This should be called by reducer cases that will be undo/redo able.
-exports.configureTreeBuilder = configureTreeBuilder;
 var remember = function remember(state, treeKey) {
   return state.updateIn(['trees', treeKey, 'undoStack'], function (stack) {
     return stack.push({
@@ -48,63 +45,61 @@ var remember = function remember(state, treeKey) {
   return /*#__PURE__*/(0, _regeneratorRuntime2["default"])().mark(function _callee() {
     var name, sourceGroup, sourceName, treeKey, webApiProps, _yield$all, _yield$all2, tree, categories, webApi, platformItem, result;
     return (0, _regeneratorRuntime2["default"])().wrap(function _callee$(_context) {
-      while (1) {
-        switch (_context.prev = _context.next) {
-          case 0:
-            _context.prev = 0;
-            name = payload.name, sourceGroup = payload.sourceGroup, sourceName = payload.sourceName, treeKey = payload.treeKey;
-            webApiProps = getWebApiProps(payload);
-            _context.next = 5;
-            return (0, _effects.all)([(0, _effects.call)(_apis.fetchTree, {
-              name: name,
-              sourceGroup: sourceGroup,
-              sourceName: sourceName,
-              include: 'bindings,categories,details,treeJson,inputs,outputs'
-            }), (0, _effects.call)(_apis.fetchTaskCategories, {
-              include: 'handlers.results,handlers.parameters,trees.parameters,trees.inputs,trees.outputs'
-            }), webApiProps ? (0, _effects.call)(_apis.fetchWebApi, (0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, webApiProps), {}, {
-              include: 'details,securityPolicies'
-            })) : {}]);
-          case 5:
-            _yield$all = _context.sent;
-            _yield$all2 = (0, _slicedToArray2["default"])(_yield$all, 3);
-            tree = _yield$all2[0].tree;
-            categories = _yield$all2[1].categories;
-            webApi = _yield$all2[2].webApi;
-            platformItem = null;
-            if (!tree.event) {
-              _context.next = 16;
-              break;
-            }
-            _context.next = 14;
-            return (0, _effects.call)(_apis.fetchPlatformItem, {
-              type: tree.platformItemType,
-              id: tree.platformItemId
-            });
-          case 14:
-            result = _context.sent;
-            platformItem = result.platformItem;
-          case 16:
-            _context.next = 18;
-            return (0, _effects.put)((0, _store.action)('TREE_LOADED', {
-              categories: categories,
-              kappSlug: webApiProps && webApiProps.kappSlug,
-              platformItem: platformItem,
-              treeKey: treeKey,
-              tree: (0, _models.deserializeTree)(tree),
-              webApi: webApiProps && (0, _models.deserializeWebApi)(webApi, webApiProps.kappSlug)
-            }));
-          case 18:
-            _context.next = 23;
+      while (1) switch (_context.prev = _context.next) {
+        case 0:
+          _context.prev = 0;
+          name = payload.name, sourceGroup = payload.sourceGroup, sourceName = payload.sourceName, treeKey = payload.treeKey;
+          webApiProps = getWebApiProps(payload);
+          _context.next = 5;
+          return (0, _effects.all)([(0, _effects.call)(_apis.fetchTree, {
+            name: name,
+            sourceGroup: sourceGroup,
+            sourceName: sourceName,
+            include: 'bindings,categories,details,treeJson,inputs,outputs'
+          }), (0, _effects.call)(_apis.fetchTaskCategories, {
+            include: 'handlers.results,handlers.parameters,trees.parameters,trees.inputs,trees.outputs'
+          }), webApiProps ? (0, _effects.call)(_apis.fetchWebApi, (0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, webApiProps), {}, {
+            include: 'details,securityPolicies'
+          })) : {}]);
+        case 5:
+          _yield$all = _context.sent;
+          _yield$all2 = (0, _slicedToArray2["default"])(_yield$all, 3);
+          tree = _yield$all2[0].tree;
+          categories = _yield$all2[1].categories;
+          webApi = _yield$all2[2].webApi;
+          platformItem = null;
+          if (!tree.event) {
+            _context.next = 16;
             break;
-          case 20:
-            _context.prev = 20;
-            _context.t0 = _context["catch"](0);
-            console.error('Caught error loading tree', _context.t0);
-          case 23:
-          case "end":
-            return _context.stop();
-        }
+          }
+          _context.next = 14;
+          return (0, _effects.call)(_apis.fetchPlatformItem, {
+            type: tree.platformItemType,
+            id: tree.platformItemId
+          });
+        case 14:
+          result = _context.sent;
+          platformItem = result.platformItem;
+        case 16:
+          _context.next = 18;
+          return (0, _effects.put)((0, _store.action)('TREE_LOADED', {
+            categories: categories,
+            kappSlug: webApiProps && webApiProps.kappSlug,
+            platformItem: platformItem,
+            treeKey: treeKey,
+            tree: (0, _models.deserializeTree)(tree),
+            webApi: webApiProps && (0, _models.deserializeWebApi)(webApi, webApiProps.kappSlug)
+          }));
+        case 18:
+          _context.next = 23;
+          break;
+        case 20:
+          _context.prev = 20;
+          _context.t0 = _context["catch"](0);
+          console.error('Caught error loading tree', _context.t0);
+        case 23:
+        case "end":
+          return _context.stop();
       }
     }, _callee, null, [[0, 20]]);
   })();
@@ -122,82 +117,80 @@ var getPlatformItemSlugs = function getPlatformItemSlugs(platformItem) {
   return /*#__PURE__*/(0, _regeneratorRuntime2["default"])().mark(function _callee2() {
     var newName, onError, onSave, overwrite, treeKey, _yield$select, kappSlug, lastSave, lastWebApi, platformItem, tree, webApi, name, sourceGroup, sourceName, _yield, error1, newTree, newWorkflow, _yield2, error2, error;
     return (0, _regeneratorRuntime2["default"])().wrap(function _callee2$(_context2) {
-      while (1) {
-        switch (_context2.prev = _context2.next) {
-          case 0:
-            _context2.prev = 0;
-            // because of the optimistic locking functionality newName / overwrite can
-            // be passed as options to the builder's save function
-            newName = payload.newName, onError = payload.onError, onSave = payload.onSave, overwrite = payload.overwrite, treeKey = payload.treeKey;
-            _context2.next = 4;
-            return (0, _effects.select)(function (state) {
-              return state.getIn(['trees', treeKey]);
-            });
-          case 4:
-            _yield$select = _context2.sent;
-            kappSlug = _yield$select.kappSlug;
-            lastSave = _yield$select.lastSave;
-            lastWebApi = _yield$select.lastWebApi;
-            platformItem = _yield$select.platformItem;
-            tree = _yield$select.tree;
-            webApi = _yield$select.webApi;
-            name = lastSave.name, sourceGroup = lastSave.sourceGroup, sourceName = lastSave.sourceName; // if a newName was passed we will be creating a new tree with the builder
-            // contents, otherwise just an update
-            // additionally, if the tree is a linked workflow then we call a core
-            // endpoint to create the workflow
-            _context2.next = 14;
-            return newName ? tree.event ? (0, _effects.call)(_apis.createWorkflow, (0, _objectSpread2["default"])({
-              workflow: (0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, (0, _models.serializeTree)(tree)), {}, {
-                name: newName
-              })
-            }, getPlatformItemSlugs(platformItem))) : (0, _effects.call)(_apis.createTree, {
-              tree: (0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, (0, _models.serializeTree)(tree, true)), {}, {
-                name: newName
-              })
-            }) : (0, _effects.call)(_apis.updateTree, {
-              name: name,
-              sourceGroup: sourceGroup,
-              sourceName: sourceName,
-              tree: (0, _models.serializeTree)(tree, overwrite)
-            });
-          case 14:
-            _yield = _context2.sent;
-            error1 = _yield.error;
-            newTree = _yield.tree;
-            newWorkflow = _yield.workflow;
-            _context2.next = 20;
-            return webApi && !error1 ? (0, _effects.call)(_apis.updateWebApi, {
-              slug: lastWebApi.get('slug'),
-              kappSlug: kappSlug,
-              webApi: webApi
-            }) : {};
-          case 20:
-            _yield2 = _context2.sent;
-            error2 = _yield2.error;
-            error = error1 || error2; // dispatch the appropriate action based on the result of the call above
-            _context2.next = 25;
-            return (0, _effects.put)(error ? (0, _store.action)('TREE_SAVE_ERROR', {
-              treeKey: treeKey,
-              error: error.message || error,
-              onError: onError
-            }) : (0, _store.action)('TREE_SAVE_SUCCESS', {
-              previousTree: lastSave,
-              treeKey: treeKey,
-              tree: newTree || newWorkflow,
-              webApi: webApi,
-              onSave: onSave
-            }));
-          case 25:
-            _context2.next = 30;
-            break;
-          case 27:
-            _context2.prev = 27;
-            _context2.t0 = _context2["catch"](0);
-            console.error(_context2.t0);
-          case 30:
-          case "end":
-            return _context2.stop();
-        }
+      while (1) switch (_context2.prev = _context2.next) {
+        case 0:
+          _context2.prev = 0;
+          // because of the optimistic locking functionality newName / overwrite can
+          // be passed as options to the builder's save function
+          newName = payload.newName, onError = payload.onError, onSave = payload.onSave, overwrite = payload.overwrite, treeKey = payload.treeKey;
+          _context2.next = 4;
+          return (0, _effects.select)(function (state) {
+            return state.getIn(['trees', treeKey]);
+          });
+        case 4:
+          _yield$select = _context2.sent;
+          kappSlug = _yield$select.kappSlug;
+          lastSave = _yield$select.lastSave;
+          lastWebApi = _yield$select.lastWebApi;
+          platformItem = _yield$select.platformItem;
+          tree = _yield$select.tree;
+          webApi = _yield$select.webApi;
+          name = lastSave.name, sourceGroup = lastSave.sourceGroup, sourceName = lastSave.sourceName; // if a newName was passed we will be creating a new tree with the builder
+          // contents, otherwise just an update
+          // additionally, if the tree is a linked workflow then we call a core
+          // endpoint to create the workflow
+          _context2.next = 14;
+          return newName ? tree.event ? (0, _effects.call)(_apis.createWorkflow, (0, _objectSpread2["default"])({
+            workflow: (0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, (0, _models.serializeTree)(tree)), {}, {
+              name: newName
+            })
+          }, getPlatformItemSlugs(platformItem))) : (0, _effects.call)(_apis.createTree, {
+            tree: (0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, (0, _models.serializeTree)(tree, true)), {}, {
+              name: newName
+            })
+          }) : (0, _effects.call)(_apis.updateTree, {
+            name: name,
+            sourceGroup: sourceGroup,
+            sourceName: sourceName,
+            tree: (0, _models.serializeTree)(tree, overwrite)
+          });
+        case 14:
+          _yield = _context2.sent;
+          error1 = _yield.error;
+          newTree = _yield.tree;
+          newWorkflow = _yield.workflow;
+          _context2.next = 20;
+          return webApi && !error1 ? (0, _effects.call)(_apis.updateWebApi, {
+            slug: lastWebApi.get('slug'),
+            kappSlug: kappSlug,
+            webApi: webApi
+          }) : {};
+        case 20:
+          _yield2 = _context2.sent;
+          error2 = _yield2.error;
+          error = error1 || error2; // dispatch the appropriate action based on the result of the call above
+          _context2.next = 25;
+          return (0, _effects.put)(error ? (0, _store.action)('TREE_SAVE_ERROR', {
+            treeKey: treeKey,
+            error: error.message || error,
+            onError: onError
+          }) : (0, _store.action)('TREE_SAVE_SUCCESS', {
+            previousTree: lastSave,
+            treeKey: treeKey,
+            tree: newTree || newWorkflow,
+            webApi: webApi,
+            onSave: onSave
+          }));
+        case 25:
+          _context2.next = 30;
+          break;
+        case 27:
+          _context2.prev = 27;
+          _context2.t0 = _context2["catch"](0);
+          console.error(_context2.t0);
+        case 30:
+        case "end":
+          return _context2.stop();
       }
     }, _callee2, null, [[0, 27]]);
   })();
@@ -208,27 +201,25 @@ var getPlatformItemSlugs = function getPlatformItemSlugs(platformItem) {
     onError = _ref3$payload.onError;
   return /*#__PURE__*/(0, _regeneratorRuntime2["default"])().mark(function _callee3() {
     return (0, _regeneratorRuntime2["default"])().wrap(function _callee3$(_context3) {
-      while (1) {
-        switch (_context3.prev = _context3.next) {
-          case 0:
-            _context3.prev = 0;
-            if (!(0, _lodashEs.isFunction)(onError)) {
-              _context3.next = 4;
-              break;
-            }
+      while (1) switch (_context3.prev = _context3.next) {
+        case 0:
+          _context3.prev = 0;
+          if (!(0, _lodashEs.isFunction)(onError)) {
             _context3.next = 4;
-            return (0, _effects.call)(onError, error);
-          case 4:
-            _context3.next = 9;
             break;
-          case 6:
-            _context3.prev = 6;
-            _context3.t0 = _context3["catch"](0);
-            console.error(_context3.t0);
-          case 9:
-          case "end":
-            return _context3.stop();
-        }
+          }
+          _context3.next = 4;
+          return (0, _effects.call)(onError, error);
+        case 4:
+          _context3.next = 9;
+          break;
+        case 6:
+          _context3.prev = 6;
+          _context3.t0 = _context3["catch"](0);
+          console.error(_context3.t0);
+        case 9:
+        case "end":
+          return _context3.stop();
       }
     }, _callee3, null, [[0, 6]]);
   })();
@@ -241,33 +232,31 @@ var getPlatformItemSlugs = function getPlatformItemSlugs(platformItem) {
   return /*#__PURE__*/(0, _regeneratorRuntime2["default"])().mark(function _callee4() {
     var tree;
     return (0, _regeneratorRuntime2["default"])().wrap(function _callee4$(_context4) {
-      while (1) {
-        switch (_context4.prev = _context4.next) {
-          case 0:
-            _context4.prev = 0;
-            if (!(0, _lodashEs.isFunction)(onSave)) {
-              _context4.next = 7;
-              break;
-            }
-            _context4.next = 4;
-            return (0, _effects.select)(function (state) {
-              return state.getIn(['trees', treeKey, 'tree']);
-            });
-          case 4:
-            tree = _context4.sent;
+      while (1) switch (_context4.prev = _context4.next) {
+        case 0:
+          _context4.prev = 0;
+          if (!(0, _lodashEs.isFunction)(onSave)) {
             _context4.next = 7;
-            return (0, _effects.call)(onSave, tree, previousTree);
-          case 7:
-            _context4.next = 12;
             break;
-          case 9:
-            _context4.prev = 9;
-            _context4.t0 = _context4["catch"](0);
-            console.error(_context4.t0);
-          case 12:
-          case "end":
-            return _context4.stop();
-        }
+          }
+          _context4.next = 4;
+          return (0, _effects.select)(function (state) {
+            return state.getIn(['trees', treeKey, 'tree']);
+          });
+        case 4:
+          tree = _context4.sent;
+          _context4.next = 7;
+          return (0, _effects.call)(onSave, tree, previousTree);
+        case 7:
+          _context4.next = 12;
+          break;
+        case 9:
+          _context4.prev = 9;
+          _context4.t0 = _context4["catch"](0);
+          console.error(_context4.t0);
+        case 12:
+        case "end":
+          return _context4.stop();
       }
     }, _callee4, null, [[0, 9]]);
   })();