wbrawner/server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

OCP\Share::unshareAll(): Deduplicate hook parameters.

Browse source
This commit is contained in:
Andreas Fischer 2013-09-23 16:22:48 +02:00 committed by Morris Jobke
parent 8872b881cc
commit 329299e34c
1 changed files with 26 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

34
lib/public/share.php
View file

@ -665,19 +665,16 @@ class Share {
public static function unshareAll($itemType, $itemSource) {
if ($shares = self::getItemShared($itemType, $itemSource)) {
// Pass all the vars we have for now, they may be useful
\OC_Hook::emit('OCP\Share', 'pre_unshareAll', array(
$hookParams = array(
'itemType' => $itemType,
'itemSource' => $itemSource,
'shares' => $shares
));
'shares' => $shares,
);
\OC_Hook::emit('OCP\Share', 'pre_unshareAll', $hookParams);
foreach ($shares as $share) {
self::delete($share['id']);
}
\OC_Hook::emit('OCP\Share', 'post_unshareAll', array(
'itemType' => $itemType,
'itemSource' => $itemSource,
'shares' => $shares
));
\OC_Hook::emit('OCP\Share', 'post_unshareAll', $hookParams);
return true;
}
return false;
@ -852,6 +849,27 @@ class Share {
return false;
}
/**
* Unshares a share given a share data array
* @param array $item Share data (usually database row)
* @return null
*/
protected static function unshareItem(array $item) {
// Pass all the vars we have for now, they may be useful
$hookParams = array(
'itemType' => $item['item_type'],
'itemSource' => $item['item_source'],
'shareType' => $item['share_type'],
'shareWith' => $item['share_with'],
'itemParent' => $item['parent'],
);
\OC_Hook::emit('OCP\Share', 'pre_unshare', $hookParams + array(
'fileSource' => $item['file_source'],
));
self::delete($item['id']);
\OC_Hook::emit('OCP\Share', 'post_unshare', $hookParams);
}
/**
* Get the backend class for the specified item type
* @param string $itemType