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

Problems with avatars #67

Closed
Yvan256 opened this issue Dec 24, 2018 · 1 comment · Fixed by #1626
Closed

Problems with avatars #67

Yvan256 opened this issue Dec 24, 2018 · 1 comment · Fixed by #1626
Labels

Comments

@Yvan256
Copy link

Yvan256 commented Dec 24, 2018

Problem #1: the avatar_width and avatar_height columns in the database are not added/updated when adding/modifying/removing an avatar.

Steps I tried:

  1. picking an avatar from the default set
  2. uploading my own avatar (PNG)
  3. uploading my own avatar (GIF)
  4. uploading my own avatar (I tried with both JPG and JPEG extensions)

In all four cases, the avatar_width and avatar_height values are never saved in the database and stay empty (NULL).

When inputing values manually into the database, the avatar_width and avatar_height variables work as they should in a template.

Problem #2: when deleting my avatar via the control panel, the avatar files on the server and the avatar_width and avatar_height columns in the database are not being deleted and still contain the values I added manually.

After my tests, the avatar directory has the following files in it:
avatar_1_1.png
avatar_1_2.png
avatar_1_3.png
avatar_1_4.png
avatar_1.gif
avatar_1.jpeg
avatar_1.png

And the "images/avatars/_thumbs" sub-directory has these files:
Yvan256.gif
Yvan256.jpeg
Yvan256.jpg
Yvan256.png

It would be nice if the system could remove the associated custom/uploaded avatar files from the server when a user's avatar is removed via the control panel.

Please note that I only tried these steps with myself (Super Admin).

System:
EE version 5.1.1
PHP version 7.0.33

@intoeetive intoeetive added the EE5 label Feb 23, 2021
mithra62 added a commit to mithra62/ExpressionEngine that referenced this issue Dec 7, 2021
@intoeetive
Copy link
Contributor

intoeetive commented Dec 30, 2021

Some of these are only relevant for EE5

The other ones will be fixed with #1685

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants