From 8b84b8f41d479039d0a5fe31bd3d2e5e231539bf Mon Sep 17 00:00:00 2001
From: Fuhu Xia <1392461+FuhuXia@users.noreply.github.com>
Date: Fri, 12 Aug 2022 00:59:39 -0400
Subject: [PATCH] add recipient only if there is email

---
 ckanext/harvest/logic/action/get.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/ckanext/harvest/logic/action/get.py b/ckanext/harvest/logic/action/get.py
index 011dd668..964d2b19 100644
--- a/ckanext/harvest/logic/action/get.py
+++ b/ckanext/harvest/logic/action/get.py
@@ -437,10 +437,11 @@ def harvest_get_notifications_recipients(context, data_dict):
     ).all()
 
     for sysadmin in sysadmins:
-        recipients.append({
-            'name': sysadmin.name,
-            'email': sysadmin.email
-        })
+        if sysadmin.email:
+            recipients.append({
+                'name': sysadmin.name,
+                'email': sysadmin.email
+            })
 
     # gather organization-admins
     if source.get('organization'):