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

Fix table borders when cells with `rowspan` are visually in the last row (#4176)

See https://community.woltlab.com/thread/289598

Fix undefined class variable

Fixes #4163

Fix collision for comment react button and comment response react button

The comment react button can bind onto the comment response react button, if there is no comment react button (e.g. because the comment author is the user themselves). Using an own `buttonSelector` for the comment response react button, fixes the problem.

Note: The problem only becomes visible if there is only one reaction (so WoltLab Suite uses the legacy like mode).

Fix typo in NewPasswordForm

Fixes #4164

Merge branch '5.2' into 5.3

Add missing return value of `readValue()` of DevTools project form fields

Remove thumbnail generation from MediaImporter

This is useless, because it can be performed using MediaRebuildDataWorker. It
might also break the import process for borked source images.

see 44f9dba4c98359b53679cdac3bb43982e716a7e1

Merge branch '5.2' into 5.3

Do not write empty i18n package information in `package.xml`

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge pull request #4143 from WoltLab/3.1-fix-sitemap

Fix generating sitemaps with factor 250 objects sitemaps

Fix generating sitemaps with factor 250 objects

Merge branch '5.2' into 5.3

Only validate minimum text length if any text is given

The case of no text having been entered and the field being required is already handled separately.

Merge pull request #4133 from WoltLab/article-update-outdated

Ensure that the creating of related rows uses up to date article fields

Merge pull request #4135 from WoltLab/httprequest-close-body

Always close Guzzle response body after reading it

Always close Guzzle response body after reading it

Missing parameter in pagination of the article list

Consider user list box conditions when using cache builder internally

Ensure that the creating of related rows uses up to date article fields

Previously the code would store outdated data in the search index and recent
activities when changing the article's timestamp while publishing the article
or when changing the article's author.

Merge pull request #4127 from WoltLab/fix-user-storage-handler

Fix requesting multiple times the UserStorage tables if no cache exists

Fix requesting multiple times the UserStorage tables if no cache exists

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Update the number of responses that will be loaded

Release 5.3.7

Fix missing space in user registration email confirm mails

Fix newline handling in user registration notification mails

Fix `UserNotification::markAllAsConfirmed()` for no unconfirmed notifications

See 81982c717bbb55c36306ede938dc189f01f85592

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

Release 5.3.6

Release 5.3.14

Merge branch '5.2' into 5.3

Updating minified JavaScript files

Release 5.2.14

Merge branch '3.1' into 5.2

Release 3.1.22

Add missing permissions check when removing moderated content

Improved visuals for code boxes on small screens

Only show top add button on empty captcha question list page

Close #4072

Correctly handle a missing .htaccess in the tmp/ folder

see be81c8401ba16bd1f84fe95f8ca71c372cf548ec

Merge pull request #4118 from WoltLab/db_fk_future_tables

Properly handle foreign keys referencing tables created later on

Properly handle foreign keys referencing tables created later on

If a foreign key of the first table references a table to be created later on, this foreign key can only be created once the referenced table was created.

Check if foreign key references unknown database table

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

Avoid one disk write with every request

Merge branch '5.2' into 5.3

Fix `reloadPageOnSuccess` from different clipboard actions for same object type

If there are multiple clipboard action classes for the same object type, for example in case a plugin adds further clipboard actions, only the `reloadPageOnSuccess` data of the first clipboard action instance is used instead of collecting the data from all relevant clipboard action instances.

See #2584

Merge pull request #4116 from WoltLab/html-processing-performance

Improve the performance of the HTMLOutputProcessor

Use `XPath::query()` for its superior performance

Use `XPath::query()` for its superior performance

`XPath::query()` is much more efficient than `getElementByTagName()`

The test document with ~8k elements took ~0.6s using `getElementsByTagName(*)`, but only 0.03s using `query("//*")`

Inline calls in `removeNode()` to improve the simplified-html/plain performance

Avoid updates to DOMNodeList for childNodes

Avoid live updates to DOMNodeList

The DOMNodeList is a live collection that is updated whenever an item is removed. This can have a significant performance impact when removing a large set of nodes.