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.

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

Split subquery into multiple lines

Remove unnecessary parentheses around join conditions

Make use of TagEngine::getSubselectForObjectsByTags() in TaggedArticleList

Commit c9613f130106f74853fbe9a50f06c00208106b8c is fun.

It renamed the getSqlForObjectsByTags() method that was previously used in
TaggedArticleList to the current name getSubselectForObjectsByTags() and fixed
a bug in the method that prevented it from working correctly.

At the same time the commit edited TaggedArticleList to no longer use this
method and instead copied over the logic.

I'm not sure what exactly happened during the development of that commit, but
we simply should use this (fixed) method.

Add missing parentheses

Merge pull request #3934 from WoltLab/box_pages_restricting_viewing

Inverted Permissions for Boxes and Pages

Add inverted permissions feature for pages

Add inverted permissions feature for boxes

Add support for inverted permissions with Simple ACL

Merge pull request #3904 from WoltLab/tag-to-object-languageID

Clean up wcf1_tag_to_object

Merge pull request #3928 from WoltLab/deprecated-updateStyleFile

Deprecate StyleUtil::updateStyleFile()

Explicitely configure `printWidth: 120` for Prettier

Merge branch '5.3'

Fix name of English language item

See 324aa486d32e0d1e7021385657fd1388d00403ca

Merge pull request #3929 from WoltLab/style-compiler

Clean up StyleCompiler

Remove redundant wcfFontFamily generation for ACP style

Fix the use of Google fonts in StyleCompiler

Use a `/*!` comment for the CSS header

This change stops the `Compressed` formatter from removing the comment,
allowing us to remove the `insertHeader()` logic which looked somewhat fragile.

Pass SCSS instead of files + invididualScss to StyleCompiler::compileStylesheet()

Make StyleCompiler::compileStylesheet() only compile

Add StyleCompiler::writeCss()

Add StyleCompiler::convertToRtl()

Add StyleCompiler::injectHeader()

Remove the StyleCompiler::$compiler property

State is bad. This method allows to prevent leaking compiler configuration
across multiple unrelated compilation processes.

Make the StyleCompiler `final`

Merge branch '5.3'

Download Google fonts before validating the individual SCSS

The individual SCSS will compile the style and this compiled CSS will later be
used for the style itself.

When first downloading a new Google font the font will not yet be available
during this test, thus the necessary CSS will not be included which in turn
will cause the font to fail to display.

Added missing parantheses

Stictly speaking this was a bug, but based on how the HTML is laid out, it will never cause any unexpected side effects. Fixing this for the sake of code clarity.

Deprecate StyleUtil::updateStyleFile()

Resolves #3927

Add explicit parentheses around mixed boolean operators in BooleanUserGroupOptionType

see #3926

Add explicit parentheses around mixed boolean operators in I18nPlural

see #3926

Fix reference URL for Unicode plural rules

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge branch '3.0' into 3.1

Fix typo in phrases

Support changing moderation queue status of closed entries (#3924)

Close #3227

Merge branch '5.3'

Merge branch '5.2' into 5.3

Ignore name of primary keys during creation (#3921)

Merge pull request #3918 from WoltLab/exporter-init

Initialize the exporter only once in ImportWorker

Fix bad merge / codestyle

When resolving the conflict in UserAvatarAction due to
621a83b075404f408dd025dfdc2cc55a43466536 it was forgotten to add the leading
backslash.

Merge branch '5.3'

Make 5.4_session_3_migrate_session.php idempotent

Fixes #3923

Long file name caused SQL error

Release 5.3.4

Merge branch '5.2' into 5.3

Release 5.2.12

Merge branch '3.1' into 5.2

Release 3.1.20

Merge branch '3.0' into 3.1

Merge branch '5.3'

Release 3.0.26

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge branch '3.0' into 3.1

Missing check for accessible user groups

Merge branch '3.0' into 3.1

Notify users of the expiring support

Merge branch '3.0' into 3.1

Ignore top-level node_modules/

Merge branch '5.3'

Merge pull request #3920 from WoltLab/http-request-proxy

Fix HTTPRequest with proxies

Stream HTTPRequest responses only of a maximum length is desired

cURL is not supported for streaming responses. As we process the full response
body for requests without a maximum length there is no benefit to streaming the
response, because we lose cURL support.

Update Guzzle

see guzzle/guzzle#2848
see guzzle/guzzle#2850

Merge pull request #3917 from WoltLab/cover-photo-webp-thumbnail

WebP support for cover photos