Skip to content

Commit 5c56de5

Browse files
authored
Organizations: check if organization has email before calling gravatar (#12419)
1 parent 084dcb2 commit 5c56de5

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

readthedocs/organizations/models.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55

66
import structlog
77
from autoslug import AutoSlugField
8+
from django.conf import settings
89
from django.contrib.auth.models import User
910
from django.contrib.contenttypes.fields import GenericRelation
1011
from django.contrib.contenttypes.models import ContentType
@@ -278,7 +279,9 @@ def get_avatar_url(self):
278279
"""
279280
if self.avatar:
280281
return self.avatar.url
281-
return get_gravatar_url(self.email, size=100)
282+
if self.email:
283+
return get_gravatar_url(self.email, size=100)
284+
return settings.GRAVATAR_DEFAULT_IMAGE
282285

283286
def delete(self, *args, **kwargs):
284287
"""Override delete method to clean up related resources."""

0 commit comments

Comments
 (0)