Skip to content
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.

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

Users cant upload Profile pictures #3735

Closed
RainerZarth opened this issue Jan 29, 2019 · 2 comments
Closed

Users cant upload Profile pictures #3735

RainerZarth opened this issue Jan 29, 2019 · 2 comments
Assignees
Labels
bug severity:minor Cosmetic issues or bugs with simple workarounds user management 😇 features related to user management

Comments

@RainerZarth
Copy link

Description

Users without permission "edit Users" cant upload their own Profile Picture in the Backend, the Error Message is: "User is not permitted to perform this action".

Steps to reproduce

1.Create an Usergroup without Permission to edit other users.
2.The Group has all rights on the Volume to the Profile pictures.
3.When the User wants to edit his Account (upper left dropdown) in Backend, he is not allowed to upload a profile Picture.
4.the right to edit Users allows the User group to edit all Profiles

Additional info

  • Craft version: 3.1.5
  • PHP version: 7.2 fpm
  • Database driver & version: psql 10
  • Plugins & versions: none
@brandonkelly brandonkelly added user management 😇 features related to user management bug severity:minor Cosmetic issues or bugs with simple workarounds labels Jan 29, 2019
@mattrothenberg
Copy link

mattrothenberg commented Feb 11, 2019

@brandonkelly @andris-sevcenko I'm fairly certain this is still broken. I'm on 3.1.8 and non-admin users still cannot upload user photos. I think it's because the return value of the following expression is a string, and that will never be equal to the userId that gets cast to an int in the lines prior.

See 04b1d9c#diff-000feae194979d9dea78c415431bef50

Craft::$app->getUser()->getIdentity()->id

brandonkelly added a commit that referenced this issue Feb 11, 2019
@brandonkelly
Copy link
Member

Yep, fixed for 3.1.9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug severity:minor Cosmetic issues or bugs with simple workarounds user management 😇 features related to user management
Projects
None yet
Development

No branches or pull requests

4 participants