Merge pull request #5186 from nextcloud/fix/moved-to-root

fix: document created in wrong location on public share

Author: elzody

cypress/e2e/new.spec.js

@@ -2,8 +2,8 @@
  * SPDX-FileCopyrightText: 2023 Julius Härtl <[email protected]>
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
-// FIXME: Re-renable once 28 has file creation again working
-describe.skip('Create new office files', function() {
+
+describe('New file menu', function() {
 
 	let randUser
 	before(function() {
@@ -18,51 +18,34 @@ describe.skip('Create new office files', function() {
 	})
 
 	it('Shows create file entries', function() {
-		cy.get('.files-controls .button.new')
+		cy.get('form[data-cy-upload-picker=""]')
 			.should('be.visible')
 			.click()
 
-		cy.get('.newFileMenu', { timeout: 10000 })
-			.should('be.visible')
-			.contains('.menuitem', 'New document')
+		cy.get('button[role="menuitem"]')
+			.contains('New document')
 			.should('be.visible')
-			.find('.icon')
-			.should('have.css', 'background-image')
 
-		cy.get('.files-controls .button.new')
+		cy.get('form[data-cy-upload-picker=""]')
 			.click()
 
-		cy.get('.newFileMenu', { timeout: 10000 })
+		cy.get('li[data-cy-upload-picker-menu-entry="upload-file"]')
 			.should('not.be.visible')
 	})
 
-	const newFileTypeLabels = [
-		'document', 'spreadsheet', 'presentation', 'diagram',
-	]
-	newFileTypeLabels.forEach((filetype) => {
-		it('Create empty ' + filetype + ' file', function() {
-			cy.get('.files-controls .button.new')
-				.should('be.visible')
-				.click()
-
-			cy.get('.newFileMenu', { timeout: 10000 })
-				.should('be.visible')
-				.contains('.menuitem', 'New ' + filetype)
-				.as('menuitem')
-				.should('be.visible')
-				.click()
-
-			cy.get('@menuitem').find('.filenameform input[type=text]').type('MyNewFile')
-			cy.get('@menuitem').find('.filenameform .icon-confirm').click()
-
-			cy.waitForViewer()
-			cy.waitForCollabora()
+	describe('Creates a new file', function() {
+		const newFileTypeLabels = [
+			'document', 'spreadsheet', 'presentation', 'diagram',
+		]
+		newFileTypeLabels.forEach((filetype) => {
+			it('Create empty ' + filetype + ' file', function() {
+				cy.newFileFromMenu(filetype, 'MyNewFile')
+				cy.waitForViewer()
+				cy.waitForCollabora()
 
-			cy.screenshot('new-file-' + filetype)
+				cy.screenshot('new-file-' + filetype)
 
-			cy.get('@loleafletframe').within(() => {
-				cy.get('#closebutton').click()
-				cy.waitForViewerClose()
+				cy.closeDocument()
 			})
 		})
 	})

cypress/e2e/share-link.js

@@ -8,7 +8,7 @@ import { randHash } from '../utils/index.js'
 const shareOwner = new User(randHash(), randHash())
 const otherUser = new User(randHash(), randHash())
 
-describe.skip('Public sharing of office documents', () => {
+describe('Public sharing of office documents', () => {
 	before(function() {
 		cy.nextcloudTestingAppConfigSet('richdocuments', 'doc_format', '')
 		cy.createUser(shareOwner)
@@ -105,6 +105,37 @@ describe.skip('Public sharing of office documents', () => {
 				})
 			})
 		})
+
+		describe('New file', () => {
+			before(() => {
+				cy.createFolder(shareOwner, '/Shared-Folder')
+				cy.createFolder(shareOwner, '/Shared-Folder/Subfolder')
+			})
+
+			it('Creates a new file in a public share as a guest', () => {
+				cy.shareLink(shareOwner, '/Shared-Folder', { permissions: 13 }).then((token) => {
+					cy.logout()
+
+					cy.visit(`/s/${token}`, {
+						onBeforeLoad(win) {
+							cy.spy(win, 'postMessage').as('postMessage')
+						},
+					})
+
+					cy.get('tr[data-cy-files-list-row-name="Subfolder"]')
+						.should('be.visible')
+						.click()
+
+					cy.newFileFromMenu('document', 'MyNewFile')
+					waitForCollabora()
+
+					// Make sure the document is still in the correct subfolder
+					cy.reload()
+					cy.get('tr[data-cy-files-list-row-name="MyNewFile.odt"]')
+						.should('be.visible')
+				})
+			})
+		})
 	})
 })
 

cypress/support/commands.js

@@ -454,3 +454,18 @@ Cypress.Commands.add('makeTalkRoomPublic', (user, token, password = '') => {
 	})
 })
 
+Cypress.Commands.add('newFileFromMenu', (fileType = 'document', fileName = 'MyNewFile') => {
+	cy.get('div[data-cy-files-content-breadcrumbs=""]')
+		.find('form[data-cy-upload-picker=""]')
+		.should('be.visible')
+		.click()
+
+	cy.get('button[role="menuitem"]')
+		.contains('New ' + fileType)
+		.should('be.visible')
+		.click()
+
+	cy.get('input[data-cy-files-new-node-dialog-input=""]')
+		.should('be.visible')
+		.type(fileName + '{enter}')
+})

lib/Listener/RegisterTemplateFileCreatorListener.php

@@ -34,7 +34,13 @@ public function handle(Event $event): void {
 			return;
 		}
 
-		if (!$this->permissionManager->isEnabledForUser() || !$this->permissionManager->userCanEdit() || empty($this->capabilitiesService->getCapabilities())) {
+		if (empty($this->capabilitiesService->getCapabilities())) {
+			return;
+		}
+
+		$user = $this->permissionManager->loggedInUser();
+		$userCanCreate = $this->permissionManager->isEnabledForUser($user) && $this->permissionManager->userCanEdit($user);
+		if ($user && !$userCanCreate) {
 			return;
 		}
 

src/public.js

@@ -10,15 +10,12 @@ import {
 	isDownloadHidden,
 } from './helpers/index.js'
 import { getCapabilities } from './services/capabilities.ts'
-import { registerNewFileMenuEntries } from './view/NewFileMenu.js'
 
 document.addEventListener('DOMContentLoaded', () => {
 	if (!isPublicShare() || !OCA.Viewer) {
 		return
 	}
 
-	registerNewFileMenuEntries()
-
 	const isEnabledFilesPdfViewer = getCapabilities().mimetypesNoDefaultOpen.includes('application/pdf')
 
 	if ((isDownloadHidden() || !isEnabledFilesPdfViewer) && isPdf()) {

src/services/api.js

@@ -4,22 +4,7 @@
  */
 
 import axios from '@nextcloud/axios'
-import { generateOcsUrl, generateFilePath } from '@nextcloud/router'
-import { getSharingToken } from '@nextcloud/sharing/public'
-
-export const createEmptyFile = async (context, mimeType, fileName, templateId = null) => {
-	const shareToken = getSharingToken()
-
-	const response = await axios.post(generateOcsUrl('apps/richdocuments/api/v1/file', 2), {
-		mimeType,
-		fileName,
-		directoryPath: context.dirname,
-		shareToken,
-		templateId,
-	})
-
-	return response.data
-}
+import { generateFilePath } from '@nextcloud/router'
 
 export const savePersonalSetting = (data) => {
 	return axios.post(generateFilePath('richdocuments', 'ajax', 'personal.php'), data)

src/view/NewFileMenu.js

@@ -114,6 +114,7 @@ public function testHandleRegistersWithoutDrawSupport() {
 	public function testHandleDoesNotRegisterIfUserCannotEdit() {
 		$event = $this->createMock(RegisterTemplateCreatorEvent::class);
 		$event->method('getTemplateManager')->willReturn($this->templateManager);
+		$this->permissionManager->method('loggedInUser')->willReturn('user');
 		$this->permissionManager->method('isEnabledForUser')->willReturn(true);
 		$this->permissionManager->method('userCanEdit')->willReturn(false);
 		$this->capabilitiesService->method('getCapabilities')->willReturn(['something']);