Show warning when trying to send email to zero recipients

diff --git a/pgcommitfest/commitfest/templatetags/commitfest.py b/pgcommitfest/commitfest/templatetags/commitfest.py
index ae5af0c4081e09bccb231b3112ec694bddd43085..07b5ea4efe93712639f623d6f941a8a52e7b08d0 100644 (file)
--- a/pgcommitfest/commitfest/templatetags/commitfest.py
+++ b/pgcommitfest/commitfest/templatetags/commitfest.py
@@ -24,6 +24,8 @@ def field_class(value, arg):
 def alertmap(value):
        if value == 'error':
                return 'alert-danger'
+       elif value == 'warning':
+               return 'alert-warning'
        elif value == 'success':
                return 'alert-success'
        else:

diff --git a/pgcommitfest/commitfest/views.py b/pgcommitfest/commitfest/views.py
index 2556cc37a7ebc7d73d36c9d402fbfa9863e47ca2..f34fd54040d47ce1d0c5c6201c060d374413891c 100644 (file)
--- a/pgcommitfest/commitfest/views.py
+++ b/pgcommitfest/commitfest/views.py
@@ -456,6 +456,10 @@ def send_email(request, cfid):
        else:
                reviewers = []
 
+       if len(authors)==0 and len(reviewers)==0:
+               messages.add_message(request, messages.WARNING, "No recipients specified, cannot send email")
+               return HttpResponseRedirect('..')
+
        messages.add_message(request, messages.INFO, "Email will be sent from: %s" % request.user.email)
        def _user_and_mail(u):
                return "%s %s (%s)" % (u.first_name, u.last_name, u.email)