@atlaskit/editor-core 185.3.0 → 185.4.0

package/dist/es2019/plugins/tasks-and-decisions/nodeviews/taskItem.js

@@ -15,6 +15,9 @@ class Task extends ReactNodeView {
         tr
       } = this.view.state;
       const nodePos = this.getPos();
+      if (typeof nodePos !== 'number') {
+        return false;
+      }
 
       // SetAttrsStep should be used to prevent task updates from being dropped when mapping task ticks
       // from a previous version of the document, such as a published page.
@@ -36,7 +39,11 @@ class Task extends ReactNodeView {
      */
     _defineProperty(this, "addListAnalyticsData", event => {
       try {
-        const resolvedPos = this.view.state.doc.resolve(this.getPos());
+        const nodePos = this.getPos();
+        if (typeof nodePos !== 'number') {
+          return false;
+        }
+        const resolvedPos = this.view.state.doc.resolve(nodePos);
         const position = resolvedPos.index();
         const listSize = resolvedPos.parent.childCount;
         const listLocalId = resolvedPos.parent.attrs.localId;

package/dist/es2019/version-wrapper.js

@@ -1,5 +1,5 @@
 export const name = "@atlaskit/editor-core";
-export const version = "185.3.0";
+export const version = "185.4.0";
 export const nextMajorVersion = () => {
   return [Number(version.split('.')[0]) + 1, 0, 0].join('.');
 };

package/dist/es2019/version.json

@@ -1,5 +1,5 @@
 {
 	"name": "@atlaskit/editor-core",
-	"version": "185.3.0",
+	"version": "185.4.0",
 	"sideEffects": false
 }

package/dist/esm/plugins/card/nodeviews/blockCard.js

@@ -41,10 +41,14 @@ export var BlockCardComponent = /*#__PURE__*/function (_React$PureComponent) {
       // don't dispatch immediately since we might be in the middle of
       // rendering a nodeview
       rafSchedule(function () {
-        return view.dispatch(registerCard({
+        var pos = getPos();
+        if (typeof pos !== 'number') {
+          return;
+        }
+        view.dispatch(registerCard({
           title: title,
           url: url,
-          pos: getPos()
+          pos: pos
         })(view.state.tr));
       })();
     });

package/dist/esm/plugins/card/nodeviews/datasource.js

@@ -0,0 +1,131 @@
+import _classCallCheck from "@babel/runtime/helpers/classCallCheck";
+import _createClass from "@babel/runtime/helpers/createClass";
+import _assertThisInitialized from "@babel/runtime/helpers/assertThisInitialized";
+import _inherits from "@babel/runtime/helpers/inherits";
+import _possibleConstructorReturn from "@babel/runtime/helpers/possibleConstructorReturn";
+import _getPrototypeOf from "@babel/runtime/helpers/getPrototypeOf";
+import _defineProperty from "@babel/runtime/helpers/defineProperty";
+function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
+function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { _defineProperty(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
+function _createSuper(Derived) { var hasNativeReflectConstruct = _isNativeReflectConstruct(); return function _createSuperInternal() { var Super = _getPrototypeOf(Derived), result; if (hasNativeReflectConstruct) { var NewTarget = _getPrototypeOf(this).constructor; result = Reflect.construct(Super, arguments, NewTarget); } else { result = Super.apply(this, arguments); } return _possibleConstructorReturn(this, result); }; }
+function _isNativeReflectConstruct() { if (typeof Reflect === "undefined" || !Reflect.construct) return false; if (Reflect.construct.sham) return false; if (typeof Proxy === "function") return true; try { Boolean.prototype.valueOf.call(Reflect.construct(Boolean, [], function () {})); return true; } catch (e) { return false; } }
+/** @jsx jsx */
+import React from 'react';
+import PropTypes from 'prop-types';
+import { css, jsx } from '@emotion/react';
+import ReactNodeView from '@atlaskit/editor-common/react-node-view';
+import { DatasourceTableView } from '@atlaskit/link-datasource';
+var containerStyles = css({
+  height: '500px',
+  overflow: 'auto'
+});
+export var DatasourceComponent = /*#__PURE__*/function (_React$PureComponent) {
+  _inherits(DatasourceComponent, _React$PureComponent);
+  var _super = _createSuper(DatasourceComponent);
+  function DatasourceComponent(props) {
+    var _this;
+    _classCallCheck(this, DatasourceComponent);
+    _this = _super.call(this, props);
+    _defineProperty(_assertThisInitialized(_this), "getPosSafely", function () {
+      var getPos = _this.props.getPos;
+      if (!getPos || typeof getPos === 'boolean') {
+        return;
+      }
+      try {
+        return getPos();
+      } catch (e) {
+        // Can blow up in rare cases, when node has been removed.
+      }
+    });
+    _defineProperty(_assertThisInitialized(_this), "getDatasource", function () {
+      return _this.props.node.attrs.datasource;
+    });
+    _defineProperty(_assertThisInitialized(_this), "getTableView", function (datasource) {
+      var views = (datasource || _this.getDatasource()).views;
+      return views.find(function (view) {
+        return view.type === 'table';
+      }) || undefined;
+    });
+    _defineProperty(_assertThisInitialized(_this), "handleColumnChange", function (columnKeys) {
+      var _this$props$view = _this.props.view,
+        state = _this$props$view.state,
+        dispatch = _this$props$view.dispatch;
+      var pos = _this.getPosSafely();
+      if (pos === undefined) {
+        return;
+      }
+      var attrs = _this.props.node.attrs;
+      var views = [{
+        type: 'table',
+        properties: {
+          columns: columnKeys.map(function (key) {
+            return {
+              key: key
+            };
+          })
+        }
+      }];
+      var tr = state.tr.setNodeMarkup(pos, undefined, _objectSpread(_objectSpread({}, attrs), {}, {
+        datasource: _objectSpread(_objectSpread({}, attrs.datasource), {}, {
+          views: views
+        })
+      }));
+      tr.setMeta('scrollIntoView', false);
+      dispatch(tr);
+    });
+    return _this;
+  }
+  _createClass(DatasourceComponent, [{
+    key: "render",
+    value: function render() {
+      var cardContext = this.context.contextAdapter ? this.context.contextAdapter.card : undefined;
+      var datasource = this.getDatasource();
+      var tableView = this.getTableView();
+      if (tableView) {
+        var _tableView$properties;
+        var visibleColumnKeys = (_tableView$properties = tableView.properties) === null || _tableView$properties === void 0 ? void 0 : _tableView$properties.columns.map(function (_ref) {
+          var key = _ref.key;
+          return key;
+        });
+
+        // [WS-2307]: we only render card wrapped into a Provider when the value is ready
+        if (cardContext && cardContext.value) {
+          return jsx(cardContext.Provider, {
+            value: cardContext.value
+          }, jsx(DatasourceTableView, {
+            datasourceId: datasource.id,
+            parameters: datasource.parameters,
+            visibleColumnKeys: visibleColumnKeys,
+            onVisibleColumnKeysChange: this.handleColumnChange
+          }));
+        }
+      }
+      return null;
+    }
+  }]);
+  return DatasourceComponent;
+}(React.PureComponent);
+_defineProperty(DatasourceComponent, "contextTypes", {
+  contextAdapter: PropTypes.object
+});
+export var Datasource = /*#__PURE__*/function (_ReactNodeView) {
+  _inherits(Datasource, _ReactNodeView);
+  var _super2 = _createSuper(Datasource);
+  function Datasource() {
+    _classCallCheck(this, Datasource);
+    return _super2.apply(this, arguments);
+  }
+  _createClass(Datasource, [{
+    key: "render",
+    value: function render() {
+      return jsx("div", {
+        css: containerStyles
+      }, jsx(DatasourceComponent, {
+        node: this.node,
+        view: this.view,
+        getPos: this.getPos
+      }));
+    }
+  }]);
+  return Datasource;
+}(ReactNodeView);

package/dist/esm/plugins/card/nodeviews/inlineCard.js

@@ -38,10 +38,15 @@ export var InlineCardComponent = /*#__PURE__*/function (_React$PureComponent) {
       // don't dispatch immediately since we might be in the middle of
       // rendering a nodeview
       rafSchedule(function () {
-        return view.dispatch(registerCard({
+        // prosemirror-bump-fix
+        var pos = getPos();
+        if (typeof pos !== 'number') {
+          return;
+        }
+        view.dispatch(registerCard({
           title: title,
           url: url,
-          pos: getPos()
+          pos: pos
         })(view.state.tr));
       })();
     });

package/dist/esm/plugins/card/pm-plugins/main.js

@@ -10,6 +10,7 @@ import reducer from './reducers';
 import { handleProvider, resolveWithProvider } from './util/resolve';
 import { getNewRequests, getPluginState, getPluginStateWithUpdatedPos } from './util/state';
 import { createAnalyticsQueue, eventsFromTransaction } from './analytics';
+import { Datasource } from '../nodeviews/datasource';
 export { pluginKey } from './plugin-key';
 export var createPlugin = function createPlugin(options, pluginInjectionApi) {
   return function (pmPluginFactoryParams) {
@@ -117,6 +118,9 @@ export var createPlugin = function createPlugin(options, pluginInjectionApi) {
               showServerActions: showServerActions
             };
             var hasIntlContext = true;
+            if (node.attrs.datasource) {
+              return new Datasource(node, view, getPos, portalProviderAPI, eventDispatcher, undefined, undefined, true, undefined, hasIntlContext).init();
+            }
             return new BlockCard(node, view, getPos, portalProviderAPI, eventDispatcher, reactComponentProps, undefined, true, undefined, hasIntlContext).init();
           },
           embedCard: function embedCard(node, view, getPos) {

package/dist/esm/plugins/date/nodeviews/date.js

@@ -12,10 +12,11 @@ export function DateNodeView(props) {
     selection = _props$view$state.selection,
     getPos = props.getPos;
   var intl = useIntl();
+  var pos = typeof getPos === 'function' ? getPos() : undefined;
 
   // We fall back to selection.$from even though it does not cover all use cases
   // eg. upon Editor init, selection is at the start, not at the Date node
-  var $nodePos = typeof getPos === 'function' ? doc.resolve(getPos()) : selection.$from;
+  var $nodePos = typeof pos === 'number' ? doc.resolve(pos) : selection.$from;
   var parent = $nodePos.parent;
   var withinIncompleteTask = parent.type === schema.nodes.taskItem && parent.attrs.state !== 'DONE';
   var color = withinIncompleteTask && isPastDate(timestamp) ? 'red' : undefined;

package/dist/esm/plugins/expand/nodeviews/index.js

@@ -84,6 +84,10 @@ export var ExpandNodeView = /*#__PURE__*/function () {
       return _this.featureFlags && !!_this.featureFlags.interactiveExpand;
     });
     _defineProperty(this, "handleClick", function (event) {
+      var pos = _this.getPos();
+      if (typeof pos !== 'number') {
+        return;
+      }
       var target = event.target;
       var _this$view2 = _this.view,
         state = _this$view2.state,
@@ -99,7 +103,7 @@ export var ExpandNodeView = /*#__PURE__*/function () {
         if (_this.view.dom instanceof HTMLElement) {
           _this.view.dom.blur();
         }
-        toggleExpandExpanded(_this.getPos(), _this.node.type)(state, dispatch);
+        toggleExpandExpanded(pos, _this.node.type)(state, dispatch);
         return;
       }
       if (target === _this.input) {
@@ -109,13 +113,17 @@ export var ExpandNodeView = /*#__PURE__*/function () {
       }
     });
     _defineProperty(this, "handleInput", function (event) {
+      var pos = _this.getPos();
+      if (typeof pos !== 'number') {
+        return;
+      }
       var target = event.target;
       if (target === _this.input) {
         event.stopPropagation();
         var _this$view3 = _this.view,
           state = _this$view3.state,
           dispatch = _this$view3.dispatch;
-        updateExpandTitle(target.value, _this.getPos(), _this.node.type)(state, dispatch);
+        updateExpandTitle(target.value, pos, _this.node.type)(state, dispatch);
       }
     });
     _defineProperty(this, "handleFocus", function (event) {
@@ -148,6 +156,10 @@ export var ExpandNodeView = /*#__PURE__*/function () {
       if (!_this.input) {
         return;
       }
+      var pos = _this.getPos();
+      if (typeof pos !== 'number') {
+        return;
+      }
       var _this$input = _this.input,
         selectionStart = _this$input.selectionStart,
         selectionEnd = _this$input.selectionEnd;
@@ -157,15 +169,19 @@ export var ExpandNodeView = /*#__PURE__*/function () {
       var state = _this.view.state;
       var expandNode = _this.node;
       if (expandNode && isEmptyNode(state.schema)(expandNode)) {
-        deleteExpandAtPos(_this.getPos(), expandNode)(state, _this.view.dispatch);
+        deleteExpandAtPos(pos, expandNode)(state, _this.view.dispatch);
       }
     });
     _defineProperty(this, "toggleExpand", function () {
+      var pos = _this.getPos();
+      if (typeof pos !== 'number') {
+        return;
+      }
       if (_this.isAllowInteractiveExpandEnabled()) {
         var _this$view4 = _this.view,
           state = _this$view4.state,
           dispatch = _this$view4.dispatch;
-        toggleExpandExpanded(_this.getPos(), _this.node.type)(state, dispatch);
+        toggleExpandExpanded(pos, _this.node.type)(state, dispatch);
       }
     });
     _defineProperty(this, "moveToOutsideOfTitle", function (event) {
@@ -203,6 +219,10 @@ export var ExpandNodeView = /*#__PURE__*/function () {
       if (!_this.input) {
         return;
       }
+      var pos = _this.getPos();
+      if (typeof pos !== 'number') {
+        return;
+      }
       var _this$input2 = _this.input,
         value = _this$input2.value,
         selectionStart = _this$input2.selectionStart,
@@ -213,13 +233,17 @@ export var ExpandNodeView = /*#__PURE__*/function () {
           dispatch = _this$view6.dispatch;
         event.preventDefault();
         _this.view.focus();
-        dispatch(state.tr.setSelection(new GapCursorSelection(state.doc.resolve(_this.node.nodeSize + _this.getPos()), Side.RIGHT)));
+        dispatch(state.tr.setSelection(new GapCursorSelection(state.doc.resolve(_this.node.nodeSize + pos), Side.RIGHT)));
       }
     });
     _defineProperty(this, "setLeftGapCursor", function (event) {
       if (!_this.input) {
         return;
       }
+      var pos = _this.getPos();
+      if (typeof pos !== 'number') {
+        return;
+      }
       var _this$input3 = _this.input,
         selectionStart = _this$input3.selectionStart,
         selectionEnd = _this$input3.selectionEnd;
@@ -229,13 +253,17 @@ export var ExpandNodeView = /*#__PURE__*/function () {
           state = _this$view7.state,
           dispatch = _this$view7.dispatch;
         _this.view.focus();
-        dispatch(state.tr.setSelection(new GapCursorSelection(state.doc.resolve(_this.getPos()), Side.LEFT)));
+        dispatch(state.tr.setSelection(new GapCursorSelection(state.doc.resolve(pos), Side.LEFT)));
       }
     });
     _defineProperty(this, "handleArrowRightFromTitle", function (event) {
       if (!_this.input) {
         return;
       }
+      var pos = _this.getPos();
+      if (typeof pos !== 'number') {
+        return;
+      }
       var _this$input4 = _this.input,
         value = _this$input4.value,
         selectionStart = _this$input4.selectionStart,
@@ -246,13 +274,17 @@ export var ExpandNodeView = /*#__PURE__*/function () {
           state = _this$view8.state,
           dispatch = _this$view8.dispatch;
         _this.view.focus();
-        setSelectionRelativeToNode(RelativeSelectionPos.End, NodeSelection.create(state.doc, _this.getPos()))(state, dispatch);
+        setSelectionRelativeToNode(RelativeSelectionPos.End, NodeSelection.create(state.doc, pos))(state, dispatch);
       }
     });
     _defineProperty(this, "handleArrowLeftFromTitle", function (event) {
       if (!_this.input) {
         return;
       }
+      var pos = _this.getPos();
+      if (typeof pos !== 'number') {
+        return;
+      }
       var _this$input5 = _this.input,
         selectionStart = _this$input5.selectionStart,
         selectionEnd = _this$input5.selectionEnd;
@@ -266,9 +298,9 @@ export var ExpandNodeView = /*#__PURE__*/function () {
         // selectionRelativeToNode is undefined when user clicked to select node, then hit left to get focus in title
         // This is a special case where we want to bypass node selection and jump straight to gap cursor
         if (getSelectionPluginState(state).selectionRelativeToNode === undefined) {
-          setSelectionRelativeToNode(undefined, new GapCursorSelection(state.doc.resolve(_this.getPos()), Side.LEFT))(state, dispatch);
+          setSelectionRelativeToNode(undefined, new GapCursorSelection(state.doc.resolve(pos), Side.LEFT))(state, dispatch);
         } else {
-          setSelectionRelativeToNode(RelativeSelectionPos.Start, NodeSelection.create(state.doc, _this.getPos()))(state, dispatch);
+          setSelectionRelativeToNode(RelativeSelectionPos.Start, NodeSelection.create(state.doc, pos))(state, dispatch);
         }
       }
     });
@@ -277,7 +309,6 @@ export var ExpandNodeView = /*#__PURE__*/function () {
       dom = _DOMSerializer$render.dom,
       contentDOM = _DOMSerializer$render.contentDOM;
     this.getPos = getPos;
-    this.pos = getPos();
     this.view = view;
     this.node = node;
     this.dom = dom;

package/dist/esm/plugins/extension/ui/Extension/Extension/index.js

@@ -30,7 +30,8 @@ function ExtensionWithPluginState(props) {
   var isMobile = editorAppearance === 'mobile';
   var hasChildren = !!children;
   var removeBorder = hideFrame && !isMobile && !hasBody || false;
-  var isTopLevelNode = props.getPos && !isNaN(props.getPos()) && props.view.state.doc.resolve(props.getPos()).depth === 0;
+  var pos = typeof props.getPos === 'function' ? props.getPos() : undefined;
+  var isTopLevelNode = typeof pos !== 'undefined' && !isNaN(pos) && props.view.state.doc.resolve(pos).depth === 0;
   var shouldBreakout =
   // Extension should breakout when the layout is set to 'full-width' or 'wide'.
   ['full-width', 'wide'].includes(node.attrs.layout) &&

package/dist/esm/plugins/media/commands/captions.js

@@ -19,6 +19,9 @@ export var selectCaptionFromMediaSinglePos = function selectCaptionFromMediaSing
 };
 export var insertAndSelectCaptionFromMediaSinglePos = function insertAndSelectCaptionFromMediaSinglePos(mediaSingleNodePos, mediaSingleNode) {
   return function (state, dispatch) {
+    if (typeof mediaSingleNodePos !== 'number') {
+      return false;
+    }
     var tr = state.tr;
 
     // node should have one child, media

package/dist/esm/plugins/media/commands/helpers.js

@@ -41,7 +41,7 @@ export var updateAllMediaSingleNodesAttrs = function updateAllMediaSingleNodesAt
     var validMediaNodePositions = mediaNodes.reduce(function (acc, _ref) {
       var getPos = _ref.getPos;
       var pos = getPos();
-      if (isNaN(pos) || typeof pos === 'number' && !isMediaNode(pos, state)) {
+      if (typeof pos !== 'number' || isNaN(pos) || typeof pos === 'number' && !isMediaNode(pos, state)) {
         return acc;
       }
       acc.push(pos);
@@ -64,7 +64,7 @@ export var updateAllMediaSingleNodesAttrs = function updateAllMediaSingleNodesAt
 export var updateCurrentMediaNodeAttrs = function updateCurrentMediaNodeAttrs(attrs, mediaNode) {
   return function (state, dispatch) {
     var pos = mediaNode.getPos();
-    if (isNaN(pos) || typeof pos === 'number' && !isMediaNode(pos, state)) {
+    if (typeof pos !== 'number' || isNaN(pos) || typeof pos === 'number' && !isMediaNode(pos, state)) {
       return false;
     }
     var tr = state.tr;
@@ -85,7 +85,7 @@ export var updateMediaSingleNodeAttrs = function updateMediaSingleNodeAttrs(id,
     }
     var tr = state.tr;
     var pos = mediaNodeWithPos.getPos();
-    if (!isMediaNode(pos, state)) {
+    if (typeof pos !== 'number' || !isMediaNode(pos, state)) {
       return false;
     }
     tr.step(new SetAttrsStep(pos, attrs)).setMeta('addToHistory', false);

package/dist/esm/plugins/media/nodeviews/mediaGroup.js

@@ -25,7 +25,11 @@ import { injectIntl } from 'react-intl-next';
 import { messages } from './messages';
 import { useSharedPluginState } from '@atlaskit/editor-common/hooks';
 var isMediaGroupSelectedFromProps = function isMediaGroupSelectedFromProps(props) {
-  return isNodeSelectedOrInRange(props.anchorPos, props.headPos, props.getPos(), props.node.nodeSize);
+  var pos = props.getPos();
+  if (typeof pos !== 'number') {
+    return false;
+  }
+  return isNodeSelectedOrInRange(props.anchorPos, props.headPos, pos, props.node.nodeSize);
 };
 var hasSelectionChanged = function hasSelectionChanged(oldProps, newProps) {
   if (isMediaGroupSelectedFromProps(oldProps) !== isMediaGroupSelectedFromProps(newProps)) {
@@ -66,7 +70,16 @@ var MediaGroup = /*#__PURE__*/function (_React$Component) {
       _this.mediaNodes = [];
       node.forEach(function (item, childOffset) {
         var getPos = function getPos() {
-          return props.getPos() + childOffset + 1;
+          var pos = props.getPos();
+          if (typeof pos !== 'number') {
+            // That may seems weird, but the previous type wasn't match with the real ProseMirror code. And a lot of Media API was built expecting a number
+            // Because the original code would return NaN on runtime
+            // We are just make it explict now.
+            // We may run a deep investagation on Media code to figure out a better fix. But, for now, we want to keep the current behavior.
+            // TODO: ED-13910 prosemirror-bump leftovers
+            return NaN;
+          }
+          return pos + childOffset + 1;
         };
         _this.mediaNodes.push(item);
         if (updatedAttrs) {
@@ -108,7 +121,16 @@ var MediaGroup = /*#__PURE__*/function (_React$Component) {
       var items = _this.mediaNodes.map(function (item, idx) {
         // We declared this to get a fresh position every time
         var getNodePos = function getNodePos() {
-          return getPos() + idx + 1;
+          var pos = getPos();
+          if (typeof pos !== 'number') {
+            // That may seems weird, but the previous type wasn't match with the real ProseMirror code. And a lot of Media API was built expecting a number
+            // Because the original code would return NaN on runtime
+            // We are just make it explict now.
+            // We may run a deep investagation on Media code to figure out a better fix. But, for now, we want to keep the current behavior.
+            // TODO: ED-13910 prosemirror-bump leftovers
+            return NaN;
+          }
+          return pos + idx + 1;
         };
 
         // Media Inline creates a floating toolbar with the same options, excludes these options if enabled
@@ -175,7 +197,16 @@ var MediaGroup = /*#__PURE__*/function (_React$Component) {
                   isMediaSingle: false
                 });
                 getPos = function getPos() {
-                  return _this2.props.getPos() + 1;
+                  var pos = _this2.props.getPos();
+                  if (typeof pos !== 'number') {
+                    // That may seems weird, but the previous type wasn't match with the real ProseMirror code. And a lot of Media API was built expecting a number
+                    // Because the original code would return NaN on runtime
+                    // We are just make it explict now.
+                    // We may run a deep investagation on Media code to figure out a better fix. But, for now, we want to keep the current behavior.
+                    // TODO: ED-13910 prosemirror-bump leftovers
+                    return NaN;
+                  }
+                  return pos + 1;
                 };
                 contextId = mediaNodeUpdater.getNodeContextId();
                 if (contextId) {

package/dist/esm/plugins/media/nodeviews/mediaNodeUpdater.js

@@ -216,7 +216,7 @@ export var MediaNodeUpdater = /*#__PURE__*/function () {
     })));
     _defineProperty(this, "uploadExternalMedia", /*#__PURE__*/function () {
       var _ref8 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7(getPos) {
-        var node, mediaProvider, uploadMediaClientConfig, mediaClient, collection, uploader, uploadableFileUpfrontIds, dimensions;
+        var node, mediaProvider, uploadMediaClientConfig, mediaClient, collection, uploader, uploadableFileUpfrontIds, dimensions, pos;
         return _regeneratorRuntime.wrap(function _callee7$(_context7) {
           while (1) switch (_context7.prev = _context7.next) {
             case 0:
@@ -226,7 +226,7 @@ export var MediaNodeUpdater = /*#__PURE__*/function () {
             case 3:
               mediaProvider = _context7.sent;
               if (!(node && mediaProvider)) {
-                _context7.next = 21;
+                _context7.next = 24;
                 break;
               }
               uploadMediaClientConfig = mediaProvider.uploadMediaClientConfig;
@@ -244,17 +244,24 @@ export var MediaNodeUpdater = /*#__PURE__*/function () {
             case 13:
               uploader = _context7.sent;
               uploadableFileUpfrontIds = uploader.uploadableFileUpfrontIds, dimensions = uploader.dimensions;
-              replaceExternalMedia(getPos() + 1, {
+              pos = getPos();
+              if (!(typeof pos !== 'number')) {
+                _context7.next = 18;
+                break;
+              }
+              return _context7.abrupt("return");
+            case 18:
+              replaceExternalMedia(pos + 1, {
                 id: uploadableFileUpfrontIds.id,
                 collection: collection,
                 height: dimensions.height,
                 width: dimensions.width,
                 occurrenceKey: uploadableFileUpfrontIds.occurrenceKey
               })(_this.props.view.state, _this.props.view.dispatch);
-              _context7.next = 21;
+              _context7.next = 24;
               break;
-            case 18:
-              _context7.prev = 18;
+            case 21:
+              _context7.prev = 21;
               _context7.t0 = _context7["catch"](10);
               //keep it as external media
               if (_this.props.dispatchAnalyticsEvent) {
@@ -264,11 +271,11 @@ export var MediaNodeUpdater = /*#__PURE__*/function () {
                   eventType: EVENT_TYPE.OPERATIONAL
                 });
               }
-            case 21:
+            case 24:
             case "end":
               return _context7.stop();
           }
-        }, _callee7, null, [[10, 18]]);
+        }, _callee7, null, [[10, 21]]);
       }));
       return function (_x3) {
         return _ref8.apply(this, arguments);
@@ -360,7 +367,7 @@ export var MediaNodeUpdater = /*#__PURE__*/function () {
     })));
     _defineProperty(this, "copyNodeFromBlobUrl", /*#__PURE__*/function () {
       var _ref11 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(getPos) {
-        var attrs, url, mediaAttrs, mediaProvider, currentCollectionName, contextId, id, collection, height, width, mimeType, name, size, uploadMediaClientConfig, mediaClient, auth, source, destination, mediaFile;
+        var attrs, url, mediaAttrs, mediaProvider, currentCollectionName, contextId, id, collection, height, width, mimeType, name, size, uploadMediaClientConfig, mediaClient, auth, source, destination, mediaFile, pos;
         return _regeneratorRuntime.wrap(function _callee10$(_context10) {
           while (1) switch (_context10.prev = _context10.next) {
             case 0:
@@ -419,7 +426,14 @@ export var MediaNodeUpdater = /*#__PURE__*/function () {
               return mediaClient.file.copyFile(source, destination);
             case 25:
               mediaFile = _context10.sent;
-              replaceExternalMedia(getPos() + 1, {
+              pos = getPos();
+              if (!(typeof pos !== 'number')) {
+                _context10.next = 29;
+                break;
+              }
+              return _context10.abrupt("return");
+            case 29:
+              replaceExternalMedia(pos + 1, {
                 id: mediaFile.id,
                 collection: currentCollectionName,
                 height: height,
@@ -428,7 +442,7 @@ export var MediaNodeUpdater = /*#__PURE__*/function () {
                 __fileMimeType: mimeType,
                 __fileSize: size
               })(_this.props.view.state, _this.props.view.dispatch);
-            case 27:
+            case 30:
             case "end":
               return _context10.stop();
           }

package/dist/esm/plugins/media/nodeviews/mediaNodeView/media.js

@@ -55,10 +55,14 @@ export var MediaNode = /*#__PURE__*/function (_Component) {
       _this.selectMediaSingle(event);
     });
     _defineProperty(_assertThisInitialized(_this), "selectMediaSingle", function (event) {
+      var propPos = _this.props.getPos();
+      if (typeof propPos !== 'number') {
+        return;
+      }
+
       // We need to call "stopPropagation" here in order to prevent the browser from navigating to
       // another URL if the media node is wrapped in a link mark.
       event.stopPropagation();
-      var propPos = _this.props.getPos();
       var state = _this.props.view.state;
       if (event.shiftKey) {
         // don't select text if there is current selection in a table (as this would override selected cells)

package/dist/esm/plugins/media/nodeviews/mediaSingle.js

@@ -169,10 +169,14 @@ var MediaSingleNode = /*#__PURE__*/function (_Component) {
     }());
     _defineProperty(_assertThisInitialized(_this), "selectMediaSingle", function (_ref3) {
       var event = _ref3.event;
+      var propPos = _this.props.getPos();
+      if (typeof propPos !== 'number') {
+        return;
+      }
+
       // We need to call "stopPropagation" here in order to prevent the browser from navigating to
       // another URL if the media node is wrapped in a link mark.
       event.stopPropagation();
-      var propPos = _this.props.getPos();
       var state = _this.props.view.state;
       if (event.shiftKey) {
         // don't select text if there is current selection in a table (as this would override selected cells)
@@ -244,6 +248,9 @@ var MediaSingleNode = /*#__PURE__*/function (_Component) {
       insertAndSelectCaptionFromMediaSinglePos(getPos(), node)(view.state, view.dispatch);
     });
     _defineProperty(_assertThisInitialized(_this), "getLineLength", function (view, pos) {
+      if (typeof pos !== 'number') {
+        return null;
+      }
       if (isRichMediaInsideOfBlockNode(view, pos)) {
         var $pos = view.state.doc.resolve(pos);
         var domNode = view.nodeDOM($pos.pos);