Merge pull request #4589 from nextcloud/backport/4588/stable27

[stable27] fix(mentions): encode user id in mentions URLs
nextcloud · Jul 27, 2023 · 5760a68 · 5760a68
2 parents 9dd4179 + 7ebe8d7
commit 5760a68
Show file tree

Hide file tree

Showing 18 changed files with 34 additions and 26 deletions.
diff --git a/js/editor.js b/js/editor.js
diff --git a/js/editor.js.map b/js/editor.js.map
diff --git a/js/files-modal.js b/js/files-modal.js
diff --git a/js/files-modal.js.map b/js/files-modal.js.map
diff --git a/js/text-editors.js b/js/text-editors.js
diff --git a/js/text-editors.js.map b/js/text-editors.js.map
diff --git a/js/text-files.js b/js/text-files.js
diff --git a/js/text-files.js.map b/js/text-files.js.map
diff --git a/js/text-public.js b/js/text-public.js
diff --git a/js/text-public.js.map b/js/text-public.js.map
diff --git a/js/text-text.js b/js/text-text.js
diff --git a/js/text-text.js.map b/js/text-text.js.map
diff --git a/js/text-viewer.js b/js/text-viewer.js
diff --git a/js/text-viewer.js.map b/js/text-viewer.js.map
diff --git a/src/components/Editor.vue b/src/components/Editor.vue
@@ -713,7 +713,7 @@ export default {
 		 * @param {object} editor The Tiptap editor
 		 */
 		debugContent(editor) {
-			const proseMirrorMarkdown = this.$syncService.options.serialize(editor.state.doc)
+			const proseMirrorMarkdown = this.$syncService.serialize(editor.state.doc)
 			const markdownItHtml = markdownit.render(proseMirrorMarkdown)
 
 			logger.debug('markdown, serialized from editor state by prosemirror-markdown')

diff --git a/src/extensions/Mention.js b/src/extensions/Mention.js
@@ -10,7 +10,7 @@ export default TipTapMention.extend({
 				tag: 'span[data-type="user"]',
 				getAttrs: element => {
 					return {
-						id: element.getAttribute('data-id'),
+						id: decodeURIComponent(element.getAttribute('data-id')),
 						label: element.innerText || element.textContent || element.getAttribute('data-label'),
 					}
 				},
@@ -36,7 +36,7 @@ export default TipTapMention.extend({
 
 	toMarkdown(state, node) {
 		state.write(' ')
-		state.write(`@[${node.attrs.label}](mention://user/${node.attrs.id})`)
+		state.write(`@[${node.attrs.label}](mention://user/${encodeURIComponent(node.attrs.id)})`)
 		state.write(' ')
 	},
 })
diff --git a/src/tests/markdown.spec.js b/src/tests/markdown.spec.js
@@ -221,6 +221,7 @@ describe('Markdown serializer from html', () => {
 
 	test('mentions', () => {
 		expect(markdownThroughEditorHtml('<span class="mention" data-label="username" data-type="user" data-id="id">username</span>')).toBe(' @[username](mention://user/id) ')
+		expect(markdownThroughEditorHtml('<span class="mention" data-label="whitespace user" data-type="user" data-id="whitespace user">whitespace user</span>')).toBe(' @[whitespace user](mention://user/whitespace%20user) ')
 	})
 })
 

diff --git a/src/tests/markdownit.spec.js b/src/tests/markdownit.spec.js
@@ -39,8 +39,15 @@ describe('markdownit', () => {
 			</ul>
 			<ul class="contains-task-list" data-bullet="*">
 			<li class="task-list-item "><input class="task-list-item-checkbox" type="checkbox" disabled="" id="task-item-1" />task</li>
-			</ul>
-`))
+			</ul>`
+		))
+	})
+
+	it('renders mentions of users with escaped whitespace', () => {
+		const rendered = markdownit.render('@[whitespace user](mention://user/whitespace%20user)')
+		expect(stripIndent(rendered)).toBe(stripIndent(`
+			<p><span class="mention" data-type="user" data-id="whitespace%20user">whitespace user</span></p>`
+		))
 	})
 
 	describe('callouts', () => {