From 1b3690eeb7d4e76d9a4ee1e194d5c3f558be2ad4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Sat, 30 Oct 2021 02:25:37 +0200
Subject: [PATCH] Do not set empty avatar when generating one

As the method is called on write and is also calling write via save, we must
ensure to save only when an avatar is filled to break the loop.

issue10914
review363041002
---
 trytond/model/avatar.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/trytond/model/avatar.py b/trytond/model/avatar.py
index 9fb7e3e182..b607673605 100644
--- a/trytond/model/avatar.py
+++ b/trytond/model/avatar.py
@@ -62,7 +62,9 @@ def avatar_mixin(size=64, default=None):
             if not records:
                 return
             for record in records:
-                record.avatar = generate(size, getattr(record, field))
+                avatar = generate(size, getattr(record, field))
+                if avatar:
+                    record.avatar = avatar
             cls.save(records)
 
         if default: