ericflo / django-avatar

The delete function should have the login_required decorator. There is a bug in the last line of views.py: a duplicate line change = login_required(change) instead of delete = login_required(delete)

Hello,

(Please let me know if I need to follow a different protocol for submitting this piece of information. It's my first time submitting anything to an open source project.)

    if AVATAR_GRAVATAR_BACKUP:
        # didn't work:
        #return "http://www.gravatar.com/avatar/%s/?%s" % (

        # works (note the slash before the ? is removed):
        return "http://www.gravatar.com/avatar/%s?%s" % (
            hashlib.md5(user.email).hexdigest(),
            urllib.urlencode({'s': str(size)}),)
            )

When defaulting to a Gravatar, my application was not finding the actual Gravatar but was instead always returning the default blue Gravatar "G". I compared the link text in the code with the link format spec on Gravatar.com and found that there was an extra slash at the end of the link. I removed this on my install, and now it works.

Thanks.

I've created a Danish locale file for django-avatar. Please include it in the project... since there doesn't seem to be an attachment option here, you can get it from:

http://daimi.au.dk/~illio/pinax/django-avatar/django.po

It's in UTF-8.