Imrove internal notes posting
Improve ticket assignment & alerts

 e.g %{staff.*} can be overwritten by a string (placeholder) as assign('staff', 'SYSTEM').

Improve variable's preg_match_all (Thanks to Jared)

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

Self merging.

Feature/new scp icons

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

Feature/client side less

Don't migrate attachments as part of the 1.6* major upgrade. Wait unit
transitioning to schema signature dd0022fb, where the new table for
file_chunks is created.

Incremental developers will already have the columns dropped from the
%ticket_attachments table and will receive an error for the cleanup script,
but the upgrader is configured to ignore errors from the cleanup scripts.

This will remove the upper limit of BLOB sizes imposed by MySQL with the
max_allowed_packet setting completely.

This adds a new table %file_chunk which will contain the filedata in smaller
chunks (256kB). It also includes a new class, AttachmentChunkedData, which
will handle reading and writing the data, abstracting away the chunks.

This is done by migrating data from the %file table to the %file_chunk
table. One must beware that this must safely (the migration that is) plug
into the both the live osTicket developers as well as the users doing a full
upgrade from osTicket-1.6*. For this, the AttachmentFile::save method was
patched to use the new AttachmentChunkedData class to write the attachment
data to the database in chunks. That is, the migrater will use the new code
on the major upgrade and bypass the filedata column of the %file table
altogether. Therefore, the patch associated with this commit will not
migrate any data for the major upgrade.

For developers doing incremental upgrades, the patch included in this
commit will transfer the data from the %file data to the new %file_chunk
table by chunking it. As written, only the first 16MB of the attachment is
migrated. This could easily be adjusted, but it seems like a reasonable
limit for now.

Fix missed table name change - a bug on new installations

Reviewed-By: Peter Rotich <peter@osticket.com> [self reviewed dev hot fix]

Allow empty variable - or whatever objects return

Feature/filters revisited

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