Return AvatarDecorator in UserProfile::getAvatar()

Implement ISafeFormatAvatar in UserAvatar

Add AvatarDecorator

Add ISafeFormatAvatar

Remove deprecated methods from IUserAvatar interface

Merge pull request #4002 from WoltLab/box-content-loading

Preload box content even for system boxes

Preload box content even for system boxes

Box::getBoxContentTitle() unconditionally loads the box contents for all box
types, because even system boxes might fall back to the box title that is
configured within the ACP.

This might result on O(n) queries when a page has several system boxes.

Fix this issue by preloading the box contents for all boxes.

Add 'name' to the contact form's `Reply-To` Mailbox

Merge branch '5.3'

Fix use of localized recipient email addresses in contact form

Update to setup-node@v2

Merge pull request #3991 from WoltLab/settings-section-anchors

Add anchors to account settings

Return the SystemException's description in extra information only if not empty

see 72ae641557703c600de12c07a04346ba16001251

Fix the visual marking of the markAll checkbox for Controller/Clipboard.ts

During the refactoring to TypeScript it was missed that after matching the
`.columnMark` class another access to `.parentNode` happens:

https://github.com/WoltLab/WCF/blob/6a623551e6d7a08ffdf7a623aa174e644af7a0d5/wcfsetup/install/files/js/WoltLabSuite/Core/Controller/Clipboard.js#L691

Fixes #3999

Fix calling UserAction::update() for 'counters' only

This fixes:
> Undefined array key "data"

see 21dd1c3dafe4cfdc31748da4aae3027d63316d70

Move `<meta charset="utf-8">` before `<title>` in WCFSetup

see #3802

Move `<meta charset="utf-8">` before `<title>` in exception handler

see #3802

Move `<meta charset="utf-8">` before `<title>` in install.php

see #3802

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge branch '3.0' into 3.1

Merge pull request #3997 from WoltLab/major-upgrade-no-upload

Reject major com.woltlab.wcf upgrades via package upload

Reject major com.woltlab.wcf upgrades via package upload

Merge branch 'master' of https://github.com/WoltLab/WCF

Added missing literal tag

Fix typo in comment in PackageInstallationConfirmPage

Add prefix to accountSecurity / settings section `id`s

Merge branch '5.3'

Stream the response in ImageProxyAction

see 5fc80baf9a348a63baaaa6fa150b1732adfc9f3c

Merge branch 'master' of github.com:WoltLab/WCF

Merge branch '5.3'

Merge branch '5.2' into 5.3

Fixes twitch clips which contain a dash char

