Also add resend only option (for the case when "Edit and Resend" is not
allowed).

This caches the explosion of the foreign key constraints, which eliminates
the need to unnecessarily explode the foreign key constraint again and again
while building queries. This represents about a 15% speed increase building
large queries.

Add in the ability for an agent to resend a response. Optionally editing the
response before sending it, and setting the signature as is possible with
the usual responses.

When the response is resent, the edited version is marked as GUARDED, and
subsequent edits will result in new links in the history chain. That is,
when a response is edited and resent by an agent, that response will remain
in the history chain.

The new role would like be assigned to power users and would all such users
access to edit any post by any user.

Agents can always edit their own posts, and department managers can edit all
posts while the ticket is in the department they manage.

Threads can be edited by marking the original as hidden and setting it's PID to
the id of the new entry. The new entry has cloned data from the original and
sets the `updated` timestamp to reflect the time of last edit. An edited flag
is added to the new entry to reflect its origin.

This patch suggests that agents can edit their own posts, department managers
can edit posts while the ticket is in their department, and that help desk
admins can edit anything.

If a post is edited more than once, only the most recent edit is kept.

Conflicts:
	include/class.dept.php
	include/class.dynamic_forms.php
	include/class.format.php
	include/class.mailfetch.php
	include/class.staff.php
	include/class.thread.php
	include/class.user.php
	include/client/profile.inc.php
	include/staff/directory.inc.php
	include/staff/staffmembers.inc.php
	include/staff/tickets.inc.php

oops: Fix bad HTML entity #2

Reviewed-By: Peter Rotich <peter@osticket.com>

Fix image CID rewrite for some pasted images

Reviewed-By: Peter Rotich <peter@osticket.com>

users: Update information on CSV import

Reviewed-By: Peter Rotich <peter@osticket.com>

i18n: Translate gb2312 to gbk charset

Reviewed-By: Peter Rotich <peter@osticket.com>

html: Ensure text internal note preserved

Reviewed-By: Peter Rotich <peter@osticket.com>

Preserve text formatting on the internal note posted to a new ticket by
staff if HTML thread is disabled.

This is necessary for some mail clients which indicate encoding in GB2312
but actually use GBK encoding.

auth: Add concept of bk passwd update and policy

Reviewed-By: Peter Rotich <peter@osticket.com>

tnef: Fix various TNEF processing issues

Reviewed-By: Peter Rotich <peter@osticket.com>

email: Properly detect bounce message with alternative content

Reviewed-By: Peter Rotich <peter@osticket.com>

fix translation for "Add new list"

Reviewed-By: Peter Rotich <peter@osticket.com>

i18n: Consider current locale for CSV delimiter

Reviewed-By: Peter Rotich <peter@osticket.com>

Fix a few translatable phrase issues

Reviewed-By: Peter Rotich <peter@osticket.com>

mailer: Embed the Message-Id token in @class

Reviewed-By: Peter Rotich <peter@osticket.com>

Ensure consistent name formatting for agents and end users

Reviewed-By: Peter Rotich <peter@osticket.com>

email: Support agents emailing as collaborators

Reviewed-By: Peter Rotich <peter@osticket.com>

This correctly handles a bounce message with the following layout:

multipart/report; report-type=delivery-status
    multipart/alternative; differences=Content-Type
        text/plain; charset="us-ascii"
        text/html; charset="us-ascii"
    message/delivery-status
    message/rfc822

Conflicts:
	include/class.dynamic_forms.php
	include/class.mailer.php
	include/class.organization.php
	include/class.orm.php
	include/class.user.php
	include/client/templates/dynamic-form.tmpl.php
	include/client/view.inc.php
	scp/lists.php

Fixes #1854

Feature :: Dynamic Forms 1.1 — address major confusion and magic issues

Reviewed-By: Peter Rotich <peter@osticket.com>

This feature addresses a major issue with the initial implementation of the
custom data system. The original system confused the usage of
database-backed field (dynamic-fields) and their corresponding
implementation. This created the need to crate awkward caching pieces to
ensure that validation errors and data was maintained. Furthermore, the
system confused the linking between form instances (dynamic-entry) and the
form used to represent that entry.

This patch addresses the confusion in two ways:

Dynamic form entries do not link directly to the dynamic form. Instead, the
::getForm() method returns something from the forms API directly.
Furthermore, the ::getFields() method does not return dynamic field
instances (database backed / designed fields). Instead, the actual
implementation of the fields from the forms API is retrieved. This allows
the fields to *always* be cached, which helps preserve data and validation
state.

Secondly, the dynamic form uses the same system, so that requests to turn a
dynamic form into a form (via ::getForm) will also result in the same
behavior, again, where the fields are represented as forms API fields rather
than the dynamic fields.

So going forward, the dynamic fields are *only* used to create corresponding
forms API field implementations. The are associated with the dynamic
counterparts as sparingly as possible.

Several mail clients do not return the Message-ID header in a In-Reply-To or
References header as they should. The assumption by such mail clients is
that the email be threaded based on the Subject header. However, osTicket
does not require the ticket number to be placed in the Subject header and so
has trouble threading email from such systems. osTicket embeds a copy of the
Message-ID header (or compatible version) in the body of the message in
hopes part of the message will be returned.

Many mail clients (such as osTicket) strip and clean the HTML when
processing HTML email. Previously, the message-id token was embedded in a
@data-mid attribute, which was likely stripped before the HTML email would
be returned back to osTicket.

This patch suggests that the token be placed in a @class attribute, which
has a much better chance of returning to osTicket.

Follow-up PR to the comment here:
https://github.com/osTicket/osTicket-1.8/commit/06d3a0f3e2a4557ca72faa916859b9315635dfe4