Merge pull request #6274 from LibreSign/fix/envelope-single-folder

fix: envelope single folder

Author: vitormattos

lib/Controller/FileController.php

@@ -414,7 +414,7 @@ private function fetchPreview(
 	 *
 	 * @param LibresignNewFile $file File to save
 	 * @param string $name The name of file to sign
-	 * @param LibresignFolderSettings $settings Settings to define the pattern to store the file. See more informations at FolderService::getFolderName method.
+	 * @param LibresignFolderSettings $settings Settings to define how and where the file should be stored
 	 * @param list<LibresignNewFile> $files Multiple files to create an envelope (optional, use either file or files)
 	 * @return DataResponse<Http::STATUS_OK, LibresignNextcloudFile, array{}>|DataResponse<Http::STATUS_UNPROCESSABLE_ENTITY, array{message: string}, array{}>
 	 *

lib/Service/EnvelopeService.php

@@ -53,14 +53,15 @@ public function createEnvelope(string $name, string $userId, int $filesCount = 0
 
 		$parentFolder = $this->folderService->getFolder();
 
-		$folderName = $name . '_' . substr(UUIDUtil::getUUID(), 0, 8);
+		$uuid = UUIDUtil::getUUID();
+		$folderName = $name . '_' . $uuid;
 		$envelopeFolder = $parentFolder->newFolder($folderName);
 
 		$envelope = new FileEntity();
 		$envelope->setNodeId($envelopeFolder->getId());
 		$envelope->setNodeTypeEnum(NodeType::ENVELOPE);
 		$envelope->setName($name);
-		$envelope->setUuid(UUIDUtil::getUUID());
+		$envelope->setUuid($uuid);
 		$envelope->setCreatedAt(new DateTime());
 		$envelope->setStatus(FileEntity::STATUS_DRAFT);
 
@@ -106,6 +107,23 @@ public function getEnvelopeByFileId(int $fileId): ?FileEntity {
 		}
 	}
 
+	public function getEnvelopeFolder(FileEntity $envelope): \OCP\Files\Folder {
+		$userId = $envelope->getUserId();
+		if (!$userId) {
+			throw new LibresignException('Envelope does not have a user');
+		}
+
+		$this->folderService->setUserId($userId);
+		$userFolder = $this->folderService->getFolder();
+
+		$envelopeFolderNode = $userFolder->getFirstNodeById($envelope->getNodeId());
+		if (!$envelopeFolderNode instanceof \OCP\Files\Folder) {
+			throw new LibresignException('Envelope folder not found');
+		}
+
+		return $envelopeFolderNode;
+	}
+
 	private function getMaxFilesPerEnvelope(): int {
 		return $this->appConfig->getValueInt(Application::APP_ID, 'envelope_max_files', 50);
 	}

lib/Service/File/UploadProcessor.php

@@ -48,9 +48,10 @@ public function getNodeFromUploadedFile(array $data): Node {
 		$extension = $this->mimeService->getExtension($content);
 		$this->validateFileContent($content, $extension);
 
-		$userFolder = $this->folderService->getFolder();
-		$folderName = $this->folderService->getFolderName($data, $data['userManager']);
-		$folderToFile = $userFolder->newFolder($folderName);
+		$folderToFile = $this->folderService->getFolderForFile($data, $data['userManager']);
+		if (!$folderToFile instanceof \OCP\Files\Folder) {
+			throw new LibresignException('Envelope folder not found');
+		}
 
 		@unlink($uploadedFile['tmp_name']);
 

lib/Service/FileService.php

@@ -120,9 +120,7 @@ public function getNodeFromData(array $data): Node {
 
 		$this->validateFileContent($content, $extension);
 
-		$userFolder = $this->folderService->getFolder();
-		$folderName = $this->folderService->getFolderName($data, $data['userManager']);
-		$folderToFile = $userFolder->newFolder($folderName);
+		$folderToFile = $this->folderService->getFolderForFile($data, $data['userManager']);
 		$filename = $this->resolveFileName($data, $extension);
 		return $folderToFile->newFile($filename, $content);
 	}

lib/Service/FolderService.php

@@ -124,6 +124,29 @@ private function getLibreSignDefaultPath(): string {
 		return $path;
 	}
 
+	/**
+	 * Get or create the folder where a file should be stored
+	 *
+	 * @param array $data Must contain 'settings' and optionally 'name', 'userManager'
+	 * @param mixed $identifier User or string identifier
+	 * @return Folder The folder where files should be created
+	 * @throws LibresignException
+	 */
+	public function getFolderForFile(array $data, $identifier): Folder {
+		$userFolder = $this->getFolder();
+
+		if (isset($data['settings']['envelopeFolderId'])) {
+			$envelopeFolder = $userFolder->getFirstNodeById($data['settings']['envelopeFolderId']);
+			if ($envelopeFolder === null || !$envelopeFolder instanceof Folder) {
+				throw new LibresignException($this->l10n->t('Envelope folder not found'));
+			}
+			return $envelopeFolder;
+		}
+
+		$folderName = $this->getFolderName($data, $identifier);
+		return $userFolder->newFolder($folderName);
+	}
+
 	/**
 	 * @param array{settings: array, name: string} $data
 	 * @param IUser $owner

lib/Service/RequestSignatureService.php

@@ -164,9 +164,9 @@ public function saveEnvelope(array $data): array {
 		try {
 			$envelope = $this->envelopeService->createEnvelope($envelopeName, $userId, $filesCount);
 
-			$envelopeFolderName = 'envelope-' . $envelope->getUuid();
+			$envelopeFolder = $this->envelopeService->getEnvelopeFolder($envelope);
 			$envelopeSettings = array_merge($data['settings'] ?? [], [
-				'folderName' => $envelopeFolderName,
+				'envelopeFolderId' => $envelopeFolder->getId(),
 			]);
 
 			foreach ($data['files'] as $fileData) {

openapi-full.json

@@ -4847,7 +4847,7 @@
                                     "settings": {
                                         "$ref": "#/components/schemas/FolderSettings",
                                         "default": [],
-                                        "description": "Settings to define the pattern to store the file. See more informations at FolderService::getFolderName method."
+                                        "description": "Settings to define how and where the file should be stored"
                                     },
                                     "files": {
                                         "type": "array",

openapi.json

@@ -4697,7 +4697,7 @@
                                     "settings": {
                                         "$ref": "#/components/schemas/FolderSettings",
                                         "default": [],
-                                        "description": "Settings to define the pattern to store the file. See more informations at FolderService::getFolderName method."
+                                        "description": "Settings to define how and where the file should be stored"
                                     },
                                     "files": {
                                         "type": "array",

src/types/openapi/openapi-full.ts

@@ -3343,7 +3343,7 @@ export interface operations {
                      */
                     name?: string;
                     /**
-                     * @description Settings to define the pattern to store the file. See more informations at FolderService::getFolderName method.
+                     * @description Settings to define how and where the file should be stored
                      * @default []
                      */
                     settings?: components["schemas"]["FolderSettings"];

src/types/openapi/openapi.ts

@@ -2865,7 +2865,7 @@ export interface operations {
                      */
                     name?: string;
                     /**
-                     * @description Settings to define the pattern to store the file. See more informations at FolderService::getFolderName method.
+                     * @description Settings to define how and where the file should be stored
                      * @default []
                      */
                     settings?: components["schemas"]["FolderSettings"];