Merge branch '5.3' into 5.4

Sync client-side poll validation with server-side validation

`PollManager` marks an empty question field as empty if poll options are given, while `WCF.Poll.Management` aborted validation once the question field is empty, no matter whether options are given or not.

Handle poll question validation errors for quick replies

Unescape BBCode attributes on editor load

Fix SearchEngineConvertInnoDbWorker when MySQL is not the active engine

Fixes #4192

Update developer documentation links in PIP info phrases

Close #4266

Merge branch '5.3' into 5.4

Fully clickable notification items

Fix iconContainer identifier

Merge pull request #4263 from Fabii547/patch-76

Fix visibility check in `Condition/Page/Dependence`

Fix visibility check in `Condition/Page/Dependence`

Merge pull request #4259 from WoltLab/session-ip-address

Mask IP address in session list

Merge pull request #4258 from WoltLab/userAgent-final

Mark `\wcf\util\UserAgent` as final

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

Code style fix

Use `->toBulletMasked()` in `accountSecurity.tpl`

Add `IpAddress::toBulletMasked()`

Rename `IpAddress::mask()` to `IpAddress::toMasked()`

Show a masked IP address in accountSecurity.tpl

Return an IP address in Session::getIpAddress()

Add `\wcf\util\IpAddress`

Fix SCSS code style

Improved UX and visuals of embedded URLs

The link now has a proper title and the tab focus will highlight the headline.

Merge branch '5.3'

Merge branch '5.2' into 5.3

Add missing `wcf.acp.devtools.pip.error.className` phrase

Close #4256

Mark `\wcf\util\UserAgent` as final

Merge pull request #4257 from WoltLab/abstract-event-listener-error

Make error message in AbstractEventListener more useful

Make error message in AbstractEventListener more useful

Merge branch '5.3'

Ensure the style logo is properly preserved in the database when not performing changes

This bug was introduced recently when the `!$file->isProcessed()` condition was
added. It was verified that the logo's filename remained stable within the file
system. However it was not verified that it was properly stored within the
database. As the previous style variables are deleted during the update we must
explicitly set the corresponding variable, otherwise it will be empty and the
logo will not be found.

see c72c06dc7bb21c5098613ab8fafc8596c8f6ce41
see #4240

