[Bugfix]nextcloud#43468: 'forbidden_chars' added the condition to drop file and added forbiddenCharacters to UploadPicker

Signed-off-by: Andrii Rublov <airublev@outlook.com>

Author: Andrii Rublov

apps/files/src/services/DropService.ts

@@ -29,9 +29,12 @@ import { getUploader } from '@nextcloud/upload'
 import { joinPaths } from '@nextcloud/paths'
 import { showError } from '@nextcloud/dialogs'
 import { translate as t } from '@nextcloud/l10n'
+import { loadState } from '@nextcloud/initial-state'
 
 import logger from '../logger.js'
 
+const forbiddenCharacters = loadState('files', 'forbiddenCharacters', '') as string
+
 export const handleDrop = async (data: DataTransfer): Promise<Upload[]> => {
 	// TODO: Maybe handle `getAsFileSystemHandle()` in the future
 
@@ -67,11 +70,18 @@ export const handleDrop = async (data: DataTransfer): Promise<Upload[]> => {
 const handleFileUpload = async (file: File, path: string = '') => {
 	const uploader = getUploader()
 
-	try {
-		return await uploader.upload(`${path}${file.name}`, file)
-	} catch (e) {
-		showError(t('files', 'Uploading "{filename}" failed', { filename: file.name }))
-		throw e
+	const forbidden = forbiddenCharacters.split('')
+	let forbiddenChar
+
+	if(forbiddenChar = forbidden.find(char => file.name.includes(char))){
+		showError(t('files', `"${forbiddenChar}" is not allowed inside a file name.`));
+	}else{
+		try {
+			return await uploader.upload(`${path}${file.name}`, file)
+		} catch (e) {
+			showError(t('files', 'Uploading "{filename}" failed', { filename: file.name }))
+			throw e
+		}
 	}
 }
 

apps/files/src/views/FilesList.vue

@@ -54,6 +54,7 @@
 
 					<!-- Uploader -->
 					<UploadPicker v-else-if="currentFolder"
+						:forbiddenCharacters="forbiddenCharacters"
 						:content="dirContents"
 						:destination="currentFolder"
 						:multiple="true"
@@ -160,6 +161,7 @@ import filesSortingMixin from '../mixins/filesSorting.ts'
 import logger from '../logger.js'
 import DragAndDropNotice from '../components/DragAndDropNotice.vue'
 
+const forbiddenCharacters = loadState('files', 'forbiddenCharacters', '') as string
 const isSharingEnabled = (getCapabilities() as { files_sharing?: boolean })?.files_sharing !== undefined
 
 export default defineComponent({
@@ -248,6 +250,10 @@ export default defineComponent({
 			return this.filesStore.getNode(fileId)
 		},
 
+		forbiddenCharacters(): string {
+			return forbiddenCharacters
+		},
+
 		/**
 		 * Directory content sorting parameters
 		 * Provided by an extra computed property for caching