Merge pull request #22913 from nextcloud/group-folder-quota-warning-m…

…ountpoint add mount point to quota warning message
nextcloud · Oct 4, 2020 · f312cfa · f312cfa
2 parents 0bca480 + 0dfdf3e
commit f312cfa
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 8 deletions.
diff --git a/apps/files/js/files.js b/apps/files/js/files.js
@@ -73,6 +73,7 @@
 				$('#upload.button').attr('data-original-title', response.data.maxHumanFilesize);
 				$('#usedSpacePercent').val(response.data.usedSpacePercent);
 				$('#usedSpacePercent').data('mount-type', response.data.mountType);
+				$('#usedSpacePercent').data('mount-point', response.data.mountPoint);
 				$('#owner').val(response.data.owner);
 				$('#ownerDisplayName').val(response.data.ownerDisplayName);
 				Files.displayStorageWarnings();
@@ -157,20 +158,23 @@
 			var usedSpacePercent = $('#usedSpacePercent').val(),
 				owner = $('#owner').val(),
 				ownerDisplayName = $('#ownerDisplayName').val(),
-				mountType = $('#usedSpacePercent').data('mount-type');
+				mountType = $('#usedSpacePercent').data('mount-type'),
+				mountPoint = $('#usedSpacePercent').data('mount-point');
 			if (usedSpacePercent > 98) {
 				if (owner !== OC.getCurrentUser().uid) {
 					OC.Notification.show(t('files', 'Storage of {owner} is full, files can not be updated or synced anymore!',
 						{owner: ownerDisplayName}), {type: 'error'}
 					);
 				} else if (mountType === 'group') {
 					OC.Notification.show(t('files',
-						'This group folder is full, files can not be updated or synced anymore!'),
+						'Group folder "{mountPoint}" is full, files can not be updated or synced anymore!',
+						{mountPoint: mountPoint}),
 						{type: 'error'}
 					);
 				} else if (mountType === 'external') {
 					OC.Notification.show(t('files',
-						'This external storage is full, files can not be updated or synced anymore!'),
+						'External storage "{mountPoint}" is full, files can not be updated or synced anymore!',
+						{mountPoint: mountPoint}),
 						{type : 'error'}
 					);
 				} else {
@@ -192,14 +196,14 @@
 					);
 				} else if (mountType === 'group') {
 					OC.Notification.show(t('files',
-						'This group folder is almost full ({usedSpacePercent}%).',
-						{usedSpacePercent: usedSpacePercent}),
+						'Group folder "{mountPoint}" is almost full ({usedSpacePercent}%).',
+						{mountPoint: mountPoint, usedSpacePercent: usedSpacePercent}),
 						{type : 'error'}
 					);
 				} else if (mountType === 'external') {
 					OC.Notification.show(t('files',
-						'This external storage is almost full ({usedSpacePercent}%).',
-						{usedSpacePercent: usedSpacePercent}),
+						'External storage "{mountPoint}" is almost full ({usedSpacePercent}%).',
+						{mountPoint: mountPoint, usedSpacePercent: usedSpacePercent}),
 						{type : 'error'}
 					);
 				} else {

diff --git a/apps/files/lib/Helper.php b/apps/files/lib/Helper.php
@@ -64,6 +64,7 @@ public static function buildFileStorageStatistics($dir) {
 			'owner' => $storageInfo['owner'],
 			'ownerDisplayName' => $storageInfo['ownerDisplayName'],
 			'mountType' => $storageInfo['mountType'],
+			'mountPoint' => $storageInfo['mountPoint'],
 		];
 	}
 

diff --git a/lib/private/legacy/OC_Helper.php b/lib/private/legacy/OC_Helper.php
@@ -545,6 +545,7 @@ public static function getStorageInfo($path, $rootInfo = null) {
 		if ($owner) {
 			$ownerDisplayName = $owner->getDisplayName();
 		}
+		[,,,$mountPoint] = explode('/', $mount->getMountPoint(), 4);
 
 		return [
 			'free' => $free,
@@ -554,7 +555,8 @@ public static function getStorageInfo($path, $rootInfo = null) {
 			'relative' => $relative,
 			'owner' => $ownerId,
 			'ownerDisplayName' => $ownerDisplayName,
-			'mountType' => $mount->getMountType()
+			'mountType' => $mount->getMountType(),
+			'mountPoint' => trim($mountPoint, '/'),
 		];
 	}