Fix displaying `runStandalone` in devtools (#4255)

Fix ids of instruction items in devtools (#4254)

Add 5.4 branch to GitHub workflows

Remove obsolete meta keywords input for monolingual pages (#4251)

See #3637

Release 5.4.0 Beta 1

Remove obsolete private property/method from `MediaImporter`

See b3b5e461e0291d25d06f8d2b07bcdc18a4c5a391

Incorrect property name

Deploy some last minute changes

Merge pull request #4248 from WoltLab/improved-article-bbcode

Overhauled article bbcode using embedded content

Use an `<img>` element with lazy loading for embedded content

Helper methods to query the style's cover photo dimensions

Merge branch 'master' into improved-article-bbcode

Add `labeledUrl` column to `update_com.woltlab.wcf_5.4.php`

Fixes #4249

Fix CSS formatting

Overhauled embedded content design

Generic styling for inline lists without list elements

Fixed code style errors

Overhauled article bbcode using embedded content

Overhauled embedded content design

Release 5.4.0 Beta 1

Removed the upgrade support from 5.3.*

Updating minified JavaScript files

Merge pull request #4245 from WoltLab/meta-description-system-pages

Add custom meta description for system pages

Merge branch '5.3'

Merge pull request #4247 from WoltLab/usersOnly-guest

Skip usersOnly permissions for the guest group during permission merge

Skip usersOnly permissions for the guest group during permission merge

The value of usersOnly permissions cannot be configured for the guest group
within the ACP, but the stored default value is taken into account during the
permission merge in UserGroupPermissionCacheBuilder nonetheless. This might result
in unexpected permissions for accounts that are not yet activated.

As an example taking away the “Can use conversations” for all groups might
still allow accounts that are not yet activated to use the conversations,
because of this preserved default.

Fix this issue by simply skipping the value of the guest group during the
merge, thus ignoring whatever value may be stored. This is safe, because the
resulting option list always includes the “Everyone” group.

see https://community.woltlab.com/thread/289879/

Wrong version number in exception during setup

Incorrect handling of empty values for `accept`

Add custom meta description for system pages

Truncate SMTP replies in error messages to 250 characters instead of 80

Incorrect position of the quote tooltip on touch devices

Incorrect import of meta keywords for CMS pages

Merge pull request #4244 from WoltLab/ajax-invoke-direct

Send HTTP 405 on direct access to an AJAXInvokeAction

Send HTTP 405 on direct access to an AJAXInvokeAction

Update version number in devtools configuration filename to 5.4

Increase the default font size and darken the text color a bit

Titles of MFA methods now link to the management section

Explicit dimensions for language icons

Merge pull request #4242 from WoltLab/fix-setup-layout

Fix layout of WCFSetup

Fix incorrect sized logo

Fix layout of WCFSetup

Fixes #4215

Merge branch '5.3'

Fix first custom smiley category not being preloaded

Merge pull request #4240 from WoltLab/pageLogo-stable-name

Do not process the pageLogo in StyleAction::updateVariables() if it is unchanged

Do not process the pageLogo in StyleAction::updateVariables() if it is unchanged

This change ensures that the filename of the logo remains the same, unless the
file actually is modified.

A fix for this issue was initially attempted in
cc17f302902fb62c78b4411dadc3243913f18c74. However that commit was much more
complex and it introduced additional bugs, because it no longer passed an
explicit `null` for removed files, thus breaking the update of the related
database columns. It was subsequently reverted in
dd7c98858d406f8ca9824770ae8a68edde451493.

Merge branch '5.3'

Reduce size of og images

Merge pull request #4239 from WoltLab/style-edit-reverts

Revert broken commits in Style Editor

Revert "Prevent rewriting filenames for processed files"

This change is bogus, because it completely changes StyleAction's API and it
introduces bugs (namely not updating the database). One of these was fixed in
88b962c25b4c2dd0c312c6f7e4c11f15a1263cf4, but other fields are also affected.

As this commit doesn't fix an actual bug, but only a small inelegance (namely
the logo's path changing whenever the style is changed, even if the logo is
unchanged) this can be safely reverted until a proper fix is developed.

This reverts commit cc17f302902fb62c78b4411dadc3243913f18c74.

Revert "Fix updating pageLogo in StyleAction"

This was a fix for the broken cc17f302902fb62c78b4411dadc3243913f18c74 which
will also be reverted.

This reverts commit 4b2f4354e6fab91ea1d7b38d7009f1f7f990d357.

Revert "Only rebuild favicons if there is a new image uploaded"

This builds upon the broken cc17f302902fb62c78b4411dadc3243913f18c74 which will
also be reverted.

This reverts commit 9bed2ebfa4e250a00794cdc75755f379dc17a696.

Revert "Cover photo deletion was not saved properly"

This was a fix for the broken cc17f302902fb62c78b4411dadc3243913f18c74 which
will also be reverted.

This reverts commit 88b962c25b4c2dd0c312c6f7e4c11f15a1263cf4.

Fix `@since` tag for StyleAction::updateCustomAssets()

see 908e40a36a87ba7039c067cd8343087d701bef70

Merge pull request #4238 from WoltLab/php7.1-compat

Remove PHP 7.1 compatibility layer in AvatarUploadFileValidationStrategy

Trim trailing whitespace

Properly delete all variants of style cover photos

This needed some workarounds to properly account for the WebP variant.

Update `$newLocation` after creating the WebP variant of a style's cover photo

This ensures that the StyleEditForm always uses the non-WebP version for processing.

Use the non-WebP version of the cover photo in StyleEditForm

Recognize IMAGETYPE_WEBP in StyleAddForm

Remove PHP 7.1 compatibility layer in AvatarUploadFileValidationStrategy

Fix detection of WebP images in `ImageUtil::createWebpVariant()`

Fix `$forceWebP` parameter in `UserCoverPhoto::getLocation()` and `::getURL()`

Add missing backslash in `Attribute` attribute for `\wcf\SensitiveArgument`

Update tslib to 2.2.0

Merge pull request #4236 from WoltLab/user-rebuild-memory-leak

Fetch UserProfiles live to avoid memory leaks through the RuntimeCache on CLI

Fix code formatting

Merge branch '5.3'

Ignore disabled smiley categories for WYSIWYG form field

Fix code formatting