Browse Source

Admin Audit - Sharing: createShare - report the full path

Signed-off-by: szaimen <szaimen@e.mail.de>
pull/28075/head
szaimen 4 years ago
committed by szaimen (Rebase PR Action)
parent
commit
4570bb4f65
  1. 18
      apps/admin_audit/lib/Actions/Sharing.php
  2. 1
      lib/private/Share20/LegacyHooks.php
  3. 1
      tests/lib/Share20/LegacyHooksTest.php

18
apps/admin_audit/lib/Actions/Sharing.php

@ -50,7 +50,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'permissions', 'permissions',
'id', 'id',
@ -62,7 +62,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'shareWith', 'shareWith',
'permissions', 'permissions',
@ -75,7 +75,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'shareWith', 'shareWith',
'permissions', 'permissions',
@ -88,7 +88,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'shareWith', 'shareWith',
'permissions', 'permissions',
@ -101,7 +101,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'shareWith', 'shareWith',
'permissions', 'permissions',
@ -114,7 +114,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'shareWith', 'shareWith',
'permissions', 'permissions',
@ -127,7 +127,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'shareWith', 'shareWith',
'permissions', 'permissions',
@ -140,7 +140,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'shareWith', 'shareWith',
'permissions', 'permissions',
@ -153,7 +153,7 @@ class Sharing extends Action {
$params, $params,
[ [
'itemType', 'itemType',
'itemTarget',
'path',
'itemSource', 'itemSource',
'shareWith', 'shareWith',
'permissions', 'permissions',

1
lib/private/Share20/LegacyHooks.php

@ -170,6 +170,7 @@ class LegacyHooks {
'shareWith' => $share->getSharedWith(), 'shareWith' => $share->getSharedWith(),
'itemTarget' => $share->getTarget(), 'itemTarget' => $share->getTarget(),
'fileTarget' => $share->getTarget(), 'fileTarget' => $share->getTarget(),
'path' => $share->getNode()->getPath(),
]; ];
\OC_Hook::emit(Share::class, 'post_shared', $postHookData); \OC_Hook::emit(Share::class, 'post_shared', $postHookData);

1
tests/lib/Share20/LegacyHooksTest.php

@ -342,6 +342,7 @@ class LegacyHooksTest extends TestCase {
'permissions' => Constants::PERMISSION_ALL, 'permissions' => Constants::PERMISSION_ALL,
'expiration' => $date, 'expiration' => $date,
'token' => 'token', 'token' => 'token',
'path' => null,
]; ];
$hookListner $hookListner

Loading…
Cancel
Save