From a67841f80db475cfcb2e5c73dbe944db6c043fbf Mon Sep 17 00:00:00 2001 From: Horilla Date: Sat, 17 May 2025 14:51:18 +0530 Subject: [PATCH] [IMP] EMPLOYEE: Optimize avatar loading by replacing external API calls with local static image fallback on employee profile image --- employee/models.py | 15 ++++----------- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/employee/models.py b/employee/models.py index b6da15d27..7caf35345 100644 --- a/employee/models.py +++ b/employee/models.py @@ -16,6 +16,7 @@ from django.db import models from django.db.models.query import QuerySet from django.db.models.signals import post_save from django.dispatch import receiver +from django.templatetags.static import static from django.utils.translation import gettext as _ from django.utils.translation import gettext_lazy as trans @@ -223,17 +224,9 @@ class Employee(models.Model): ) def get_avatar(self): - """ - Method will retun the api to the avatar or path to the profile image - """ - url = ( - f"https://ui-avatars.com/api/?name={self.get_full_name()}&background=random" - ) - if self.employee_profile: - full_filename = self.employee_profile.name - if default_storage.exists(full_filename): - url = self.employee_profile.url - return url + if self.employee_profile and default_storage.exists(self.employee_profile.name): + return self.employee_profile.url + return static("images/ui/default_avatar.jpg") def get_leave_status(self): """