This commit includes several final fixes in general that needed to be made.

PHP 7.2 Compatibility

jq: Update jQuery UI References

This commit makes osTicket compatible with PHP 7.2

Conflicts:
	include/class.filter.php

- Pull latest code from osTicket
- Fix Filter ORM compatibility issues that resulted from merge

Oops: Modify 0 in Short Answer Field Fix

This updates the references to the old jQuery UI filename to the new
filename. (this needs #4672 merged first)

Fixes an unexpected side effect of Pull Request #4652

profile: Reply Redirect

Conflicts:
	scp/login.php

Issue: Can’t use “0” in Short Answer Field

This commit fixes an issue where a Short Answer Field would show as Empty if the value saved in it was '0'

Issue: Edit Export Column Heading

issue: Task Response With Collaborators

issue: Scrollable Quickfilters

Task Collaborators

Issue: Duplicate Search Results

Improve the Staff login

Issue: Tasks Within Tickets

This pull adds a Profile Setting for Agents to choose where to redirect
after replying to ticket. The two options are Queue (redirect to queue to
work on other tickets) or Ticket (redirect back to same ticket to continue
working) with the latter being the default. The setting is configurable for
each Agent in the helpdesk.

issue: Hide Task Loading Overlay

This addresses an issue where having more than 30+ choices for a Quickfilter
makes the dropdown run off the page causing some choices to be hidden. This
makes it to where if you have more than 20+ choices we will make the
dropdown a fixed height and make it scrollable.

This addresses a fatal error when adding a new response to a Task with
Collaborators. When we implemented the MailingList for Users/Collaborators
we forgot about the little Task Collaborators (poor little guys). This adds
a new function called `getRecipients()` to `class Thread` that utilizes
MailingList instead of UserList. This also adds two new functions to `class
Util` that retrieves the User’s `user_id`.

This addresses an issue where the Loading overlay on Tasks will not
disappear after the request was submitted. This adds two lines to first hide
the "Loading" modal and then toggle the overlay.

Peter approved merge

- Make sure we only set Filter Flags if they need to be changed (don't update them every time we save)
- Make sure we sure we use the email_id if we have actually set it. Otherwise, we lose what should be set for the Target Channel

Codebase Updates

This commit fixes an issue where we did not correctly change the values passed to the render function that was updated to work with PHP 7.1

This commit ensures that we do not return duplicate results when searching for Tickets with an Advanced Search that contains keywords.

issue: Dashboard Export Period

Clarify User Import Instructions

This commit addresses issues we had with viewing Task(s) within a Ticket:

1. When viewing the table of all Tasks on a Ticket, the Options dropdown should only allow the Agent to Reopen or Close the Tasks based on the status of the Tasks.
Ex: If there is only 1 Open Task, you should only see the 'Close' Option
    If there are 2 Closed Tasks, you should only see the 'Reopen' Option
    If there are multiple Tasks in which some are Open and some are Closed, you should see both the 'Reopen' and 'Close' options

2. When viewing an individual Task within a Ticket, the status options were the opposite of what they should have been. Now, if the Task is Open, the Agent will see the option to 'Close' the Task. If the Task is Closded, the Agent will see the option to 'Reopen' the Task.

This commit updates the instructions for User imports letting the Agent know they should type 'Name, Email' on the first line and then put each new User Name/Email on the following lines to import Users.

This addresses an issue where exporting statistics via the dashboard within
a certain timeframe (eg. 07/01/2018 -> One Quarter) will always export the
statistics from the selected date up to today. This is due to the Period
option always being set to "Up to today" when the Export button is clicked
which in return exports the stats from the selected start date to today.
This adds JS to set the period the Agent selects so the Export will return
the stats from the selected date to the selected period.

This commit addresses an issue where we did not save headings for export fields.
We just need to leave the heading in the array rather than filtering it out so that we can save its new heading.

This commit changes the wording for Collaborators on Tasks to match the wording for Collaborators on Tickets.

oops: markAs() Typo

issue: markAs() elseif