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>

Set new activation code on rerequesting them

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.

Remove obsolete style logo upload field descriptions

See #3456

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

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Remove duplicate `class` attribute in cacheList.tpl

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

Add check for GD support to PackageEnableUpgradeOverrideForm

Add check for GD support to PackageEnableUpgradeOverrideForm

Validate the `actionName` in MessageQuoteAction

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>

Release 5.3.10

Updating minified JavaScript files

Sorting in article category had no effect

Add support for version upgrades (#4160)

* Add WCF::AVAILABLE_UPGRADE_VERSION

* Expose the available upgrade version to packageList.tpl

* Show notice in packageList if an upgrade is available

* Add upgrade override support to PackageUpdateServer

* Show notice about enabled upgrade override in packageList and packageUpdateServerList

* Add PackageEnableUpgradeOverrideForm

* Support disabling the upgrade override if there are issues

* Add version checks from update_com.woltlab.wcf_5.4_checkSystemRequirements.php to PackageEnableUpgradeOverrideForm

* Add language items for WoltLab Suite upgrades

* Clear package lists when toggling the upgrade override

* Add requested-woltlab-suite-version header to package list requests

* Improved the messages and restrict the upgrade to the maintenance mode

* Restrict the experimental upgrade to the maintenance mode

* Removed the upgrade notice from the package server list

Co-authored-by: Alexander Ebert <ebert@woltlab.com>

Fix poll insertion after message edit if no poll exists

Previously, an empty `div.jsInlineEditorHideContent` element was added.

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge branch '5.2' into 5.3

Fix typo in de.xml

Use “Moderation” instead of “Moderatoren” in de.xml

Reset link-related fields after creating box

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Set 'accept-encoding: identity' by default in HTTPRequest

RFC 7231#5.3.4 allows a server to use an arbitrary 'content-encoding' when no
'accept-encoding' request header is sent.

Make it clear that no 'content-encoding' is acceptable by only accepting the
'identity' encoding.

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix 'to its full extent' in en.xml

Make `WysiwygAttachmentFormField` only available if upload is allowed

The same check exists in the non-form builder template `messageFormTabs.tpl`.

Pages in the Template format were not imported correctly

Add ES 5 syntax check (#4291)

Fix links to developer documentation in phrases

Release 5.3.9

Updating minified JavaScript files

ES6+ syntax trips the legacy UglifyJS compiler

Release 5.3.8

Updating minified JavaScript files

Incorrect removal of formatting when pasting as plain text

Fixes #4276

Delay the caret positioning on iOS

Rely on the dialog's auto focus with proper iOS handling

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

Fully clickable notification items

Merge branch '5.2' into 5.3

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

Close #4256

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

Wrong version number in exception during setup

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

Incorrect import of meta keywords for CMS pages

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 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

Ignore disabled smiley categories for WYSIWYG form field

Fix media clipboard inserting when having multiple editors

Ensure that the clipboard insertion event listener is only added once and that the active media manager is set in the clipboard handler when opening a media manager window.

Ensure uniqueness of media insert dialogs when having multiple editors

Preserve deleted filter when using article list pagination in ACP

Incorrect badge selector

Fix email in Apple Mail with darkmode enabled

Fix #4188

Add missing `since` tag

Properly position errors below fields in `.inputAddon` elements (#4219)

Incorrect redirect url after login in multi domain setups

Incorrect fade out effect in iOS Safari

Fix attachment ids in metacode elements during import (#4213)

* Fix attachment ids in metacode elements during import

Close #4212

* Update wcfsetup/install/files/lib/system/importer/AbstractAttachmentImporter.class.php

Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>
Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>

Do not delete media files if file is replaced with itself

Do not open media manager dialog after replacing media file

… and always reload the media list page in the ACP after editing/replacing media file to ensure updated data.

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge pull request #4209 from WoltLab/comment_moderation_queue

Delete moderation queue entries after deleting comments/comment responses

Delete moderation queue entries after deleting comments/comment responses

Close #4208

Add missing `$definitionName` value for comment response moderation queue handlers

Merge branch '5.2' into 5.3

Preserve article sorting when using pagination

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix ignoring the disable state of an article category

Deactivated categories are currently only hidden in the overview of categories in the frontend. However, articles in these categories (and the category itself, via the direct link) were still accessible. This patch solves the problem by hiding the category for all users and making articles and the category itself no longer accessible for any user.

Properly reset internal state in image viewer widget

See #3835

Merge branch '5.2' into 5.3

Fix error messages of numeric form fields for minimum/maximum

Add missing template events for monolingual pages

See #3268

Fix position of `messageFieldsMultilingual` event in `pageAdd.tpl`

The message tabs have to be below the relevant WYSIWYG editor. Inserting additional fields at this position would split them apart.

Merge branch '5.2' into 5.3

Add missing `license` element in `package.xsd`

Incorrect position of icon overlays on `screen-md`

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Reset language form field in language import form after saving

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix detection of nested tables in messages

Close #4175