Skip to content

Fix getting the access list on external storage - #14747

Merged
MorrisJobke merged 1 commit into
masterfrom
fix/access_list_external_storage
Mar 19, 2019
Merged

Fix getting the access list on external storage#14747
MorrisJobke merged 1 commit into
masterfrom
fix/access_list_external_storage

Conversation

@rullzer

@rullzerrullzer commented Mar 19, 2019

Copy link
Copy Markdown
Member

If a file is on external storage there is no owner. WHich means we can't
check. So just return an empty array then.

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

@rullzer
Fix getting the access list on external storage
230dc50
If a file is on external storage there is no owner. WHich means we can't
check. So just return an empty array then.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzerrullzer added bug 3. to review Waiting for reviews labels Mar 19, 2019
@rullzerrullzer added this to the Nextcloud 16 milestone Mar 19, 2019
@rullzer

rullzer commented Mar 19, 2019

Copy link
Copy Markdown
MemberAuthor

/backport to stable15

ChristophWurst
ChristophWurst approved these changes Mar 19, 2019
View reviewed changes

@ChristophWurstChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Makes sense

MorrisJobke
MorrisJobke approved these changes Mar 19, 2019
View reviewed changes

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code makes sense

@MorrisJobke
MorrisJobke merged commit a5a2c7d into masterMar 19, 2019
@MorrisJobke
MorrisJobke deleted the fix/access_list_external_storage branch March 19, 2019 17:43
@backportbot-nextcloudbackportbot-nextcloudBot mentioned this pull request Mar 19, 2019
Merged
@backportbot-nextcloud

backportbot-nextcloudBot commented Mar 19, 2019

Copy link
Copy Markdown

backport to stable15 in #14751

@MorrisJobkeMorrisJobke mentioned this pull request Mar 20, 2019
Merged
9 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MorrisJobkeMorrisJobkeMorrisJobke approved these changes
@ChristophWurstChristophWurstChristophWurst approved these changes
@nickvergessennickvergessenAwaiting requested review from nickvergessen
@icewind1991icewind1991Awaiting requested review from icewind1991

Assignees

No one assigned

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Milestone

Nextcloud 16

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@MorrisJobke@ChristophWurst