From f72c0576ef98592a4bd35aa927b4430b54d0fdbc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julien=20Fastr=C3=A9?= <julien.fastre@champs-libres.coop>
Date: Wed, 6 Jul 2022 21:36:37 +0200
Subject: [PATCH] check for user email before writing msgraph metadata

---
 .../Connector/MSGraph/MapCalendarToUser.php               | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/Bundle/ChillCalendarBundle/RemoteCalendar/Connector/MSGraph/MapCalendarToUser.php b/src/Bundle/ChillCalendarBundle/RemoteCalendar/Connector/MSGraph/MapCalendarToUser.php
index 158982fdd..7e7d55739 100644
--- a/src/Bundle/ChillCalendarBundle/RemoteCalendar/Connector/MSGraph/MapCalendarToUser.php
+++ b/src/Bundle/ChillCalendarBundle/RemoteCalendar/Connector/MSGraph/MapCalendarToUser.php
@@ -130,6 +130,10 @@ class MapCalendarToUser
 
     public function hasUserId(User $user): bool
     {
+        if (null === $user->getEmail() || '' === $user->getEmail()) {
+            return false;
+        }
+
         if (!array_key_exists(self::METADATA_KEY, $user->getAttributes())) {
             return false;
         }
@@ -139,6 +143,10 @@ class MapCalendarToUser
 
     public function writeMetadata(User $user): User
     {
+        if (null === $user->getEmail() OR '' === $user->getEmail()) {
+            return $user;
+        }
+
         if (null === $userData = $this->getUserByEmail($user->getEmailCanonical())) {
             $this->logger->warning('[MapCalendarToUser] could not find user on msgraph', ['userId' => $user->getId(), 'email' => $user->getEmailCanonical()]);