Use `data:` URIs to load assets in WCFSetup

With this change the HTML responses generated by WCFSetup are completely
self-contained and do not rely on external requests.

This avoids the proxying of the assets using PHP from the temporary folder at
the expense of slightly bloating the HTML with the embedded CSS.

The total size of the HTML is 530kB before gzip and 200kB after gzip.

Resolves #4394

Merge branch '5.4'

Removed a legacy condition used in the old session system

This is an old condition that dates back to the early days of the keep alive system. The condition does not work with the new session system anymore, because `lastActivityTime` is always mapped to `TIME_NOW`.

Merge pull request #4383 from WoltLab/article_embedded

Fix recursively loading embedded objects in articles

Merge pull request #4398 from WoltLab/smtp-options

Clean Up STARTTLS options

Merge branch 'master' into smtp-options

Merge pull request #4346 from mutec/currencydecimals

support custom number of decimals in currency modifier

Saving WebP images in GD requires a True Color palette

https://stackoverflow.com/questions/39292617/fatal-error-paletter-image-not-supported-by-webp/47469726#47469726

Merge pull request #4399 from WoltLab/mailform-captcha-option

Remove PROFILE_MAIL_USE_CAPTCHA

Actually check for TLS support in update_com.woltlab.wcf_5.5_starttls.php

Remove PROFILE_MAIL_USE_CAPTCHA

see 67e204a044077c2ad96685386aabaf6803a61d21
see #3032
see #3151

Merge branch '5.4'

Merge branch '5.3'

Use `emailConfirmed` status to exclude unconfirmed emails

With 5.3 we introduced the new `emailConfirmed` status to separate the status of the email and the activation status of the user. Therefore, we want to send emails to users, which emails are enabled. The user activation status is not relevant anymore for sending emails.

Merge pull request #4397 from WoltLab/favicon-is-processed

Improve Favicon handling in StyleAction

Devtools did not work correctly under Windows

Incorrect keystroke

Missing `px` units for the image dimensions

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

Missing cover photo in the list of articles in a footer box

Replaces #4378

Remove support for MAIL_SMTP_STARTTLS = 'may'

Resolves #4246

Merge branch '5.4'

Upgrade the default STARTTLS level to `encrypt`

see #4246

Update the default SMTP port to 587

25 is not meant for submission.

Explicitly mark styles for 5.2+ as compatible with 5.4

Regenerate Favicon images only when the template changes

Delete manifest.json and browserconfig.xml when deleting a favicon

Merge pull request #4384 from WoltLab/cronjob_xsd

Remove obsolete elements from `cronjob.xsd`

Merge branch '5.4'

Merge pull request #4392 from WoltLab/rename-totp-field

Rename TOTP code form field to `onetimecode`

support custom number of decimals in currency modifier

Rename TOTP code form field to `onetimecode`

The name `code` for the form field is too generic and should be specified. `onetimecode` is a more fitting identifier for the input name. In addition, 1password fills the totp code automatically, if the field is named `onetimecode` (See https://1password.community/discussion/comment/604200/#Comment_604200).

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'

Remove obsolete elements from `cronjob.xsd`

These elements are no longer (if ever) used.

Fix recursively loading embedded objects in articles

Close #4382
See 3af7b77d40ed93c95cebb22004ab745bf69683e2

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