Twitch clips can contain a dash (´-`).
Example: https://clips.twitch.tv/NeighborlyManlyBaconMikeHogu-_9zfXbRK47vik1WO

Resolves #3996

The mail notification type was unselectable

Merge pull request #3994 from WoltLab/simplify-notification-phrases

Simplify phrases for notifications

Updated privacy policy regarding profile information of third-party logins

Closes #3925

Merge branch '5.2' into 5.3

Fix deleting an empty list of objects

It can happen that content that was previously counted no longer exists and therefore fewer runs are required than initially assumed. This can happen if, for example, a user's posts and threads are deleted and the user has only written posts in his or her own threads (the posts are then deleted along with the threads). This may have led to errors before because an empty list of objects was passed.

Adjusted/unified html attributes for avatars

Use guest support of UserFunctionTemplatePlugin

See #3989

Simplify phrases for notifications

See #3958

Merge branch '5.3'

Merge pull request #3992 from WoltLab/export-mail-address-permission

Bind exportMailAddress bulk process to `canEditMailAddress` instead of `canMailUser`

Bind exportMailAddress bulk process to `canEditMailAddress` instead of `canMailUser`

Unfortunately the `permissions` form a logical disjunction, not a conjunction,
thus we can't check both permissions. The `canEditMailAddress` is more fitting,
because even without `canMailUser` the admin could simply look up all users
manually if they are allowed to edit (and thus see) mail addresses. It's just a
larger effort.

Resolves #3963

Make SystemException implement IExtraInformationException

This allows logging the `$description`.

Resolves #3944

Make use of anchors when linking to AccountSecurityPage

Add `id`s to sections in AccountSecurityPage

Add `id`s to sections in SettingsForm

Resolves #3972

Merge pull request #3990 from WoltLab/guest-user-function-template-plugin

Support guest users in UserFunctionTemplatePlugin

Merge pull request #3987 from WoltLab/visit-tracker-after-registration

Mark contents created before user registration as read

Merge pull request #3986 from WoltLab/wcfsetup-all-languages

Remove language selection from WCFSetup

Support guest users in UserFunctionTemplatePlugin

Closes #3989

Mark contents created before user registration as read

Resolves #3971

Remove language selection from WCFSetup

see #3969

Merge pull request #3962 from WoltLab/devtools-efficient-scan

Make DevtoolsPackageArchive::openArchive() more efficient

Add return types to anonymous functions

Merge branch '5.3'

Deprecate wcf\system\exception\HTTP*Exception

see 5ff8745012d43b1bcdc5df3c74a19532379d182f
see 33893f8e7201269f326117c3b1eed1af31106c06

Deprecate \wcf\util\exception\HTTPException

see 5ff8745012d43b1bcdc5df3c74a19532379d182f

Deprecated classes and methods related to the master password

see #3698

Remove unused imports

Use HttpFactory::makeClientWithTimeout()

Add HttpFactory::makeClientWithTimeout()

Use proper return types in HttpFactory

Use Guzzle's RequestOptions constants

These allow one to easily view the option's description within the IDE.

Replace GuzzleException by ClientExceptionInterface

ClientExceptionInterface is the standardized Exception and thus preferable.
GuzzleException extends that interface.

Make DevtoolsPackageArchive::openArchive() more efficient

Resolves #3830

Merge branch '5.3'

Merge branch '5.2' into 5.3

Additional template event in articleAdd template

Add `RewriteEngine On` to generated Apache mod_rewrite rules

Resolves #3930

Use unicode escape for bullets in BackupMultifactorMethod

Delete unused require.build-acp.js

Set optimize to 'none' for r.js

This was forgotten to commit while adjusting extra/*.

see e9fa719f372fb3aeb2fa1c1579699164f2ca7fe6
see #3952

Explicitly load tslib

This should resolve #3951, because this performs a blocking load.

Merge pull request #3952 from WoltLab/jsmin-ts

Make extra/* compatible with TypeScript / ES 6

Log user authentication failure type (#3950)

See #3395

Make extra/* compatible with TypeScript / ES 6

Merge pull request #3935 from WoltLab/use-getSubselectForObjectsByTags

Make use of TagEngine::getSubselectForObjectsByTags() in TaggedArticleList

Fix editing system pages

Fix codestyle

Use `User::getUserOptionID` instead of internal cache
Fixes #3936

Merge pull request #3947 from WoltLab/ts-template-circular

Remove the circular dependency between Core/Language and Core/Template

Consistently use "Attention" in English texts

See #3801

Mark Core/Template#compiled as readonly

Use Object.entries() in Core/Language#addObject()

Update MysqlSearchEngine::getFulltextMinimumWordLength() for InnoDB

At the same time copy over the deprecation comment from AbstractSearchEngine.

see 7d059540531e82c12b0aec2ad50e38a6788657cc
Resolves #3948

Constistently print "Achtung" in bold in German language item

See #3801

Consistently use "ungültig" in German language item

See #3801

Add `WCFMessageQuoteManager` interface to replace temporary `any` type (#3949)

Remove the circular dependency between Core/Language and Core/Template

The old work-around for this issue was no longer working well since the
migration to TypeScript. This commit untangles both modules, by each splitting
them into a low level and a high level interface.

The largest change is that language items are compiled when add()ing them to
the language store instead of when get()ting the contents.

This might slightly reduce the initialization performance on pages with a large
number of unused language items and it also might increase memory usage, due to
needing to store functions instead of strings.

It however improves the readability of the code and of course it also fixes
this breakage introduced by TypeScript. If it turns out that the change
actually *is* an issue then the logic can be optimized, e.g. by skipping the
template compiler if no `{` can be found within the phrase that is being
add()ed to the language store.

Merge branch '5.3'

Merge branch '5.2' into 5.3

Fix mixed boolean operators in condition in `TI18nFormField::validate()`

… and improve the preceding comment.

See #3926

Implement Language#addObject() in terms of Language#add()

Use monospace font for language item value fields in devtools

Close #3933

Update `.git-blame-ignore-revs`

Merge pull request #3937 from WoltLab/query_formatting

Improve formatting of SQL queries in PHP code

Split additional joins into multiple lines