Change to make one translation less since `%s is currently assigned to <b>%s</b>` is already used in ajax.tasks.php and ajax.tickets.php

In addition to 71a6b2a0 splitted "this department" here.

files: Require authentication to view attachments

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

upgrader: Split change and drop into 2 statements

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

This feature adds a setting to the control panel to require signing in to
view attachments. This is in addition to the security already provided in
the download URLs. Currently, download URLs are signed for a specific help
desk, and automatically expire after about 24 hours. The exact timing is the
following midnight allowing for at least 12 hours cache time.

Administrators can impose this extra security feature to refuse serving
attachment files if the user is not currently signed in. This could prevent
third-party users from viewing an attachment if they were able to get access
to the download URL before it expired.

Department  Access and Membership

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

* Separate primary from extended access members when listing department
  access.

* Add ability to update primary department members role

* Quick staff add - inherit role of the primary department as default role

Add ability get sorted department members. Extended members are sorted, annotated
and cached.

Consolidate agents name sorting in Staff::nsort. The routing expects a query
set.

isMember routine wrongly assumed getMembers returned staff_id indexed
hashtable.

lint: Allow $this inside anonymous functions

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

Session: Create SessionData object on demand

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

Update class.auth.php

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

closed-by: Credit the agent opening a ticket on close

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

Session backend issues update via write on new session which caused
assertion to fail.

If the Return-Path header does not match the From header, emails are significantly more likely to be blocked by spam filters.

This patch forces the return header to match the from address without having to change php.ini and therefore works properly on servers that have multiple apps sending from multiple email addresses for example shared hosting, multiple support emails in one instance etc.

oops: Fix call to missing create(), only fetch user form once

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

Email aliases

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

files: FileUploadWidget render

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

Add callback filter to standard iteration method to skip deleted items.

Use file id instead of attachment id to unset new
Get new files objects as array

orm: Partially revert 8ab4432f

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

orm: Remove references to the retired slot feature

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

org: Remove users from deleted org.

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

files: Attempt to standardize thread entry attaching

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

oops: Fix manual sort on help topic admin listing page

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

org: Propagate LEFT joins in join paths

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

Credit @Jared

Include username when checking if a system email already exists. This
is important for emails that use aliases.

Use org_id to filter users to update on organization delete

Within this commit, several files got updated to split up strings like "Disable for this team" / "Disable for this department" into "Disable for" and "this team"/"this department".
So a) a little bit less translations and b) all strings of "this (ticket|task|API|canned|FAQ|depart|email|group|site|SLA|agent|team|template|help|ban|category|custom|end user|role)" can match already existing "this ..." translations now.
PS: Didn't remove all that whitespaces in include/staff/tpl.inc.php - seems like my php editor app has done that automatically :/

Note: Since I'm not sure if that's all (replacing %s %s with %1$s %2$s) which is necessary to make this work, I'm sorry in advance for a potential "bad commit".

As already written on Crowdin, in German the order is wrong, so my suggestion is to make it possible to change the order by transforming "%s %s" to "%1$s and %2$s".
Format in English: Successfully assigned 4 selected tasks
Needed German: Successfully 4 selected tasks assigned

forms: Fix lag updating the CDATA table

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