Fix display of public link shares in case avatars are disabled#37945
Merged
karakayasemi merged 1 commit intomasterfrom Sep 26, 2020
Merged
Fix display of public link shares in case avatars are disabled#37945karakayasemi merged 1 commit intomasterfrom
karakayasemi merged 1 commit intomasterfrom
Conversation
|
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
85b623b to
df2f82d
Compare
df2f82d to
8a75b5e
Compare
Codecov Report
@@ Coverage Diff @@
## master #37945 +/- ##
=========================================
Coverage 64.71% 64.71%
Complexity 19442 19442
=========================================
Files 1286 1286
Lines 76022 76022
Branches 1336 1336
=========================================
Hits 49196 49196
Misses 26432 26432
Partials 394 394
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fix display of public link shares in case avatars are disabled
Related issue
https://github.com/owncloud/enterprise/issues/4190
Motivation and Context
This PR fixes the display of public link shares in case avatars are disabled through config.php. Previously, a "sharing is not allowed" message was displayed and public link was not visible in the sharing panel of the "All files" view. The reason for this was a missing check at https://github.com/owncloud/core/blob/master/apps/files/js/filelist.js#L1902 for avatars being enabled/disabled.
How Has This Been Tested?
Manually:
'enable_avatars' => false,in config.phpBefore this fix:
The public link was correctly created in DB but when leaving the sharing panel and trying to access it again via the "All files" view, a "sharing is not allowed" message was shown. It could be accessed via the "Shared with you" page only.
With this fix:
Public link is correctly displayed in the sharing panel of the "All files" view.
Types of changes
Checklist: