Merge pull request #5469 from nextcloud/fix/set_guestname

fix(session): Fix setting a guest name
nextcloud · Mar 13, 2024 · f6c7c7f · f6c7c7f
2 parents f32d786 + b227be0
commit f6c7c7f
Show file tree

Hide file tree

Showing 6 changed files with 28 additions and 5 deletions.
diff --git a/cypress/e2e/share.spec.js b/cypress/e2e/share.spec.js
@@ -35,6 +35,7 @@ describe('Open test.md in viewer', function() {
 		cy.createFolder('folder')
 		cy.uploadFile('test.md', 'text/markdown', 'folder/test.md')
 		cy.uploadFile('test.md', 'text/markdown', 'folder/Readme.md')
+		cy.uploadFile('test.md', 'text/markdown', 'test3.md')
 		cy.uploadFile('test.md', 'text/markdown', 'test2.md')
 		cy.uploadFile('test.md', 'text/markdown')
 
@@ -122,6 +123,27 @@ describe('Open test.md in viewer', function() {
 			})
 	})
 
+	it('Opens the editor as guest', function() {
+		cy.shareFile('/test3.md')
+			.then((token) => {
+				cy.logout()
+				cy.visit(`/s/${token}`)
+			})
+			.then(() => {
+				cy.getEditor().should('be.visible')
+				cy.getContent()
+					.should('contain', 'Hello world')
+					.find('h2').should('contain', 'Hello world')
+
+				cy.intercept({ method: 'POST', url: '**/apps/text/public/session/*/session' }).as('updateSession')
+				cy.get('button.avatar-list').click()
+				cy.get('.guest-name-dialog input[type="text"]')
+					.type('someone{enter}')
+				cy.wait('@updateSession')
+					.its('response.body.guestName').should('eq', 'someone')
+			})
+	})
+
 	it('Share a file with download disabled shows an error', function() {
 		cy.shareFileToUser('test.md', recipient, {
 			attributes: '[{"scope":"permissions","key":"download","enabled":false}]',

diff --git a/src/components/Editor/GuestNameDialog.vue b/src/components/Editor/GuestNameDialog.vue
@@ -72,12 +72,12 @@ export default {
 		},
 	},
 	beforeMount() {
-		this.guestName = this.$syncService.session.guestName
+		this.guestName = this.$syncService.connection.session.guestName
 		this.updateBufferedGuestName()
 	},
 	methods: {
 		setGuestName() {
-			const previousGuestName = this.$syncService.session.guestName
+			const previousGuestName = this.$syncService.connection.session.guestName
 			this.$syncService.updateSession(this.guestName).then(() => {
 				localStorage.setItem('nick', this.guestName)
 				this.updateBufferedGuestName()

diff --git a/src/components/Editor/SessionList.vue b/src/components/Editor/SessionList.vue
@@ -108,7 +108,7 @@ export default {
 			},
 		},
 		participantsPopover() {
-			if (this.currentSession.guestName) {
+			if (this.currentSession?.guestName) {
 				return this.participantsWithoutCurrent
 			}
 			return this.participants

diff --git a/src/components/Editor/Status.vue b/src/components/Editor/Status.vue
@@ -36,7 +36,7 @@
 			<p slot="lastSaved" class="last-saved">
 				{{ t('text', 'Last saved') }}: {{ lastSavedString }}
 			</p>
-			<GuestNameDialog v-if="$isPublic && !currentSession.userId" :session="currentSession" />
+			<GuestNameDialog v-if="$isPublic && currentSession && !currentSession.userId" :session="currentSession" />
 		</SessionList>
 	</div>
 </template>

diff --git a/src/services/SessionApi.js b/src/services/SessionApi.js
@@ -75,6 +75,7 @@ export class Connection {
 		this.#content = content
 		this.#documentState = documentState
 		this.#options = options
+		this.isPublic = !!options.shareToken
 		this.closed = false
 	}
 

diff --git a/src/services/SyncService.js b/src/services/SyncService.js
@@ -132,7 +132,7 @@ class SyncService {
 
 	updateSession(guestName) {
 		if (!this.connection.isPublic) {
-			return
+			return Promise.reject(new Error())
 		}
 		return this.connection.update(guestName)
 			.catch((error) => {