Merge pull request #1937 from owncloud/check-resharing

Check resharing setting
This commit is contained in:
Bart Visscher 2013-02-28 11:39:11 -08:00
commit 84aa2c66d6
2 changed files with 28 additions and 3 deletions

View File

@ -61,6 +61,7 @@ class Share {
private static $shareTypeGroupUserUnique = 2; private static $shareTypeGroupUserUnique = 2;
private static $backends = array(); private static $backends = array();
private static $backendTypes = array(); private static $backendTypes = array();
private static $isResharingAllowed;
/** /**
* @brief Register a sharing backend class that implements OCP\Share_Backend for an item type * @brief Register a sharing backend class that implements OCP\Share_Backend for an item type
@ -567,6 +568,24 @@ class Share {
throw new \Exception($message); throw new \Exception($message);
} }
/**
* @brief Check if resharing is allowed
* @return Returns true if allowed or false
*
* Resharing is allowed by default if not configured
*
*/
private static function isResharingAllowed() {
if (!isset(self::$isResharingAllowed)) {
if (\OC_Appconfig::getValue('core', 'shareapi_allow_resharing', 'yes') == 'yes') {
self::$isResharingAllowed = true;
} else {
self::$isResharingAllowed = false;
}
}
return self::$isResharingAllowed;
}
/** /**
* @brief Get a list of collection item types for the specified item type * @brief Get a list of collection item types for the specified item type
* @param string Item type * @param string Item type
@ -840,7 +859,10 @@ class Share {
continue; continue;
} }
} }
// Check if resharing is allowed, if not remove share permission
if (isset($row['permissions']) && !self::isResharingAllowed()) {
$row['permissions'] &= ~PERMISSION_SHARE;
}
// Add display names to result // Add display names to result
if ( isset($row['share_with']) && $row['share_with'] != '') { if ( isset($row['share_with']) && $row['share_with'] != '') {
$row['share_with_displayname'] = \OCP\User::getDisplayName($row['share_with']); $row['share_with_displayname'] = \OCP\User::getDisplayName($row['share_with']);
@ -978,7 +1000,7 @@ class Share {
throw new \Exception($message); throw new \Exception($message);
} }
// Check if share permissions is granted // Check if share permissions is granted
if ((int)$checkReshare['permissions'] & PERMISSION_SHARE) { if (self::isResharingAllowed() && (int)$checkReshare['permissions'] & PERMISSION_SHARE) {
if (~(int)$checkReshare['permissions'] & $permissions) { if (~(int)$checkReshare['permissions'] & $permissions) {
$message = 'Sharing '.$itemSource $message = 'Sharing '.$itemSource
.' failed, because the permissions exceed permissions granted to '.$uidOwner; .' failed, because the permissions exceed permissions granted to '.$uidOwner;

View File

@ -28,7 +28,7 @@ class Test_Share extends PHPUnit_Framework_TestCase {
protected $groupBackend; protected $groupBackend;
protected $group1; protected $group1;
protected $group2; protected $group2;
protected $resharing;
public function setUp() { public function setUp() {
OC_User::clearBackends(); OC_User::clearBackends();
@ -56,11 +56,14 @@ class Test_Share extends PHPUnit_Framework_TestCase {
OCP\Share::registerBackend('test', 'Test_Share_Backend'); OCP\Share::registerBackend('test', 'Test_Share_Backend');
OC_Hook::clear('OCP\\Share'); OC_Hook::clear('OCP\\Share');
OC::registerShareHooks(); OC::registerShareHooks();
$this->resharing = OC_Appconfig::getValue('core', 'shareapi_allow_resharing', 'yes');
OC_Appconfig::setValue('core', 'shareapi_allow_resharing', 'yes');
} }
public function tearDown() { public function tearDown() {
$query = OC_DB::prepare('DELETE FROM `*PREFIX*share` WHERE `item_type` = ?'); $query = OC_DB::prepare('DELETE FROM `*PREFIX*share` WHERE `item_type` = ?');
$query->execute(array('test')); $query->execute(array('test'));
OC_Appconfig::setValue('core', 'shareapi_allow_resharing', $this->resharing);
} }
public function testShareInvalidShareType() { public function testShareInvalidShareType() {