include @greezybacon suggestion on fix

Fixes #2350 in `develop`

orgs: Show user search box adding user to org

Peter Rotich <peter@osticket.com>

Fixes #2410

This patch drops the content after the last boundary in a multi-part message
which prevents it from being considered in some case where the available
content sections are not favorable.

Backport me to v1.9

search: Fix malformed results on remote user search

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

Revert "db: Fix ticket queue counts for large datasets"

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

org: Fix matching of empty domain name

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

search: Fix ticket number search on client portal

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

This reverts commit c4ace2d5.

If a user is created with an empty domain name (e.g. <user@>), then the user
would be associated with the first organization without a main domain
specified.

This patch ensures that the domain is not empty when making a match with an
organization.

xss: Properly escape choices when rendered

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

This fixes an issue where, when the mailfetch subsystem fetches more than
one email and one of them has a TNEF attachment; each message body after the
TNEF message will use the TNEF message body.

This patch fixes the issue by clearing the TNEF body in each cycle through
the message fetching and parsing loop.

search: Auto reindex when _search is created

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

This patch automatically configures reindexing of the search table when the
table is dropped and recreated.

variable: The HTML balancing system urlencodes @Hamsteren 

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

The DomDocument::saveHtml() method will properly urlencode() values in the
a[@href] attributes when saving. This patch allows for the sanitization
while also allowing the variable replacement system to work properly with
the sanitized values.

oops: Fix missing data from new form entries

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

cli: Fix incorrect option parsing

Peter Rotich <peter@osticket.com>

When using the php mail() function, the To: header encoded in the Mail_mime
library was not properly transferred to the $to variable passed to the mail
backend. The mail() function will add its own To: header which will
mimic the received destination address.

This patch ensures that the $to parameter is properly encoded.

Pass data to the callabale by reference

Reviewed-By: Jared Hancock <jared@osticket.com>

This fixes a regression in the option parsing where --setup would be
interpreted as --setup -e -t -u -p

Also, some parts of osTicket, like internationalization and plugins need
read access to a directory in order to get a list of installed plugins and
language packs.

When using call_user_func_array arguments should be passed by reference in order to allow callable receive them by reference (only defining callable to receive data by reference is not enough).

Fixes #2143

Ensure OoO message does not clear closing agent

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

This patch will help ensure that if a ticket is not reopened on a message
(perhaps because of a out-of-office auto-reply from an enduser), then the
closing agent on a closed ticket should not be cleared.

This fixes an issue where text scrubbed with the new balancing system would
receive a leading ' <div>' if the text started with a leading whitespace.