fix(document load): possible race condition

When two people create a session roughly at the same time
they will both be handed the content of the file
as no steps have been pushed yet.

Use `setContent` rather than loading the document with the content
as it is idempotent.

Signed-off-by: Max <max@nextcloud.com>

Author: max-nextcloud

src/EditorFactory.js

@@ -47,7 +47,7 @@ const loadSyntaxHighlight = async (language) => {
 	}
 }
 
-const createEditor = ({ content, onCreate, onUpdate = () => {}, extensions, enableRichEditing, session, relativePath }) => {
+const createEditor = ({ onCreate, onUpdate = () => {}, extensions, enableRichEditing, session, relativePath }) => {
 	let defaultExtensions
 	if (enableRichEditing) {
 		defaultExtensions = [
@@ -68,7 +68,6 @@ const createEditor = ({ content, onCreate, onUpdate = () => {}, extensions, enab
 	}
 
 	return new Editor({
-		content: content + '<p/>',
 		onCreate,
 		onUpdate,
 		editorProps: {

src/components/Editor.vue

@@ -347,7 +347,7 @@ export default {
 		parseContent(documentSource) {
 			return !this.isRichEditor
 				? `<pre>${escapeHtml(documentSource)}</pre>`
-				: markdownit.render(documentSource)
+				: markdownit.render(documentSource) + '<p/>'
 		},
 
 		initSession() {
@@ -495,7 +495,6 @@ export default {
 					this.$editor = createEditor({
 						relativePath: this.relativePath,
 						session,
-						content: documentState ? '' : this.parseContent(documentSource),
 						onCreate: ({ editor }) => {
 							this.$syncService.startSync()
 						},
@@ -529,7 +528,9 @@ export default {
 						enableRichEditing: this.isRichEditor,
 					})
 					this.hasEditor = true
-
+					if (!documentState && documentSource) {
+						this.setContent(documentSource)
+					}
 					this.listenEditorEvents()
 
 				})