As suggested by: https://github.com/osTicket/osTicket-1.8/issues/755

The user should not have to step over it

Send an alert to assigned staff/team on an internal note even when a ticket is
closed.

Also use single quotes instead of double quotes in style attributes which
do not require escaping to "

This is necessary to avoid the confusion caused by group names like
"Managers" and "Staff".

If an email contains both inline and separate attachments, the previous
logic would detect all the attachments as inline.

Separate attachments have the `cid` set to `false`. This will pass the
`isset` test, which would cause the logic to search through the body for a
string of `cid:`, which would very likely be found if there was another
inline image somewhere in the body of the email.

With the advent of 1.9, the original, plain text email is saved in the
thread table. If a system has the HTML ticket thread enabled (which is the
default), if the plain text message is placed in the email template (which
is always HTML), it will not be formatted as correct HTML and the whitespace
will be collapsed.

This patch addresses the issue by always returning HTML from the asVar()
method, which is used to embed the thread post into a email template.