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

Don't show link shares hash in inherited share list 

This adds no value at all. Just showing the text "(Shared link)" now.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
This commit is contained in:
Roeland Jago Douma 2020-01-10 10:03:08 +01:00
parent 0f238623f6
commit fc77f530d7
No known key found for this signature in database
GPG Key ID: F941078878347C0C
3 changed files with 32 additions and 50 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
apps/files_sharing/lib/Controller/ShareAPIController.php
View File

@ -222,12 +222,12 @@ class ShareAPIController extends OCSController {
$group = $this->groupManager->get($share->getSharedWith()); $group = $this->groupManager->get($share->getSharedWith());
$result['share_with'] = $share->getSharedWith(); $result['share_with'] = $share->getSharedWith();
$result['share_with_displayname'] = $group !== null ? $group->getDisplayName() : $share->getSharedWith(); $result['share_with_displayname'] = $group !== null ? $group->getDisplayName() : $share->getSharedWith();
} else if ($share->getShareType() === Share::SHARE_TYPE_LINK) { } else if ($share->getShareType() === IShare::TYPE_LINK) {
// "share_with" and "share_with_displayname" for passwords of link // "share_with" and "share_with_displayname" for passwords of link
// shares was deprecated in Nextcloud 15, use "password" instead. // shares was deprecated in Nextcloud 15, use "password" instead.
$result['share_with'] = $share->getPassword(); $result['share_with'] = $share->getPassword();
$result['share_with_displayname'] = $share->getPassword(); $result['share_with_displayname'] = '(' . $this->l->t('Shared link') . ')';
$result['password'] = $share->getPassword(); $result['password'] = $share->getPassword();

6
apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
View File

@ -664,7 +664,7 @@ class ShareAPIControllerTest extends TestCase {
'share_type' => \OCP\Share::SHARE_TYPE_LINK, 'share_type' => \OCP\Share::SHARE_TYPE_LINK,
'password' => 'password', 'password' => 'password',
'share_with' => 'password', 'share_with' => 'password',
'share_with_displayname' => 'password', 'share_with_displayname' => '(Shared link)',
'send_password_by_talk' => false, 'send_password_by_talk' => false,
'uid_owner' => 'initiatorId', 'uid_owner' => 'initiatorId',
'displayname_owner' => 'initiatorDisplay', 'displayname_owner' => 'initiatorDisplay',
@ -3709,7 +3709,7 @@ class ShareAPIControllerTest extends TestCase {
'file_target' => 'myTarget', 'file_target' => 'myTarget',
'password' => 'mypassword', 'password' => 'mypassword',
'share_with' => 'mypassword', 'share_with' => 'mypassword',
'share_with_displayname' => 'mypassword', 'share_with_displayname' => '(Shared link)',
'send_password_by_talk' => false, 'send_password_by_talk' => false,
'mail_send' => 0, 'mail_send' => 0,
'url' => 'myLink', 'url' => 'myLink',
@ -3761,7 +3761,7 @@ class ShareAPIControllerTest extends TestCase {
'file_target' => 'myTarget', 'file_target' => 'myTarget',
'password' => 'mypassword', 'password' => 'mypassword',
'share_with' => 'mypassword', 'share_with' => 'mypassword',
'share_with_displayname' => 'mypassword', 'share_with_displayname' => '(Shared link)',
'send_password_by_talk' => true, 'send_password_by_talk' => true,
'mail_send' => 0, 'mail_send' => 0,
'url' => 'myLink', 'url' => 'myLink',

26
build/integration/sharing_features/sharing-v1.feature
View File

@ -343,26 +343,8 @@ Feature: sharing
| permissions | 16 | | permissions | 16 |
And As an "user1" And As an "user1"
And accepting last share And accepting last share
# And folder "first" of user "user0" is shared with user "user1"
# And creating a share with
# | path | /first/second |
# | shareType | 0 |
# | shareWith | user2 |
# | permissions | 16 |
And folder "first/second" of user "user0" is shared with user "user2" And folder "first/second" of user "user0" is shared with user "user2"
# And As an "user1"
# And creating a share with
# | path | /first/test1.txt |
# | shareType | 0 |
# | shareWith | user3 |
# | permissions | 8 |
And file "first/test1.txt" of user "user0" is shared with user "user3" And file "first/test1.txt" of user "user0" is shared with user "user3"
# And As an "user2"
# And creating a share with
# | path | /second/test2.txt |
# | shareType | 0 |
# | shareWith | user3 |
# | permissions | 8 |
And file "first/second/test2.txt" of user "user0" is shared with user "user3" And file "first/second/test2.txt" of user "user0" is shared with user "user3"
# get inherited shares from the owner PoV # get inherited shares from the owner PoV
And As an "user0" And As an "user0"
@ -372,14 +354,14 @@ Feature: sharing
And User "user0" should not be included in the response And User "user0" should not be included in the response
And User "user1" should be included in the response And User "user1" should be included in the response
And User "user2" should be included in the response And User "user2" should be included in the response
And User "user3" should be included in the response And User "user3" should not be included in the response
When sending "GET" to "/apps/files_sharing/api/v1/shares/inherited?path=first/test1.txt" When sending "GET" to "/apps/files_sharing/api/v1/shares/inherited?path=first/test1.txt"
Then the OCS status code should be "100" Then the OCS status code should be "100"
And the HTTP status code should be "200" And the HTTP status code should be "200"
And User "user0" should not be included in the response And User "user0" should not be included in the response
And User "user1" should be included in the response And User "user1" should be included in the response
And User "user2" should not be included in the response And User "user2" should not be included in the response
And User "user3" should be included in the response And User "user3" should not be included in the response
# get inherited shares from the a user with no shares rights # get inherited shares from the a user with no shares rights
And As an "user2" And As an "user2"
When sending "GET" to "/apps/files_sharing/api/v1/shares/inherited?path=first/test1.txt" When sending "GET" to "/apps/files_sharing/api/v1/shares/inherited?path=first/test1.txt"
@ -393,13 +375,13 @@ Feature: sharing
And User "user0" should not be included in the response And User "user0" should not be included in the response
And User "user1" should not be included in the response And User "user1" should not be included in the response
And User "user2" should be included in the response And User "user2" should be included in the response
And User "user3" should be included in the response And User "user3" should not be included in the response
When sending "GET" to "/apps/files_sharing/api/v1/shares/inherited?path=first/test1.txt" When sending "GET" to "/apps/files_sharing/api/v1/shares/inherited?path=first/test1.txt"
Then the OCS status code should be "100" Then the OCS status code should be "100"
And the HTTP status code should be "200" And the HTTP status code should be "200"
And User "user0" should not be included in the response And User "user0" should not be included in the response
And User "user1" should not be included in the response And User "user1" should not be included in the response
And User "user2" should not be included in the response And User "user2" should not be included in the response
And User "user3" should be included in the response And User "user3" should not be included in the response
# See sharing-v1-part2.feature # See sharing-v1-part2.feature