diff options
author | Roeland Jago Douma <rullzer@users.noreply.github.com> | 2020-11-12 15:22:07 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-12 15:22:07 +0100 |
commit | 38b060628b33c6368704e13e64eeb97d15612c0a (patch) | |
tree | e366ba1f5e3eff49b09d9401096ef133b7e430d3 | |
parent | 4baf1868be7d029d24cc11ccdc2bd4e7439f4503 (diff) | |
parent | b9db1cede3d40bf84d394d007827260f14015d8d (diff) | |
download | nextcloud-server-38b060628b33c6368704e13e64eeb97d15612c0a.tar.gz nextcloud-server-38b060628b33c6368704e13e64eeb97d15612c0a.zip |
Merge pull request #23168 from nextcloud/backport/22913/stable19
[stable19] add mount point to quota warning message
-rw-r--r-- | apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php | 6 | ||||
-rw-r--r-- | apps/files/js/files.js | 18 | ||||
-rw-r--r-- | apps/files/lib/Helper.php | 1 | ||||
-rw-r--r-- | lib/private/legacy/OC_Helper.php | 8 |
4 files changed, 25 insertions, 8 deletions
diff --git a/apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php b/apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php index db79ea21b0e..2207f9e6410 100644 --- a/apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php @@ -303,6 +303,9 @@ class DirectoryTest extends \Test\TestCase { ->method('getFileInfo') ->willReturn($this->info); + $mountPoint->method('getMountPoint') + ->willReturn('/user/files/mymountpoint'); + $dir = new Directory($this->view, $this->info); $this->assertEquals([200, -3], $dir->getQuotaInfo()); //200 used, unlimited } @@ -338,6 +341,9 @@ class DirectoryTest extends \Test\TestCase { ->method('getMountPoint') ->willReturn($mountPoint); + $mountPoint->method('getMountPoint') + ->willReturn('/user/files/mymountpoint'); + $this->view->expects($this->once()) ->method('getFileInfo') ->willReturn($this->info); diff --git a/apps/files/js/files.js b/apps/files/js/files.js index cdc1c70ffc5..2f570f450c9 100644 --- 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,7 +158,8 @@ 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!', @@ -165,12 +167,14 @@ ); } 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 index 1f728565ca6..2d5a3e2f26e 100644 --- a/apps/files/lib/Helper.php +++ b/apps/files/lib/Helper.php @@ -64,6 +64,7 @@ class Helper { '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 index d9c0c16de0d..f7ef398ca49 100644 --- a/lib/private/legacy/OC_Helper.php +++ b/lib/private/legacy/OC_Helper.php @@ -545,6 +545,11 @@ class OC_Helper { if ($owner) { $ownerDisplayName = $owner->getDisplayName(); } + if (substr_count($mount->getMountPoint(), '/') < 3) { + $mountPoint = ''; + } else { + [,,,$mountPoint] = explode('/', $mount->getMountPoint(), 4); + } return [ 'free' => $free, @@ -554,7 +559,8 @@ class OC_Helper { 'relative' => $relative, 'owner' => $ownerId, 'ownerDisplayName' => $ownerDisplayName, - 'mountType' => $mount->getMountType() + 'mountType' => $mount->getMountType(), + 'mountPoint' => trim($mountPoint, '/'), ]; } |