@seafile/sdoc-editor 0.1.166 → 0.1.167

package/dist/basic-sdk/editor/index.js

@@ -3,23 +3,18 @@ import _slicedToArray from "@babel/runtime/helpers/esm/slicedToArray";
 import React, { useMemo, useEffect, useState, useCallback, useImperativeHandle, forwardRef } from 'react';
 import deepCopy from 'deep-copy';
 import context from '../../context';
-import { MODE } from '../../constants';
 import CommonLoading from '../../components/common-loading';
 import { PAGE_EDIT_AREA_WIDTH } from '../constants';
 import { createDefaultEditor } from '../extension';
 import withNodeId from '../node-id';
 import { withSocketIO } from '../socket';
-import DiffViewer from '../views/diff-viewer';
 import SDocEditor from './sdoc-editor';
 import './index.css';
 var Editor = forwardRef(function (_ref, ref) {
-  var mode = _ref.mode,
-    document = _ref.document,
-    setDiffChanges = _ref.setDiffChanges;
+  var document = _ref.document;
   var editor = useMemo(function () {
     var defaultEditor = createDefaultEditor();
     var editorConfig = context.getEditorConfig();
-    defaultEditor.mode = MODE.EDITOR;
     var newEditor = withNodeId(withSocketIO(defaultEditor, {
       document: document,
       config: editorConfig
@@ -31,22 +26,14 @@ var Editor = forwardRef(function (_ref, ref) {
 
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);
-  var _useState = useState(document.children),
+  var _useState = useState(false),
     _useState2 = _slicedToArray(_useState, 2),
-    slateValue = _useState2[0],
-    _setSlateValue = _useState2[1];
-  var _useState3 = useState(false),
+    isLoading = _useState2[0],
+    _setLoading = _useState2[1];
+  var _useState3 = useState(document.children),
     _useState4 = _slicedToArray(_useState3, 2),
-    isLoading = _useState4[0],
-    _setLoading = _useState4[1];
-  var _useState5 = useState(null),
-    _useState6 = _slicedToArray(_useState5, 2),
-    diffContent = _useState6[0],
-    _setDiffContent = _useState6[1];
-  var _useState7 = useState(null),
-    _useState8 = _slicedToArray(_useState7, 2),
-    currentContent = _useState8[0],
-    setCurrentContent = _useState8[1];
+    slateValue = _useState4[0],
+    _setSlateValue = _useState4[1];
 
   // useMount: init socket connection
   useEffect(function () {
@@ -70,20 +57,6 @@ var Editor = forwardRef(function (_ref, ref) {
       setLoading: function setLoading(isLoading) {
         _setLoading(isLoading);
       },
-      setDiffContent: function setDiffContent(content) {
-        setCurrentContent(deepCopy(_objectSpread(_objectSpread({}, document), {}, {
-          children: slateValue
-        })));
-        _setDiffContent(content);
-      },
-      setEditorMode: function setEditorMode(mode) {
-        editor.cursors = {};
-        editor.selection = null;
-        if (mode === MODE.EDITOR) {
-          editor.operations = [];
-        }
-        editor.mode = mode;
-      },
       setSlateValue: function setSlateValue(document) {
         _setSlateValue(document.children);
       },
@@ -105,16 +78,6 @@ var Editor = forwardRef(function (_ref, ref) {
       className: "h-100 w-100 d-flex align-items-center justify-content-center"
     }, /*#__PURE__*/React.createElement(CommonLoading, null));
   }
-  if (mode === MODE.DIFF_VIEWER) {
-    return /*#__PURE__*/React.createElement(DiffViewer, {
-      showToolbar: true,
-      showOutline: true,
-      editor: editor,
-      currentContent: currentContent,
-      lastContent: diffContent,
-      didMountCallback: setDiffChanges
-    });
-  }
   return /*#__PURE__*/React.createElement(SDocEditor, {
     slateValue: slateValue,
     editor: editor,

package/dist/basic-sdk/socket/helpers.js

@@ -4,7 +4,6 @@ import { Editor, Operation } from '@seafile/slate';
 import { getNode } from '../extension/core';
 import * as OPERATION from '../node-id/constants';
 import { setCursor } from '../cursor/helper';
-import { MODE } from '../../constants';
 export var getNodePathById = function getNodePathById(rootNode, nodeId) {
   var path = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : [];
   if (rootNode.id === nodeId) return path;
@@ -253,7 +252,6 @@ export var reExecRevertOperationList = function reExecRevertOperationList(editor
   }
 };
 export var syncRemoteOperations = function syncRemoteOperations(editor, remoteOperations) {
-  if (editor.mode !== MODE.EDITOR) return;
   if (remoteOperations.length === 0) return;
   Editor.withoutNormalizing(editor, function () {
     for (var i = 0; i < remoteOperations.length; i++) {

package/dist/basic-sdk/socket/socket-manager.js

@@ -7,7 +7,7 @@ import { syncRemoteOperations, reExecRevertOperationList, revertOperationList, s
 import SocketClient from './socket-client';
 import { clientDebug, conflictDebug, serverDebug, stateDebug } from '../utils/debug';
 import { deleteCursor } from '../cursor/helper';
-import { EXTERNAL_EVENT, MODE } from '../../constants';
+import { EXTERNAL_EVENT } from '../../constants';
 
 //  idle --> sending --> conflict --> idle
 //       --> conflict --> idle
@@ -56,7 +56,6 @@ var SocketManager = /*#__PURE__*/_createClass(function SocketManager(editor, _do
     _this.sendOperations();
   };
   this.sendOperations = function () {
-    if (_this.editor.mode !== MODE.EDITOR) return;
     if (_this.state !== STATE.IDLE) return;
     stateDebug("State changed: ".concat(_this.state, " -> ").concat(STATE.SENDING));
     _this.state = STATE.SENDING;

package/dist/basic-sdk/socket/with-socket-io.js

@@ -1,4 +1,3 @@
-import { MODE } from '../../constants';
 import { generateCursorData } from '../cursor/helper';
 import EventBus from '../utils/event-bus';
 import SocketManager from './socket-manager';
@@ -22,7 +21,6 @@ var withSocketIO = function withSocketIO(editor, options) {
   newEditor.onChange = function () {
     var document = options.document,
       config = options.config;
-    if (newEditor.mode === MODE.DIFF_VIEWER) return;
     var operations = newEditor.operations;
     if (!newEditor.isRemote && operations.length > 0) {
       var isAllSetSelection = operations.every(function (operation) {
@@ -30,23 +28,18 @@ var withSocketIO = function withSocketIO(editor, options) {
       });
       var _socketManager = SocketManager.getInstance(newEditor, document, config);
       if (!isAllSetSelection) {
-        if (newEditor.mode === MODE.EDITOR) {
-          // get update content value operations
-          var updateOperations = operations.filter(function (operation) {
-            return operation.type !== 'set_selection';
-          });
-          _socketManager.onReceiveLocalOperations(updateOperations);
-        }
+        // get update content value operations
+        var updateOperations = operations.filter(function (operation) {
+          return operation.type !== 'set_selection';
+        });
+        _socketManager.onReceiveLocalOperations(updateOperations);
       }
-      if (newEditor.mode === MODE.EDITOR) {
-        _socketManager.sendCursorLocation(editor.selection);
-      }
-    }
-    if (newEditor.mode === MODE.EDITOR) {
-      // dispatch editor change event
-      var eventBus = EventBus.getInstance(newEditor, document, config);
-      eventBus.dispatch('change');
+      _socketManager.sendCursorLocation(editor.selection);
     }
+
+    // dispatch editor change event
+    var eventBus = EventBus.getInstance(newEditor, document, config);
+    eventBus.dispatch('change');
     onChange();
   };
   newEditor.rebaseContent = function (document, originFileVersion) {

package/dist/basic-sdk/views/index.js

@@ -1,3 +1,4 @@
-import DiffViewer from './diff-viewer';
 import SDocViewer from './sdoc-viewer';
-export { DiffViewer, SDocViewer };
+import DiffViewer from './sdoc-diff-viewer';
+import RevisionDiffViewer from './revision-diff-viewer';
+export { SDocViewer, DiffViewer, RevisionDiffViewer };

package/dist/basic-sdk/views/revision-diff-viewer.js

@@ -0,0 +1,44 @@
+import _slicedToArray from "@babel/runtime/helpers/esm/slicedToArray";
+import React, { useEffect, useState } from 'react';
+import context from '../../context';
+import { getDiff } from '../utils/diff';
+import SDocViewer from './sdoc-viewer';
+import Loading from '../../components/loading';
+import '../../assets/css/diff-viewer.css';
+var RevisionDiffViewer = function RevisionDiffViewer(_ref) {
+  var revisionContent = _ref.revisionContent,
+    didMountCallback = _ref.didMountCallback;
+  var _useState = useState(true),
+    _useState2 = _slicedToArray(_useState, 2),
+    isLoading = _useState2[0],
+    setIsLoading = _useState2[1];
+  var _useState3 = useState(null),
+    _useState4 = _slicedToArray(_useState3, 2),
+    diff = _useState4[0],
+    setDiff = _useState4[1];
+  useEffect(function () {
+    setIsLoading(true);
+    context.getSeadocOriginFileContent().then(function (res) {
+      var originContentString = res.data.content;
+      var originContent = JSON.parse(originContentString);
+      var diff = getDiff(revisionContent, originContent);
+      setDiff(diff);
+      didMountCallback && didMountCallback(diff);
+      setIsLoading(false);
+    }).catch(function (error) {
+      console.log('error');
+      setIsLoading(false);
+    });
+  }, [revisionContent, didMountCallback]);
+  if (isLoading) {
+    return /*#__PURE__*/React.createElement(Loading, null);
+  }
+  return /*#__PURE__*/React.createElement(SDocViewer, {
+    showToolbar: true,
+    showOutline: true,
+    document: {
+      children: diff.value
+    }
+  });
+};
+export default RevisionDiffViewer;

package/dist/basic-sdk/views/sdoc-diff-viewer.js

@@ -0,0 +1,21 @@
+import React, { useEffect } from 'react';
+import { getDiff } from '../utils/diff';
+import SDocViewer from './sdoc-viewer';
+import '../../assets/css/diff-viewer.css';
+var DiffViewer = function DiffViewer(_ref) {
+  var currentContent = _ref.currentContent,
+    lastContent = _ref.lastContent,
+    didMountCallback = _ref.didMountCallback;
+  var diff = getDiff(currentContent, lastContent);
+  useEffect(function () {
+    didMountCallback && didMountCallback(diff);
+  }, [didMountCallback, diff]);
+  return /*#__PURE__*/React.createElement(SDocViewer, {
+    showToolbar: false,
+    showOutline: false,
+    document: {
+      children: diff.value
+    }
+  });
+};
+export default DiffViewer;

package/dist/constants/index.js

@@ -14,11 +14,6 @@ export var EXTERNAL_EVENT = {
   REMOVE_DOCUMENT: 'remove_document',
   REMOVE_DOCUMENT_ERROR: 'remove_document_error'
 };
-export var MODE = {
-  DIFF_VIEWER: 'diff_viewer',
-  VIEWER: 'viewer',
-  EDITOR: 'editor'
-};
 export var TIP_TYPE = {
   DELETE_NO_CHANGES_REVISION: 'delete_no_changes_revision',
   MERGE: 'merge',

package/dist/pages/simple-editor.js

@@ -11,10 +11,11 @@ import { generateDefaultDocContent } from '../utils';
 import context from '../context';
 import ErrorBoundary from './error-boundary';
 import TipDialog from '../components/tip-dialog';
-import { EXTERNAL_EVENT, MODE, TIP_TYPE, REVISION_FIRST_LOAD_KEY, REVISION_FIRST_LOAD_VALUE } from '../constants';
+import { EXTERNAL_EVENT, TIP_TYPE, REVISION_FIRST_LOAD_KEY, REVISION_FIRST_LOAD_VALUE } from '../constants';
 import { getRebase, hasConflict } from '../basic-sdk/utils/rebase';
 import toaster from '../components/toast';
 import { EventBus, SDocEditor } from '../basic-sdk';
+import { RevisionDiffViewer } from '../basic-sdk/views';
 import '../assets/css/simple-editor.css';
 var SimpleEditor = function SimpleEditor(_ref) {
   var isStarred = _ref.isStarred,
@@ -36,12 +37,12 @@ var SimpleEditor = function SimpleEditor(_ref) {
     setShowChanges = _useState6[1];
   var _useState7 = useState(null),
     _useState8 = _slicedToArray(_useState7, 2),
-    document = _useState8[0],
-    setDocument = _useState8[1];
-  var _useState9 = useState(MODE.EDITOR),
+    revisionContent = _useState8[0],
+    setRevisionContent = _useState8[1];
+  var _useState9 = useState(null),
     _useState10 = _slicedToArray(_useState9, 2),
-    mode = _useState10[0],
-    setMode = _useState10[1];
+    document = _useState10[0],
+    setDocument = _useState10[1];
   var _useState11 = useState([]),
     _useState12 = _slicedToArray(_useState11, 2),
     changes = _useState12[0],
@@ -166,32 +167,11 @@ var SimpleEditor = function SimpleEditor(_ref) {
       errorCallback && errorCallback(errorMessage);
     });
   }, []);
-  var setDiffContent = useCallback(function () {
-    context.getSeadocOriginFileContent().then(function (res) {
-      var originContentString = res.data.content;
-      var originContent = JSON.parse(originContentString);
-      editorRef.current.setDiffContent(originContent);
-      editorRef.current.setLoading(false);
-    }).catch(function (error) {
-      console.log('error');
-    });
-  }, []);
   var toggleViewChanges = useCallback(function (isShowChanges) {
-    var revisionContent = editorRef.current.getSlateValue();
-    if (isShowChanges && hasConflict(revisionContent.children)) {
-      setTipType(TIP_TYPE.HAS_CONFLICT_BEFORE_VIEW_CHANGES);
-      setShowTip(true);
-      return;
-    }
-    editorRef.current.setLoading(true);
-    setShowChanges(isShowChanges);
-    var mode = isShowChanges ? MODE.DIFF_VIEWER : MODE.EDITOR;
-    setMode(mode);
-    editorRef.current.setEditorMode(mode);
     if (!isShowChanges) {
       loadDocument(function (document) {
         setDocument(document);
-        editorRef.current.setSlateValue(document);
+        setShowChanges(isShowChanges);
         editorRef.current.setLoading(false);
       }, function (errorMessage) {
         setErrorMessage(errorMessage);
@@ -199,10 +179,26 @@ var SimpleEditor = function SimpleEditor(_ref) {
       });
       return;
     }
-    setDiffContent();
 
+    // Get the contents of the current revision
+    var revisionContent = editorRef.current.getSlateValue();
+
+    // Prevent users from switching if document contains conflicting content
+    if (hasConflict(revisionContent.children)) {
+      setTipType(TIP_TYPE.HAS_CONFLICT_BEFORE_VIEW_CHANGES);
+      setShowTip(true);
+      return;
+    }
+
+    // Record the contents of the current revision
+    setRevisionContent(revisionContent);
+    setShowChanges(isShowChanges);
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [document, editorRef.current]);
+  var setDiffChanges = useCallback(function (diff) {
+    setChanges(diff.changes);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [isShowChanges]);
   var publishRevision = useCallback( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime().mark(function _callee() {
     return _regeneratorRuntime().wrap(function _callee$(_context) {
       while (1) switch (_context.prev = _context.next) {
@@ -260,12 +256,7 @@ var SimpleEditor = function SimpleEditor(_ref) {
           return _context.stop();
       }
     }, _callee);
-  })), [isShowChanges, mode]);
-  var setDiffChanges = useCallback(function (diff) {
-    setChanges(diff.changes);
-
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [isShowChanges]);
+  })), [isShowChanges]);
   var jumpToURL = useCallback(function (url) {
     window.location.href = url;
   }, []);
@@ -305,11 +296,13 @@ var SimpleEditor = function SimpleEditor(_ref) {
         context.updateSettings({
           'originFileVersion': origin_file_version
         });
+        // if revision diff view change to sdoc editor
+        if (isShowChanges) {
+          setShowChanges(false);
+        }
         editorRef.current.setLoading(true);
         editorRef.current.setSlateValue(mergeValue);
         editorRef.current.updateDocumentVersion(mergeValue);
-        setMode(MODE.EDITOR);
-        editorRef.current.setEditorMode(MODE.EDITOR);
         editorRef.current.setLoading(false);
         setShowTip(false);
       }).catch(function (error) {
@@ -341,11 +334,12 @@ var SimpleEditor = function SimpleEditor(_ref) {
     changes: changes,
     toggleViewChanges: toggleViewChanges,
     publishRevision: publishRevision
-  })), /*#__PURE__*/React.createElement(Content, null, /*#__PURE__*/React.createElement(SDocEditor, {
+  })), /*#__PURE__*/React.createElement(Content, null, !isShowChanges && /*#__PURE__*/React.createElement(SDocEditor, {
     ref: editorRef,
-    document: document,
-    mode: mode,
-    setDiffChanges: setDiffChanges
+    document: document
+  }), isShowChanges && /*#__PURE__*/React.createElement(RevisionDiffViewer, {
+    revisionContent: revisionContent,
+    didMountCallback: setDiffChanges
   }))), showTip && /*#__PURE__*/React.createElement(TipDialog, {
     toggle: closeTip,
     submit: tipSubmit,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@seafile/sdoc-editor",
-  "version": "0.1.166",
+  "version": "0.1.167",
   "private": false,
   "description": "This is a sdoc editor",
   "main": "dist/index.js",

package/dist/basic-sdk/views/diff-viewer.js

@@ -1,33 +0,0 @@
-import React, { useEffect } from 'react';
-import context from '../../context';
-import { getDiff } from '../utils/diff';
-import SDocViewer from './sdoc-viewer';
-import '../../assets/css/diff-viewer.css';
-var DiffViewer = function DiffViewer(_ref) {
-  var editor = _ref.editor,
-    currentContent = _ref.currentContent,
-    lastContent = _ref.lastContent,
-    didMountCallback = _ref.didMountCallback,
-    showToolbar = _ref.showToolbar,
-    showOutline = _ref.showOutline;
-  var diff = getDiff(currentContent, lastContent);
-  context.initSettings();
-  useEffect(function () {
-    didMountCallback && didMountCallback(diff);
-
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, []);
-  return /*#__PURE__*/React.createElement(SDocViewer, {
-    editor: editor,
-    showToolbar: showToolbar,
-    showOutline: showOutline,
-    document: {
-      children: diff.value
-    }
-  });
-};
-DiffViewer.defaultProps = {
-  showToolbar: false,
-  showOutline: false
-};
-export default DiffViewer;