storage: Avoid using `strlen` to count bytes

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

Fix wrong ticket count on basic ticket search

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

bad links fixed

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

On validation errors, `$faq` will be set to something non-null, like
`false`, but will not be an instance of a Faq article.

For PHP installations that have `mbstring.func_overload` enabled (set to a
value including `2`), the `strlen` function will be overloaded to use the
`mb_strlen` equivalent. Problematically, the internal encoding of `UTF-8`
will be applied to all file content, which will count UTF-8 characters
rather than bytes. This will cause the data to be saved correctly; however,
the `size` recorded in the %file table will be recorded incorrectly.

This patch allows the backend to report the size of the contents saved with
the request and provides a failsafe mechanism which will use the mbstring
equivalent if available, and the mbstring version is coded to use the `8bit`
as the encoding which will prevent reading characters.

References:
https://github.com/osTicket/osTicket-1.8/issues/552

Basic search is now limited to ticket number, email address and name.

Show more drop down for users with edit permission

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

Users with edit permission should be able to change ticket owner in line.

Clear settings on department deletion

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

Clear ticket routing settings on topic, email or filter tables on department
deletion.

ui: Fixup unnecessary extra spacing in thread body

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

html: Avoid corrupting quoted style attributes

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

For instance, current this style we be converted as
```
<span style="font-family:'courier new';">
```
to
```
<span style="font-family:&quot;">
```

Also discard Microsoft Office specific style attributes such as `mso-list`
and friends

This stems from a confusing similarity between the + operator for arrays
and array_merge() in php. Adding arrays will ignore items in the RHS where
keys are present in the LHS. Therefore, when adding numerically indexed
arrays together, only items on the RHS that have a key higher than the
greatest key on the LHS will be included.

It is perfectectly fine to have an image cited in an HTML body similar to:

<img width=909 height=302 src=cid:image002.jpg@01CF5426.BF5A72A0 alt=image>

Which may or may not have quoted @src attribute, and may very well have an
at sign (@) somewhere in the attribute text. The previous regular expression
would not match such a @src attribute.

Inline images in email without a 'cid:' indicator

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

It is the defacto standard to reference an inline image in an email by
referencing the Content-ID header of the referenced attachment in the @src
attribute of the image. For instance, `<imc src="cid:image001.png">` where
the `cid:` scheme of the attribute indicates that the referenced image URL
is a separate content of the email. The image attached to the email would
have an accompanying header like: `Content-Id: <image001.png>`. However,
some mail systems, including a certain fax to email service, do not
correctly indicate the location of the image with the content-id URL.
Instead, the referenced image tag would be `<img src="image001.png">`

This patch addresses the issue by searching the message body for all
references to attached content-id's in all @src attributes with or without
the content-id URL scheme indicator. Previously, such images would not be
displayed inline in the ticket thread.

References:
https://tools.ietf.org/html/rfc2392

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

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

If the `view.php` page is requested and an auth token is not present in the
URL, do not attempt to sign the user on since it will always fail and will
result in a false "Failed login attempt" warning to the system log.

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

Allow regular staff members to show assigned open tickets on open queue

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

Previously only admins and department managers could show assigned tickets
on open queue -even- when the feature is disabled globally. This pull
request extends the same previlege to all agents.

email: Fix inline images sent in old HTML markup

Reviewed-By: @enricoandreoli
Reviewed-By: @sgozzi

Such images are attached to emails by Lotus Notes. Technically, these images
are valid attachments, although historically osTicket would reject them

Update class.osticket.php

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

Fix several bugs preventing file downloads

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

This is a regression introduced somewhere along the way in the new storage
API system for osTicket.