Bug Fixes

Author: RusherRG

Diff for: .vscode/settings.json

@@ -20,13 +20,21 @@
 for participant in participants:
     # Checking if the participant was sent a mail previously
     if participant['email'] not in sent_mails:
+        name = participant['name']
+        email = participant['email']
+        phone = participant['phone']
+        payment_status = participant['payment']
+
         # Generating a message from the template
-        msg = _render_template(participant['name'], participant['payment'])
+        message = _render_template(name, payment_status)
+
         # Generating a custom image
-        image_path = pass_gen(participant['name'], participant['email'], participant['phone'])
+        image_path = pass_gen(name, email, phone)
+
         # Sending the message to the participant via mail
-        response = sendmail(to_email=participant['email'], message=msg, image_path=image_path)
+        response = sendmail(email, message, image_path, payment_status)
         print(response)
-        if response['email_status'] == "Success" and participant['payment'] == "paid":
+
+        if response['email_status'] == "Success" and payment_status == "paid":
             # if mail was sent successfully append the email to sentmails.txt
             write_file(participant['email'])

Diff for: automate.py

@@ -24,11 +24,13 @@ def write_csv(data):
 def read_file():
     # opening the file in read mode and reading lines
     with open("sentmails.txt", 'r') as f:
-        sent_mails = f.readlines()
+        sent_mails = f.read().split('\n')[:-1]
+        # Another way
+        # sent_mails = [mail.strip() for mail in f.readlines()]
     return sent_mails
 
 
 def write_file(mail):
     # opening the file in append mode and appending the mail at the end
     with open("sentmails.txt", 'a') as f:
-        f.write(mail + "\n")
+        f.write(mail + '\n')

Diff for: file_utils.py

@@ -4,7 +4,7 @@
 import smtplib
 
 
-def sendmail(to_email, message, image_path):
+def sendmail(to_email, message, image_path, payment_status):
 
     from_email = ""
     msg = MIMEMultipart('alternative')
@@ -16,10 +16,11 @@ def sendmail(to_email, message, image_path):
     msg.attach(content)
 
     # This example assumes the image is in the given directory
-    fp = open(image_path, 'rb')
-    msgImage = MIMEImage(fp.read())
-    fp.close()
-    msg.attach(msgImage)
+    if payment_status == 'paid':
+        fp = open(image_path, 'rb')
+        msgImage = MIMEImage(fp.read())
+        fp.close()
+        msg.attach(msgImage)
 
     response = {}
     try:

Diff for: mail.py

@@ -1,5 +1,5 @@
 Hi {{name}},
-{% if payment == 'paid' %}
+{% if payment_status == 'paid' %}
 This mail is to confirm your payment for the event. Attached with this mail is the pass which will be needed as proof for confirmation.
 {% else %}
 We noticed you have registered for our event. But, we haven't recieved any payments. Please make sure to confirm your presence by paying a refundable deposit.