mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Optimize isSharingDisabledForuser

This commit is contained in:
Roeland Jago Douma 2016-05-03 11:58:33 +02:00 committed by Roeland Jago Douma
parent d0a2515e2b
commit ae3d0d96fe
No known key found for this signature in database
GPG Key ID: 1E152838F164D13B
2 changed files with 34 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
lib/private/Share20/Manager.php
View File

@ -24,25 +24,24 @@
namespace OC\Share20;
use OC\Cache\CappedMemoryCache;
use OC\Files\Mount\MoveableMount;
use OCP\Files\IRootFolder;
use OCP\Files\NotFoundException;
use OCP\IUserManager;
use OCP\Share\IManager;
use OCP\Share\IProviderFactory;
use OC\Share20\Exception\BackendError;
use OCP\IConfig;
use OCP\IL10N;
use OCP\ILogger;
use OCP\Security\ISecureRandom;
use OCP\Security\IHasher;
use OCP\Files\Mount\IMountManager;
use OCP\IGroupManager;
use OCP\Files\File;
use OCP\Files\Folder;
use OCP\Share\Exceptions\ShareNotFound;
use OCP\Files\IRootFolder;
use OCP\Files\Mount\IMountManager;
use OCP\Files\NotFoundException;
use OCP\IConfig;
use OCP\IGroupManager;
use OCP\IL10N;
use OCP\ILogger;
use OCP\IUserManager;
use OCP\Security\IHasher;
use OCP\Security\ISecureRandom;
use OCP\Share\Exceptions\GenericShareException;
use OCP\Share\Exceptions\ShareNotFound;
use OCP\Share\IManager;
use OCP\Share\IProviderFactory;
/**
* This class is the communication hub for all sharing related operations.
@ -69,6 +68,9 @@ class Manager implements IManager {
private $userManager;
/** @var IRootFolder */
private $rootFolder;
/** @var CappedMemoryCache */
private $sharingDisabledForUsersCache;
/**
* Manager constructor.
@ -106,6 +108,7 @@ class Manager implements IManager {
$this->factory = $factory;
$this->userManager = $userManager;
$this->rootFolder = $rootFolder;
$this->sharingDisabledForUsersCache = new CappedMemoryCache();
}
/**
@ -1209,6 +1212,10 @@ class Manager implements IManager {
* @return bool
*/
public function sharingDisabledForUser($userId) {
if (isset($this->sharingDisabledForUsersCache[$userId])) {
return $this->sharingDisabledForUsersCache[$userId];
}
if ($this->config->getAppValue('core', 'shareapi_exclude_groups', 'no') === 'yes') {
$groupsList = $this->config->getAppValue('core', 'shareapi_exclude_groups_list', '');
$excludedGroups = json_decode($groupsList);
@ -1224,10 +1231,13 @@ class Manager implements IManager {
// if the user is only in groups which are disabled for sharing then
// sharing is also disabled for the user
if (empty($remainingGroups)) {
$this->sharingDisabledForUsersCache[$userId] = true;
return true;
}
}
}
$this->sharingDisabledForUsersCache[$userId] = false;
return false;
}

14
lib/public/Util.php
View File

@ -63,6 +63,9 @@ class Util {
const ERROR=3;
const FATAL=4;
/** \OCP\Share\IManager */
private static $shareManager;
/**
* get the current installed version of ownCloud
* @return array
@ -171,13 +174,14 @@ class Util {
*
* @return boolean
* @since 7.0.0
* @deprecated 9.1.0 Use \OC::$server->getShareManager()->sharingDisabledForUser
*/
public static function isSharingDisabledForUser() {
return \OC_Util::isSharingDisabledForUser(
\OC::$server->getConfig(),
\OC::$server->getGroupManager(),
\OC::$server->getUserSession()->getUser()
);
if (self::$shareManager === null) {
self::$shareManager = \OC::$server->getShareManager();
}
return self::$shareManager->sharingDisabledForUser(\OC::$server->getUserSession()->getUser()->getUID());
}
/**