cron_ticket_email_parser.php - send new ticket opened email to unknown clients

[cybershieldms]

• Will now send unknown client (client_id = 0) a ticket open notification email
• Added regex to not parse bounce e-mails (e.g. From: postmaster, mailer-daemon)
• Fixed ticket description missing from new ticket notifications sent to others.
• Added E-Mail From line display name to beginning of ticket description/e-mail quote with e-mail address and date.

[wrongecho]

Test these changes at: https://ticketfeatures1076.pr-review.itflow.org
(automatic message)

[wrongecho]

Hey @cybershieldms

Thanks for the PR! :)

I've tested and this seems to work nicely. I like the way you've put the persons name in bold and email in brackets too!

Great concept with excluding bounces but I'd suggest we exclude admin/reply as these can often be used legitimately/for messages you want.

[sonarcloud[bot]]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[cybershieldms]

Great concept with excluding bounces but I'd suggest we exclude admin/reply as these can often be used legitimately/for messages you want.

@wrongecho - Thanks! First time with Github. Done... Changed the regex to just include postmaster and daemon.

In the future, I could see a configurable variable in Settings to make a customizable block list.

[johnnyq]

@cybershieldms looks good Great first commit @wrongecho I think everything looks good, I approve the merge just wanted to get any last minute thoughts

[wrongecho]

@wrongecho - Thanks! First time with Github. Done... Changed the regex to just include postmaster and daemon.

In the future, I could see a configurable variable in Settings to make a customizable block list.

You've done great for your first time!! :)

Agreed, it would be nice to make the ignore list configurable.

[johnnyq]

awesome, we'll pull this one in

[wrongecho]

Nice work Craig!! :)