First: Thanks for reviewing Jared!
Second: Re-checked all occurrences of `sprintf('%s %s` to make sure other translations are correct. Just found this one here to be not correct, so others are fine.

"This should be ´_S()´ because it is posting a permanent note to the task thread. Therefore should use the system language rather than the current user's language."

This commit updates a bunch of files and changes a lot of phrases. Basically it extracts "Internal error occurred" from existing phrases.
I've re-ordered the phrases so "Internal error occurred" is always at the end after the actual informative error message.

Example comparison before and after this commit:
Before: 1 Phrase: "Internal error: Unable to print to PDF"
After: 2 Phrases: "Unable to print to PDF." + "Internal error occurred"

Update three phrases with from "Internal error" to "Internal error occurred" since it's the most common phrase

Update two phrases with from "Internal error - Try again!" to "Internal error occurred - Try again!" since it's the most common phrase

Corrected "over-W-ridden" and removed the unnecessary "w" from the word "overridden"

Updated several files to split up phrases like "Your lock has expired. Please try again!" into "Your lock has expiredx" and "Please try again!"

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

email:  Regression for commit #3075 

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

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>