Merge pull request #3166 from nextcloud/backport/3161/stable28

[stable28] fix(settings): Calculate correct default quota option sizes

appinfo/info.xml

@@ -38,6 +38,12 @@ Note: Encrypting the contents of group folders is currently not supported.]]></d
 		<job>OCA\GroupFolders\BackgroundJob\ExpireGroupTrash</job>
 	</background-jobs>
 
+	<repair-steps>
+		<post-migration>
+			<step>OCA\GroupFolders\Migration\WrongDefaultQuotaRepairStep</step>
+		</post-migration>
+	</repair-steps>
+
 	<commands>
 		<command>OCA\GroupFolders\Command\ExpireGroup\ExpireGroupBase</command>
 		<command>OCA\GroupFolders\Command\ListCommand</command>

lib/Migration/WrongDefaultQuotaRepairStep.php

@@ -0,0 +1,50 @@
+<?php
+
+declare(strict_types=1);
+
+/**
+ * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+namespace OCA\GroupFolders\Migration;
+
+use OCA\GroupFolders\Folder\FolderManager;
+use OCP\DB\Exception;
+use OCP\Migration\IOutput;
+use OCP\Migration\IRepairStep;
+
+class WrongDefaultQuotaRepairStep implements IRepairStep {
+	public function __construct(
+		private FolderManager $manager,
+	) {
+
+	}
+
+	public function getName() {
+		return 'Adjust Groupfolders with wrong default quotas';
+	}
+
+	/**
+	 * @param IOutput $output
+	 * @throws Exception
+	 */
+	public function run(IOutput $output): void {
+		foreach ($this->manager->getAllFolders() as $id => $folder) {
+			$quota = $folder['quota'];
+
+			$changed = false;
+			if ($quota === 1073741274) {
+				$quota = 1024 ** 3;
+				$changed = true;
+			} elseif ($quota === 10737412742) {
+				$quota = 1024 ** 3 * 10;
+				$changed = true;
+			}
+
+			if ($changed) {
+				$this->manager->setFolderQuota($id, $quota);
+			}
+		}
+	}
+}

src/settings/App.tsx

@@ -14,10 +14,11 @@ import AdminGroupSelect from './AdminGroupSelect'
 import SubAdminGroupSelect from './SubAdminGroupSelect'
 import { loadState } from '@nextcloud/initial-state'
 
+const bytesInOneGibibyte = Math.pow(1024, 3)
 const defaultQuotaOptions = {
-	'1 GB': 1073741274,
-	'5 GB': 5368709120,
-	'10 GB': 10737412742,
+	'1 GB': bytesInOneGibibyte,
+	'5 GB': bytesInOneGibibyte * 5,
+	'10 GB': bytesInOneGibibyte * 10,
 	Unlimited: -3,
 }