Issue #3346912 by zanvidmar: Profile pictures missing on landing page #3333
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.
Profile pictures missing on landing page content type for LU that is not VU.
Problem
This is the FUP of 3237
For authenticated user, which is not verified user, the user profile image is not loaded and appears broken instead of the default placeholder.
Solution
Use permission system instead of role based logic which is not covering all the cases.
Issue tracker
3346912
Theme issue tracker
N/A
How to test
social_private_filesocial_file_private modules enabled$settings['file_private_path']
in settings.phpDefinition of done
Before merge
After merge
Screenshots
N/A
Release notes
When the website was using private file, then the profile images appear to be broken on landing pages for Authenticated users without verified user role, if a featured content or a stream view was added. We resolved the access problem for given users and now the default profile picture is displayed.
Change Record
N/A
Translations
N/A