Merge pull request #20259 from nextcloud/backport/19781/stable17

[stable17] Dont always use the current users quota when calculating storage info
This commit is contained in:
Roeland Jago Douma 2020-04-02 19:22:45 +02:00 committed by GitHub
commit b3f142a67c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 17 deletions

View file

@ -43,6 +43,8 @@
* along with this program. If not, see <http://www.gnu.org/licenses/>
*
*/
use OCP\IUser;
use Symfony\Component\Process\ExecutableFinder;
/**
@ -503,19 +505,14 @@ class OC_Helper {
|| $storage->instanceOfStorage('\OC\Files\ObjectStore\HomeObjectStoreStorage')
) {
/** @var \OC\Files\Storage\Home $storage */
$userInstance = $storage->getUser();
$user = ($userInstance === null) ? null : $userInstance->getUID();
$user = $storage->getUser();
} else {
$user = \OC::$server->getUserSession()->getUser()->getUID();
}
if ($user) {
$quota = OC_Util::getUserQuota($user);
} else {
$quota = \OCP\Files\FileInfo::SPACE_UNLIMITED;
$user = \OC::$server->getUserSession()->getUser();
}
$quota = OC_Util::getUserQuota($user);
if ($quota !== \OCP\Files\FileInfo::SPACE_UNLIMITED) {
// always get free space / total space from root + mount points
return self::getGlobalStorageInfo();
return self::getGlobalStorageInfo($quota);
}
}
@ -561,11 +558,10 @@ class OC_Helper {
/**
* Get storage info including all mount points and quota
*
* @param int $quota
* @return array
*/
private static function getGlobalStorageInfo() {
$quota = OC_Util::getUserQuota(\OCP\User::getUser());
private static function getGlobalStorageInfo($quota) {
$rootInfo = \OC\Files\Filesystem::getFileInfo('', 'ext');
$used = $rootInfo['size'];
if ($used < 0) {

View file

@ -250,8 +250,7 @@ class OC_Util {
) {
/** @var \OC\Files\Storage\Home $storage */
if (is_object($storage->getUser())) {
$user = $storage->getUser()->getUID();
$quota = OC_Util::getUserQuota($user);
$quota = OC_Util::getUserQuota($storage->getUser());
if ($quota !== \OCP\Files\FileInfo::SPACE_UNLIMITED) {
return new \OC\Files\Storage\Wrapper\Quota(array('storage' => $storage, 'quota' => $quota, 'root' => 'files'));
}
@ -374,11 +373,10 @@ class OC_Util {
/**
* Get the quota of a user
*
* @param string $userId
* @param IUser|null $user
* @return float Quota bytes
*/
public static function getUserQuota($userId) {
$user = \OC::$server->getUserManager()->get($userId);
public static function getUserQuota(?IUser $user) {
if (is_null($user)) {
return \OCP\Files\FileInfo::SPACE_UNLIMITED;
}