From 1078327eca49296fa57c05915dd6cb7cdd785b89 Mon Sep 17 00:00:00 2001
From: Dale Wahl <dalewahl@gmail.com>
Date: Thu, 7 Sep 2023 16:28:19 +0200
Subject: [PATCH] do not default to yes

---
 backend/lib/processor.py                            |  3 +--
 webtool/templates/components/processor-details.html | 13 +++++++++----
 webtool/templates/create-dataset.html               |  2 +-
 3 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/backend/lib/processor.py b/backend/lib/processor.py
index fbe67b208..137a8f958 100644
--- a/backend/lib/processor.py
+++ b/backend/lib/processor.py
@@ -332,8 +332,7 @@ def after_process(self):
 				try:
 					send_email([owner], message)
 				except (SMTPException, ConnectionRefusedError, socket.timeout) as e:
-					self.log.error("Sending email to %s" % owner)
-
+					self.log.error("Error sending email to %s" % owner)
 
 	def remove_files(self):
 		"""
diff --git a/webtool/templates/components/processor-details.html b/webtool/templates/components/processor-details.html
index d84b215bb..8e0955fb7 100644
--- a/webtool/templates/components/processor-details.html
+++ b/webtool/templates/components/processor-details.html
@@ -61,10 +61,15 @@ <h4>{{ processor.title }}</h4>
                 {% for option in processor_options %}
                     {% include "components/processor-option.html" %}
                 {% endfor %}
-            {% if __user_config("mail.server") and '@' in current_user.get_name() %}
-                <div class="processor-option">
-                    <label for="email">Email address</label>
-                    <input type="email" name="email" id="email" value="{{ current_user.get_name() }}" placeholder="Email address">
+            {% if __user_config("mail.server") %}
+                <div class="form-element">
+                    <label for="data-email-complete">Receive email on completion:</label>
+                    <div class="filter-parameters">
+                        <label><input type="checkbox" name="email-complete" id="data-email-complete"></label>
+                        <label><input id="data-email-user" type="email" name="email-user" value={{ current_user.get_name() }}></label>
+                        <button class="tooltip-trigger" aria-controls="tooltip-dataset-email" aria-label="Extended help for option">?</button>
+                    </div>
+                    <p role="tooltip" id="tooltip-dataset-email">This will only function if your username is your email address.</p>
                 </div>
             {% endif %}
             </fieldset>
diff --git a/webtool/templates/create-dataset.html b/webtool/templates/create-dataset.html
index d698b825b..0607c9a4f 100644
--- a/webtool/templates/create-dataset.html
+++ b/webtool/templates/create-dataset.html
@@ -64,7 +64,7 @@ <h2><span>Create new dataset</span></h2>
                     <div class="form-element">
                         <label for="data-email-complete">Receive email on completion:</label>
                         <div class="filter-parameters">
-                            <label><input type="checkbox" name="email-complete" id="data-email-complete" checked=""></label>
+                            <label><input type="checkbox" name="email-complete" id="data-email-complete"></label>
                             <label><input id="data-email-user" type="email" name="email-user" value={{ current_user.get_name() }}></label>
                             <button class="tooltip-trigger" aria-controls="tooltip-dataset-email" aria-label="Extended help for option">?</button>
                         </div>