From d3aa2e7239a8013808f00dfb4bedc99d5ab494ed Mon Sep 17 00:00:00 2001
From: johnnyq <johnny@pittpc.com>
Date: Sat, 20 Jan 2024 16:45:39 -0500
Subject: [PATCH] Only allow A-Za-z- for Ticket Prefix

---
 cron_ticket_email_parser.php | 3 ++-
 settings_ticket.php          | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/cron_ticket_email_parser.php b/cron_ticket_email_parser.php
index 702a62e3..9d9cd316 100644
--- a/cron_ticket_email_parser.php
+++ b/cron_ticket_email_parser.php
@@ -255,7 +255,7 @@ function addReply($from_email, $date, $subject, $ticket_number, $message, $attac
             
             addToMailQueue($mysqli, $data);
 
-            return false;
+            return true;
         }
 
         // Check WHO replied (was it the owner of the ticket or someone else on CC?)
@@ -517,6 +517,7 @@ if ($emails) {
             imap_setflag_full($imap, $email, "\\Seen");
             imap_mail_move($imap, $email, $imap_folder);
         } else {
+            // Basically just flags all emails keep them unread and it doesnt move closed tickets
             echo "Failed to process email - flagging for manual review.";
             imap_setflag_full($imap, $email, "\\Flagged");
         }
diff --git a/settings_ticket.php b/settings_ticket.php
index ad699b67..29afe15b 100644
--- a/settings_ticket.php
+++ b/settings_ticket.php
@@ -18,7 +18,7 @@ require_once "inc_all_settings.php";
                         <div class="input-group-prepend">
                             <span class="input-group-text"><i class="fa fa-fw fa-life-ring"></i></span>
                         </div>
-                        <input type="text" class="form-control" name="config_ticket_prefix" placeholder="Ticket Prefix" value="<?php echo nullable_htmlentities($config_ticket_prefix); ?>">
+                        <input type="text" class="form-control" name="config_ticket_prefix" placeholder="Ticket Prefix" value="<?php echo nullable_htmlentities($config_ticket_prefix); ?>" pattern="^[A-Za-z-]+$" title="Only letters and hyphens are allowed" required>
                     </div>
                 </div>