Merge branch '5.2' into 5.3

Fix hiding/showing tab menu entries due to form builder dependencies

Merge pull request #3810 from WoltLab/user-group-option-limited-removal

Prevent the unintentional removal of values for groups that are not being edited

Merge branch '5.2' into 5.3

Excempt the admin panel from application overrides

Fixes #3762

Prevent the unintentional removal of values for groups that are not being edited

Merge pull request #3807 from WoltLab/imagick-quantize-images

Greatly improve the performance of GIF processing with ImageMagick

Removed the legacy workaround for ancient ImageMagick versions

Typo

Prevent the trophy content being squished together

Greatly improve the performance of GIF processing with ImageMagick

Merge branch '5.2' into 5.3

Merge pull request #3799 from WoltLab/nonnull_in_primary_key

Validate primary indices with null columns in PHP DB API

Reload styleEditor after updating image path
Because the style is compiled after the variables have been set, we have to reload the style object once with the correct `imagePath` so that it can be used correctly in the initial compilation.

Remove obsolete PACKAGE_ID check in ImportCLICommand

Merge branch '5.2' into 5.3

Fix doubled validation errors for CaptchaFormField

Validate primary indices with null columns in PHP DB API

Close #3792

Restored the user online location for profiles

There has been a change in 5.3 that introduced the `$userOnline` value, however this is the target user itself and not the user's content they are viewing. The `$user` variable is useless, because it is a plain `User` object that is not compatible with `{$user}`.

Fix `wcf.user.register.needEmailConfirmation`

Fix `wcf.user.register.needEmailConfirmation`

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Force the use of the attachment thumbnail if the original is inaccessible

Workaround for the misaligned caret in Safari caused by 0x200B

Improved the iOS WebKit click detection inside the editor margin

Force a whitespace before and after inserted smileys

Merge pull request #3793 from WoltLab/style-import-permissions

Validate package installation permissions in StyleImportForm

Validate package installation permissions in StyleImportForm

Merge pull request #3786 from WoltLab/like-rebuild

Optimize LikeRebuildDataWorker performance

Only close the user panel drop-downs if no key modifier was pressed

Discard the current selection when saving a quote

Prevent the tooltip being selectable while the user changes the selection

Unified behavior of the notification list and the corresponding menu drop-down

Consistent width of menus in the mobile menu

Merge branch '5.2' into 5.3

Fix dependencies for single media form field

Add missing space after comma in ReactionHandler::renderInlineList()

Simplify selection of existing like objects in LikeRebuildDataWorker

Since the previous commit it is expected that only a single objectTypeID is hit
per worker execution in the usual case, because all objects for a single
objectTypeID are processed before proceeding to the next objectTypeID. This
patch optimizes this case by sending a separate query per objectTypeID to fetch
the existing like objects instead of combining all the objects in a single
query.

This should help MySQL in optimizing the query, because the objectTypeID (which
is the first column in the UNIQUE KEY on wcf1_like_object) is fixed for a
single query and this is immediately obvious without needing to scan all the
tuples passed to the IN(…) condition.

In the general case the number of queries does not change. In the iterations
where the last objectID for an objectTypeID is reached two queries will be sent
instead of a single one. Those two combined should be more efficient that the
single old query, though.

Allow LikeRebuildDataWorker to leverage the wcf1_like.objectTypeID key for sorting

Merge branch '5.2'

Ignore invalid file ids during deleting uploaded files
There can be several reasons why the files are no longer known. We can ignore this and simply return a success message so that it doesn't pop up in the frontend when unknown files are deleted.

Fix detecting images with capitalised file extensions

Only rebuild favicons if there is a new image uploaded

Fix updating pageLogo in StyleAction

Improved wrapping of reaction filter buttons on mobile

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Incorrect version comparison for accessible packages

Prevent rewriting filenames for processed files

Fix copy and paste error in en.xml

see 9e2ccd976daa8646b2b86f4d27e52d3f226ff94e

Merge branch '5.2' into 5.3

Merge pull request #3763 from WoltLab/http-request-timeout

HTTP request timeouts

Set an explicit timeout for ImageProxyAction's HttpClient

Set an explicit timeout for FontManager's HttpClient

Replace HTTPRequest's timeout by connect_timeout + read_timeout

The timeout in 5.2 only applied to the connect() syscall. Guzzle's timeout
option applies to the total transfer. Replace it by connect_timeout +
read_timeout for backwards compatibility for code that expected the actual
transfer to take an arbitrary amount of time.

Fix typo in de.xml

Update the orphanedComments remark in package.xml

Fix typo in informal German phrases

Verweide -> Vermeide

Closes #3752

[Tim: rephrased the commit message]

Improved randomized sort order of related articles

Release 5.3.1

Merge branch '5.2' into 5.3

Missing check for empty object ids

Merge branch '5.2' into 5.3

randomized sort order of related articles

Changed related articles to new layout

Release 5.2.10

Merge branch '3.1' into 5.2

Release 3.1.18

Use modified package servers when running in enterprise mode

Merge branch '5.2' into 5.3

Whitelist `iterator_count` in enterprise mode

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge pull request #3744 from Krymonota/patch-18

Add missing JSDoc to `EmailSmtpTest`

Merge pull request #3745 from Krymonota/patch-19

Fix JSDoc module typo in `RewriteGenerator`

Merge branch '5.2' into 5.3

Fix point calculation for removing reactions from an object

Release 5.3.1

Fix JSDoc module typo in `RewriteGenerator`

Add missing JSDoc to `EmailSmtpTest`

Force the removal of orphaned comments during the upgrade

Updating minified JavaScript files

Merge branch '5.2' into 5.3

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Updating minified JavaScript files

Workaround for a Chromium bug on Windows

Verbose style compatibility for 5.2/5.3

Fixes #3737

Permit images uploaded as media to be wrapped in a link

Merge branch '5.2' into 5.3

Clean up comments when removing moderation queues (#3740)

* Delete comments when moderation queues are being removed

See #3707

* Clean-up script for orphaned comments

* Use the `CommentHandler` to remove comments

* Apply suggestions from code review

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

Enabled three column layout for boxes top / bottom

Skip DOM updates when the article is not present

Merge pull request #3739 from WoltLab/fix-style-preview

Fix style preview images when upgrading from 5.2

Fix style preview images when upgrading from 5.2

Fixed html in acp search results

Add note about the email_html.tpl update to package.xml

Merge branch '5.2' into 5.3

Replace the selector for the font-family in email_html.tpl

The Emogrifier version used in 5.2 does not fully support all features
of the `:not()` selector. Depending on the environment the use of
`:not()` results in slightly different output. Specifically the number
of `:not()`s chained there caused the result to have a very large
specificity, possibly overriding the font-size of the headline tags.

Use `body, body *` instead, it will not put any useless CSS onto heading
tags and still catch all relevant elements. The biggest difference is
that the `<br>` tag will carry a `font-family` and `font-size`.