refactor(settings): Return entire folder on creation

Signed-off-by: provokateurin <[email protected]>

lib/Controller/FolderController.php

@@ -14,6 +14,7 @@
 use OCA\GroupFolders\Service\FoldersFilter;
 use OCP\AppFramework\Http;
 use OCP\AppFramework\Http\DataResponse;
+use OCP\AppFramework\OCS\OCSNotFoundException;
 use OCP\AppFramework\OCSController;
 use OCP\Files\IRootFolder;
 use OCP\IGroupManager;
@@ -145,10 +146,15 @@ private function getRootFolderStorageId(): ?int {
 	/**
 	 * @RequireGroupFolderAdmin
 	 * @NoAdminRequired
+	 * @throws OCSNotFoundException
 	 */
 	public function addFolder(string $mountpoint): DataResponse {
 		$id = $this->manager->createFolder(trim($mountpoint));
-		return new DataResponse(['id' => $id]);
+		$folder = $this->manager->getFolder($id, $this->rootFolder->getMountPoint()->getNumericStorageId());
+		if ($folder === false) {
+			throw new OCSNotFoundException();
+		}
+		return new DataResponse($folder);
 	}
 
 	/**

src/settings/Api.ts

@@ -85,10 +85,10 @@ export class Api {
 		}).then((data) => data.data)
 	}
 
-	createFolder(mountPoint: string): Thenable<number> {
+	createFolder(mountPoint: string): Thenable<Folder> {
 		return $.post(this.getUrl('folders'), {
 			mountpoint: mountPoint
-		}, null, 'json').then((data: OCSResult<{ id: number; }>) => data.ocs.data.id)
+		}, null, 'json').then((data: OCSResult<Folder>) => data.ocs.data)
 	}
 
 	deleteFolder(id: number): Thenable<void> {

src/settings/App.tsx

@@ -89,17 +89,9 @@ export class App extends Component<{}, AppState> implements OC.Plugin<OC.Search.
 			return
 		}
 		this.setState({ newMountPoint: '' })
-		this.api.createFolder(mountPoint).then((id) => {
+		this.api.createFolder(mountPoint).then((folder) => {
 			const folders = this.state.folders
-			folders.push({
-				mount_point: mountPoint,
-				groups: {},
-				quota: -3,
-				size: 0,
-				id,
-				acl: false,
-				manage: [],
-			})
+			folders.push(folder)
 			this.setState({ folders })
 		})
 	}