Skip to content

Show sharees via propfind #14429

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rullzer merged 11 commits into from
Aug 13, 2019
Merged

Show sharees via propfind #14429

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e3991fb
Show sharees via propfind
tobiasKaminsky Feb 26, 2019
b81fb18
wip
tobiasKaminsky Jul 22, 2019
efa0733
wip
tobiasKaminsky Jul 23, 2019
06a9ade
Use proper caching
rullzer Jul 30, 2019
2a45763
Use proper ShareeList
rullzer Jul 30, 2019
21477bc
Unify share fetching
rullzer Jul 30, 2019
42f00e8
add display name to sharee list
tobiasKaminsky Jul 31, 2019
85d5400
Fix tests
rullzer Aug 1, 2019
82ae6fb
use namespace everywhere
tobiasKaminsky Aug 6, 2019
eb564e8
get all shares, not only first one per type
tobiasKaminsky Aug 8, 2019
a853926
Fix tests
rullzer Aug 11, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
use namespace everywhere 
no camelCase

Signed-off-by: tobiasKaminsky <[email protected]>
  • Loading branch information
@tobiasKaminsky
tobiasKaminsky committed Aug 6, 2019
commit 82ae6fb6d31274a84040b1984a4f35c79c8a400a
6 changes: 3 additions & 3 deletions apps/dav/lib/Connector/Sabre/ShareeList.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,9 @@ public function __construct(array $shares) {
function xmlSerialize(Writer $writer) {
foreach ($this->shares as $share) {
$writer->startElement('{' . self::NS_NEXTCLOUD . '}sharee');
$writer->writeElement("id", $share->getSharedWith());
$writer->writeElement("displayName", $share->getSharedWithDisplayName());
$writer->writeElement('type', $share->getShareType());
$writer->writeElement('{' . self::NS_NEXTCLOUD . '}id', $share->getSharedWith());
$writer->writeElement('{' . self::NS_NEXTCLOUD . '}display-name', $share->getSharedWithDisplayName());
Copy link
Member

@georgehrke georgehrke Aug 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
$writer->writeElement('{' . self::NS_NEXTCLOUD . '}display-name', $share->getSharedWithDisplayName());
$writer->writeElement('{' . self::NS_NEXTCLOUD . '}display-name', $share->getSharedWithDisplayName() ?? $share->getSharedWith());

Copy link
Member Author

@tobiasKaminsky tobiasKaminsky Aug 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wrote a test for android library and there the problem also exists.
It seems that always one of the display names is set to userId.

$writer->writeElement('{' . self::NS_NEXTCLOUD . '}type', $share->getShareType());
$writer->endElement();
}
}
Expand Down