Merge branch '5.4'

Convert tabs to spaces

Merge branch '5.3'

Merge branch '5.2'

Merge branch '3.1'

Don't leak an account's activation status (#4365)

* Don't leak an account's activation status

* Update wcfsetup/install/files/lib/form/RegisterNewActivationCodeForm.class.php

Co-authored-by: Tim Düsterhus <timwolla@googlemail.com>
Co-authored-by: Tim Düsterhus <timwolla@googlemail.com>

Merge branch '5.4'

Merge branch '5.3'

Set new activation code on rerequesting them

Merge pull request #4389 from WoltLab/email-serialize-size

Reduce the size of the Mailbox objects

Reduce the size of the Mailbox objects

This introduces a small BC break for classes inheriting from Mailbox or
UserMailbox that directly access the `language` or `user` properties instead of
the getter methods. But this is easily fixed.

Resolves #4387

Release 5.4.0 RC 2

Merge pull request #4385 from WoltLab/package-update-compat

Do not store package compatibility from update servers within the database

Do not store package compatibility from update servers within the database

These do not appear to be used anywhere and the official package servers do not
expose them either, making them effectively empty even in current versions.

see #4371

Merge branch '5.4'

Add update_com.woltlab.wcf_5.4_styleCoverPhoto.php to package.xml

Do not use the cache in update_com.woltlab.wcf_5.4_styleCoverPhoto.php

Fix variable name in `ViewableArticleContentList::readObjects()`

Do not block matching abbreviation if the package is the same

The previous code would block the installation of an update of the same package.

Fix for #4357

Release 5.4.0 RC 2

Updating minified JavaScript files

Improved the visuals of the list of MFA options

The content is now vertically centered and the somewhat tiny smartphone icon is enlarged to better match the other icons.

Use `<small>` instead of `<p class="small">`

Avoid use of escapeString() in UserNotificationEventAction

This part of the code appears to be completely unused, but the same logic can
be found in UserNotificationEventPackageInstallationPlugin, so it should be
correct.

Merge pull request #4371 from WoltLab/api-compat-no-store

Do not store API compatibility information within the database

Merge pull request #4381 from WoltLab/gzopen64

Remove gzopen64() workaround

Remove gzopen64() workaround

PHP bug #53829 (https://bugs.php.net/bug.php?id=53829) is fixed. This should no
longer be required.

Mark wcf1_package_compatibility as deprecated in install.sql

Remove list of compatible API versions in PackagePage

Clear wcf1_package_compatibility when upgrading to 5.5

Do not store API compatibility information within the database

These are automatically converted into an exclude of Core 6.0.0 Alpha 1 in
PackageArchive and thus redundant.

Merge branch '5.4'

Fix `this` value for `Core.enableLegacyInheritance()` (#4380)

Without explicitly binding `this` to `thisValue`, it will be `constructed` resulting in missing data only present in `thisValue`.

Merge branch '5.4'

Explicitly show error message if import queue is empty

Close #3804
See #4379

Merge branch '5.4'

Merge branch '5.3' into 5.4

Fix reading value of `CheckboxFormField`

In contrast to `BooleanFormField` for which an explicit "no" input field exists, a normal checkbox does not submit any data if no checked.

Merge pull request #4376 from WoltLab/rebuild-active-app

Remove the need to rebuild the active application

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

Incorrect type cast to `number`

Fix conversion of `Dictionary` to `Map` in `Acp/Ui/Menu/Item/Handler`

See 2727575fbe8a2ca2fe4a19288c7a982969170882

Search for articles toggled the incorrect container

Explicitly delete obsolete style logo description phrases

See dcb0fe8ec9560f4a7905d9fd1241bc72d1a5c337

Merge branch '5.4'

Merge branch '5.3' into 5.4

Remove obsolete style logo upload field descriptions

See #3456

Merge pull request #4375 from WoltLab/controller-workaround

Remove old workaround for the upgrade from 2.1 to 3.0

Merge pull request #4370 from WoltLab/devtools-api-compat

Remove support for the API compatibility tags from dev tools

Merge branch '5.4'

Fixed missing language version for embedded articles

Use temporary variables for `Request` objects in FontManager

This is for consistency with other code sending HTTP requests and to shorten
the lines, as one of them exceeded 120 characters.

Merge branch '5.4'

Merge remote-tracking branch 'origin/5.4' into 5.4

Merge branch '5.3' into 5.4

Add missing encoding of `$family` in font download requests

Remove the need to rebuild the active application

Calculating this value live does not come with a relevant performance loss and
simplifies the request handling logic.

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

Remove old workaround for the upgrade from 2.1 to 3.0

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 support for the API compatibility tags from dev tools

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