Simplify extraction of abbreviation for active application check

Merge branch '5.4'

Add missing since tag for `User::requiresMultifactor()`

Move call to `checkAppEvaluation()` into `handle()`

It fits better here to cleanly separate the handling from the building. No
behavioral change was found.

Move the check for expired app evaluations into a dedicated method

Make the evaluation check independent of local variables

Simplify extraction of abbreviation for evaluation check

Add `@since` to RequestHandler::checkOfflineMode()

see 60fff547e76067211270f9f5eb7da9f257f847fc

Move the check for offline mode into a dedicated method

Use `getActiveRequest()` method in favor of directly accessing the class property

Use `isACPRequest()` method in favor of a local variable in RequestHandler::handle()

Remove unused local variable `$appendSession` in LinkHandler::getLink()

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Remove duplicate `class` attribute in cacheList.tpl

Support sorting users in `UserOnlineListBoxController` (#4372)

Close #4182

Remove obsolete import

Remove cleanup of `userID` and `password` cookies

They are no longer used as of WoltLab Suite 5.4, however the removal remained
in the code to clean up the browser's cookie store on logout. Remove it, the
cookies will naturally expire in the future.

Set `optimize-autoloader: true` in composer.json

Merge pull request #4369 from WoltLab/composer

Update composer dependencies

Update fileDelete.xml for composer updates

Update composer dependencies

Merge branch '5.4'

Check for WebP support in update_com.woltlab.wcf_5.4_checkSystemRequirements.php

Merge branch '5.3' into 5.4

Merge pull request #4368 from WoltLab/upgrade-check-webp

Add check for GD support to PackageEnableUpgradeOverrideForm

Add check for GD support to PackageEnableUpgradeOverrideForm

Export `WCFMessageQuoteManager` interface in `Ui/Message/Quote`

See #3876

Merge branch '5.4'

Merge branch '5.3' into 5.4

Validate the `actionName` in MessageQuoteAction

Merge branch '5.4'

Fix adding pages if only one language is available

See 128dbea9130f7decfb255d0d6e04f83e291021e6

Merge branch '5.4'

Fix code formatting issue

Merge branch '5.4'

Ensure that app abbreviation is unique before app installation

Close #4344

Fix path of database PIP file located in `files_wcf/` for DevTools syncs

Close #4359

Make renaming columns with PHP DB API idempotent (#4367)

An error message should only be shown if the neither a column with the old name, nor with the name name exists so that after the rename, a second rename is a no-op.

Close #4362
See #3765

Merge branch '5.4'

Merge branch '5.3' into 5.4

Also applied the second change from #4366.

Co-authored-by: Sascha Greuel <sascha@softcreatr.de>

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix typo in de.xml

Closes #4366

Co-authored-by: Sascha Greuel <sascha@softcreatr.de>

Add missing `@since` to AbstractOauth2Action

Make AbstractOauth2Action::getHttpClient() non-final

It might be desirable to be able to add custom headers, specifically a custom
`user-agent`, for non-stock IDPs.

Replace use of `WCF.Clipboard` with `Controller/Clipboard`

See #3876

Merge branch '5.4'

Merge pull request #4356 from WoltLab/user-logged-in

Add UserLoggedIn event and cancel lost password requests upon login

Add cancelLostPasswordRequest to UserAction

Cancel lost password requests when the user logs in

Resolves #3922

Implement the UserLoggedIn event

Add a UserLoggedIn event

Merge pull request #4353 from WoltLab/ColorFormField

Modernize color picker and add `ColorFormField`

Support wildcard for category name in `AclFormField` (#4355)

Fix TOTP QR code in dark styles

Release 5.3.10

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge pull request #4354 from WoltLab/systemcheck-performance

Improve performance of SystemCheckPage::validateWritableDirectories()

Improve performance of SystemCheckPage::validateWritableDirectories()

The previous implementation resulted in a syscall hell for large attachment or
image directories. RecursiveDirectoryIterator requires two syscalls for
directory entry just for iteration and that does not yet include any custom
logic.

Replace the implementation by a simple `glob()`. This is a small behavior
change, as a `*` will only check the files immediately below the directory in
question instead of recursing all the way down. However this likely is what was
intended anyway.

Updating minified JavaScript files

Add `ColorFormField`

Add new phrases to JavaScript for existing uses of color pickers

Deprecate `WCF.ColorPicker` in favor of `Ui/Color/Picker`

Update `Ui/Color/Picker` to use native TypeScript-based implementation

Instead of having to implement a color picker ourselves, we delegate the task to the browser by using a `input[type=color]` element. Because this color input does not support transparency, there is an additional `input[type=range]` element for transparency.

Modified update instructions

Add `ColorUtil.stringToRgba()`

Add `ColorUtil.isValidColor()`

Add `ColorUtil.rgbaToString()`

Add `ColorUtil.rgbaToHex()`

Use `.toString(16)` for `ColorUtil.rgbToHex()`

Export color interfaces from `ColorUtil`

Merge pull request #4352 from WoltLab/twitter-auth

Modernize Twitter Auth

Merge branch '5.4'

Handle Facebook not providing an email address

Fixes #4241

Correctly handle Twitter not providing an email address

Migrate Twitter authentication to Guzzle

Improve readability of OAuth header generation in TwitterAuthAction

Use `getControllerLink()` to generate TwitterAuth callback URL

Use constant time b64 encoder in TwitterAuthAction

Use `\hash_equals()` for token validation in TwitterAuthAction

Fix use of unused parameter in `AbstractOauth2Action::codeToAccessToken()`

Make `failure` parameter of `Popover.ajaxApi()` optional

There is no reason to make the custom failure callback a required parameter here because in most cases, there is no need fur such a custom handler.

Merge remote-tracking branch 'origin/master'

Merge branch '5.4'

Improve speed of PIPs to delete files

… by fetching all existing database records at once and deleting obsolete records in one transaction.

See #4269
See #4289

Register reauthentication after MFA in ACP only

It was not desired that a login within the frontend also registers a
reauthentication, as the actions requiring a reauthentication are very rarely
used and can benefit from this additional password check. In the ACP however
each request requires a reauthentication, thus it makes sense to register it
during login.

see 4624ceeda0e19109245d19e883dc6fd388e95519

Merge branch '5.4'

Merge pull request #4350 from WoltLab/update-server-db

Adjust the URL of the update servers in install.sql

Merge branch '5.4'

Merge branch '5.3' into 5.4

Sorting in article category had no effect

Adjust the URL of the update servers in install.sql

This is technically unnecessary as the proper URLs will be inserted during
runtime, but might improve debugging, as it is clear which version was used to
install the software.

Merge branch '5.4'

Add acpTemplate PIP to update instructions from 5.4.0 Beta 2

Use RejectEverythingFormField in PackageEnableUpgradeOverrideForm

Clear WCF::AVAILABLE_UPGRADE_VERSION