please correct me if I am wrong, but I believe that currently multiple users can pick the same avatar. It would be nice to have the currently used avatars removed from the selection list (falling back to a default avatar when the selection list is empty). Thank you.

CommentFileSizeAuthor
#1 avatar_selection_200197.patch3.62 KBstella
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stella’s picture

Assigned: Unassigned » stella
Status: Active » Fixed
FileSize
3.62 KB

Please try out the attached patch or download the latest dev version (available later today).

Avatar images already in use by another user are removed from the selection list. When there are no more available avatars the default avatar image (set on user settings page) will be used. If random default avatar assignment is enabled, then the user will have no random avatar assigned as the random avatar assignment uses the same pool as what the user can select from. However, if default avatar image is set on the user settings page, then that will be used instead.

Cheers,
Stella

skizzo’s picture

Applying the patch to 5.x.2.2 gives me error on Hunk #2 FAILED at 220. Should I have applied the patch against 5.x-2.x-dev maybe? I also tried to download module from CVS HEAD but get errors in "Manage avatars". No big deal, I will wait for next release update and then test it... Thank you Stella.

stella’s picture

Yeah, sorry that would be a patch against 5.x-2.x. If downloading via CVS you need to grab the files from the DRUPAL-5--2 branch.

skizzo’s picture

did some testing with the new feature, and everything seems to work fine. I tried also to delete users, and noticed that the avatar is released back to the pool, as expected. Now, if I only could find 3.000 nice looking avatars with transparent background.... Thank you again.

stella’s picture

Released in Avatar Selection 5.x-2.3.

Cheers,
Stella

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.