Merge branch '3.1' into 5.2

Improved visuals of the fullscreen template comparison

Fixes #3473

Merge branch '3.1' into 5.2

Fix trophy icon size in badges on groupedUserTrophyList

Duplicate `<br>` inserted on paste

Fixes #3454

Workaround for Safari when removing the text formatting with a collapsed selection

Merge branch '3.1' into 5.2

Adjusted appearance of the "sign in with google" button

Merge branch '3.1' into 5.2

Reopened dialogs were hidden behind new dialogs

Incorrect blacklisting of `register_activation_method`

Prevent the focus being trapped if the user intentionally clicks outside

Clicking on a link or button outside of the overlay caused the browser to close the drop-down and shift the focus on the menu element.

Merge pull request #3525 from WoltLab/rich-metacode-in-quick-reply

Prevent rich metacode being dysfunctional in new messages

Merge pull request #3523 from WoltLab/dropdown-scrollbar-large-tablets-fix

Fix the scrollbar for dropdowns on large tablets

Prevent rich metacode being dysfunctional in messages just created by replying

The `isset` strategy does not work properly, because in some cases the HTML is evaluated before being placed in the final template, causing the variable to be set already.

This changes modifies the behavior to always attempt to initialize the JavaScript, but made changes to ensure that each subsequent call is idempotent.

Fix the scrollbar for dropdowns on large tablets
Because tablets are not the platform "desktop", the scrollbar is not shown for tablets, even though they are rendered with desktop view.
Fixes #3522

Disable the search for updates buttons while active

Merge pull request #3521 from Krymonota/patch-13

Remove redundant `implements IWorker`

Remove redundant `implements IWorker`

Fix deprecation warnings in form builder in PHP 8

Close #3489

Updating minified JavaScript files

Prefer longer group names for mentions

Skip display-name of Mailbox if display-name == addr-spec

If the human readable $name of a Mailbox matches the $address then leave
out the $name within the header representation and specify a bare email
address. The name is redundant in cases like this and this might lead to
negative spam scoring.

Merge branch '3.1' into 5.2

Merge pull request #3500 from Krymonota/patch-12

Add missing phrases to English 3rd party register language items

Fix resizing images in Safari
Fixes #3506

Merge branch '3.1' into 5.2

Prevent username overflow in the message sidebar

Merge branch '3.1' into 5.2

Prevent changes to the showOrder setting of system boxes during an update

Merge branch '3.1' into 5.2

Prevent changes to the visibility settings of system boxes during an update/upgrade

Merge pull request #3502 from Krymonota/timing-safe-comparison-social-login

Use timing safe comparison to validate `state` parameter for social login

Fix type of `options` parameter in HTTPRequest PHPDoc (#3504)

Use timing safe comparison to validate `state` parameter for social login

The Twitter social login is left out because the implementation still uses OAuth 1.0, which does not support the `state` parameter.

Closes #3501

Add missing phrases to English 3rd party register language items

Merge pull request #3499 from WoltLab/5.2-like-rebuild-data-fix

Fix calculation the cached reactions

Use proper placeholders for fetching the reactions

Fix calculation the cached reactions

Merge pull request #3498 from WoltLab/dbo-action-php-8

Fix bogus call to `call_user_func_array` in AbstractDBOAction::validateAction()

Fix bogus call to `call_user_func_array` in AbstractDBOAction::validateAction()

Fixes #3490

Remove superfluous `array_unique` call
No double value can occur in the array. When deleting, we already check if the object still exists in our file array and delete it only if it really still exists.

Fix return value of sort callback in TemplateListPage

Found using PHP 8's new warning:

> Message: uasort(): Returning bool from comparison function is deprecated,
> return an integer less than, equal to, or greater than zero

Fix PHP 8 compatibility for WCFSetup's error handler

see 0267fa9af7e18aa6449726f748e672cdac192d12

Added missing informal variant

Merge pull request #3466 from Sir-Will/patch-1

Fixes exception when not using primary PayPal email

Merge branch '3.1' into 5.2

Fix PHP 5.5 compatibility

see b044815dd9b6509fc44219684d7076cd28a80aa6
see #3480

Verify if business argument is set in PayPal callback

Fix handling of hidden form field values via AJAX

See #3053

Merge branch '3.1' into 5.2

Merge pull request #3462 from SoftCreatR/patch-14

Add detection for Chromium based Edge browser

Merge pull request #3471 from WoltLab/disable-spider-visit-tracking

Disable visit tracking for search engines

Merge branch '3.1' into 5.2

Do not decrement wcf1_user.articles when deleting unpublished articles

see b044815dd9b6509fc44219684d7076cd28a80aa6
see #3480

Merge branch '3.1' into 5.2

Update wcf1_user.articles when deleting articles

Fixes #3480

Added detection for Chromium based Edge browser

Add note, that the page.xml PIP instruction is needed with the next update
See #3474

Merge branch '3.1'

Fix updating `requireObjectID` for existing pages

The `data-user-id` is already set by the ReactionHandler

Fixes #3463

Prevent writes to the session rather than reads

Disable visit tracking for search engines

Revert "Whitelist `unserialize()` when running in enterprise mode"

`unserialize()` is unsafe, because it potentially allows for arbitrary
code execution.

This reverts commit 564ba8525e42c9d4677ee1ddac58c4c9c67fc113.

Include PayPal business email in exception

Fixes exception when not using primary PayPal email

PayPal allows adding multiple emails, if you are using one of the alternative emails instead of the primary email then WoltLab will throw the exception.
The `receiver_email` is always the primary email while `business` is the secondary in the IPN.

Clear language cache when clearing cronjob cache

Close #3465

Invalid suffix value caused an error when rendering the form

Approved `uniqid` for the use when running in enterprise mode

Merge branch '3.1' into 5.2

Moved the remark on the cleanup script

Merge pull request #3450 from WoltLab/cms-embedded-object-remove

Removed embedded object references when deleting articles / pages

Add update_com.woltlab.wcf_embeddedObjectOrphans.php

Add note about deletion of orphaned embedded objects

see #3429

Remove embedded object references when deleting pages

see #3429

Remove embedded object references when deleting articles

see #3429

Merge branch '3.1' into 5.2

Fix PHP 5.5 compatibility

Fix copy and paste bug

see 3cd8f5f63c1327c1534a4e24908b3e49d4074871

Merge branch '3.1' into 5.2

Properly update page search index when editing

see #3430

Properly update article search index when editing

see #3430

Merge branch '3.1' into 5.2

Improved detection of the links to twitch's clips

Fix VoidFormDataProcessor for `null` values

Release 5.2.8

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Release 3.1.16

Add close callback to form builder dialog

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Prevent Chromium browser form injecting base64 encoded images

Fix button class for single media form field

Fix duplicate data handler registration for captcha form fields

Close #3441

Fixed error message in Form Builder

Whitelist `unserialize()` when running in enterprise mode

Merge branch '3.1' into 5.2

Merge pull request #3439 from WoltLab/cronjob-fix2

Fix calculation of nextExec of cronjobs