This commit fixes an issue where referrals were being considered successfully saved even if that referral already existed, resulting in multiple thread events for the same referral.

Instead, if we find that the thread is already referred to the referee, refer should return false so that an error message is displayed stating that we were unable to refer the ticket.

Conflicts:
	scp/css/scp.css

This reverts commit 14bcd42d.

Custom Queues Merged for 1.11

This removes a significant number of queries used to render the drop-down
menu for the queues.

Each queue displayed on the page previously required a database query to
determine its children. This patch changes the strategy to fetch all the
queues and organize them as a tree. The tree can then be walked as the menu
is rendered and does not require any further queries.

On my test system, it reduces the number of queries for the ticket listing
page from 56 to 46.

This might happen if the close date was added to a queue and filtered
through a date/time filter.

Also fix warning emit when editing queue configuration

Also fix a few crashes if there is a naming mismatch on the field named in
search criteria and the current name of the field. And fix a regression
where the queue criteria would not be properly updated when saved.

If a queue inherits columns from a parent, the conditions would also be
inherited. This fix changes the association by linking the inherited
column to the child queue at run-time. This allows the queue column to
fetch the queue conditions from the child queue which inherited the
columns.

Check 'inherit columns' checkbox when cloning a queue

Fix doubled saved search listing

Help Topic Required Error Message:

issue: IE Select2 Target

issue: Select2 Placeholder Bug

issue: Saved Searches Flags

issue: Custom File Upload Dropping

This addresses issue 4155 where editing a ticket using a different session
causes custom file uploads to drop from the ticket.

This commit addresses an issue where the validation error for trying to save a Ticket without a Help Topic was not being shown as a result of the error message that was added to ensure that the Help Topic selected is Active.

This addresses an issue where Select2 Placeholders are being cutoff due to a
bug in Select2.

Setting - Require Help Topic to Close:

This commit adds an Admin setting that requires a Ticket to have a Help Topic before it can be closed. It can be set by going to:

Admin Panel | Settings | Tickets | Require Help Topic to Close

If enabled, Tickets will have a warning banner across the bottom of the page, a warning banner when trying to change the Ticket status to Closed, and a warning flag beside the field when editing the Ticket.

By default, this setting is NOT enabled.

This addresses an issue where Microsoft IE 11 apparently does not like
`event.target` syntax in javascript and breaks the Canned Response Select2.

This addresses an issue where class Queue sets SavedSearches flags to
`FLAG_QUEUE` causing the SavedSearches to become Primary Queues.

Email Name Format:

issue: Deleted Field Thread Events

This commit addresses an issue where email alerts were not being properly sent out if the name format is set as 'Last, First' or if it has special characters. We now encode names with quotation marks to ensure that emails are sent out properly (escape possible commas).

issue: Priority Field Template Variable

This addresses an issue where deleting a field and all it's data will
delete the form field record in the database causing all events containing
the field to crash the ticket page.

Help Topic Inline Save Fix:

This commit adds a validation for Help Topics that are saved using Inline Edit. In the event that a Ticket has a disabled Help Topic assigned to it, we want to display the name of the Help Topic, but we want to make sure that value cannot be saved.

In addition, it makes sure that if the Help Topic is disabled, it is the only disabled Help Topic that shows up in the list.

This addresses an issue with PriorityField Template Variables introduced
with 4359 where the priority variable will appear as "Array" instead of the
actual value. This adds an `asVar()` function to `class PriorityField` to
get the appropriate data.

issue: Redactor Select2 Cursor Position

This address an issue introduced with 4311 where selecting a canned response
does not insert it at last cursor position, rather at the top of the text
box.

Reset Button Fix:

Mailer Typo:

This commit fixes a mistake where a variable was incorrectly set. This in turn fixes the lint tests.

This commit makes sure that when the Reset button is hit, we only reset File Upload Field attachments for the specific form we're on.

Ex: Viewing a Ticket as an Agent and switching between 'Post Reply' and 'Post Internal Note'. If you had an image in the Post Reply file upload but switched to Post Internal Note and hit Reset, only the text/attachments on the Post Internal Note tab would be cleared.

(Improves upon #4369)