Fix deprecation warnings in form builder in PHP 8

Close #3489

Fix 'Undefined index: additionalData' in Box

This error can be encountered when calling `new Box(123)` with
`123` not being the ID of a valid box.

Closes #3594

Fix page title for system check page

Close #3591

Merge branch '3.1' into 5.2

Removed obsolete code

There is no article counter for users (wcf1_user.articles) in version 3.1.

Add missing language variable `wcf.upload.error.uploadFailed`

Automatic removal of commas from tag names

Merge branch '3.1' into 5.2

Merge pull request #3585 from WoltLab/overdue-cronjob

Improved the wording of the cronjob error message

Improved readability

Co-authored-by: Matthias Schmidt <gravatronics@live.com>

Removed obsolete code

Merge branch '3.1' into 5.2

Default value blocked saving of empty date fields

Incorrect parameter order

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

Improved the wording of the cronjob error message

Page title could not be renamed via page management

Fixed broken autofocus in login form

Fix (non)empty dependency for radiobuttons

Show always an no selection option in custom select options build with the OptionHandler
Fixes #3579

Merge branch '3.1' into 5.2

Incorrect link to the user edit form

Merge pull request #3577 from WoltLab/filereader-range

Disable 'Range' handling for 0-byte files in FileReader

Disable 'Range' handling for 0-byte files in FileReader

see https://community.woltlab.com/thread/285960-download-eines-leeren-textdokuments-gibt-http-error-416-aus/

Add missing language variables

Updating minified JavaScript files

Increased the z-index of the reaction popover

This prevents the reactions from being overlapped by an adjacent tab menu on mobile devices.

Do not block the installation of not yet installed projects

Fixes #3572

This commit rectifies the change in d70a577df3662d168612e40035a4237c19bed5b0

Merge branch '3.1' into 5.2

Prevent data being pasted twice in Chromium on Android

Use the `selectionchange` event to suppress the selection of the quote tooltip

Prevent the read access to devtools projects without a valid path

Merge branch '3.1' into 5.2

Missing link target causes buttons to be unresponsive in some touch browers

Use consistent hostname in description of mail_smtp_host

Fix typo in error message in MultipleSelectionFormField

Fix PHP 8 compatibility in GDImageAdapter (#3558)

Fixed image resource validation

Resolves #3492

Fix PHP 8 compatibility in HTTPRequest (#3557)

Fixed PHP8 related error message (Argument #1 ($arg) must be passed by reference, value given)

Closes #3554

Fixed icon alignment in the "login with google" button

Merge branch '3.1' into 5.2

Prevent Android from collapsing the keyboard on [Enter]

Somehow the linkify redactor plugin causes Android to collapse the on-screen
keyboard if a new line is entered.

This plugin was already removed for 5.2 in commit 19af26fc22bc31d699db6021dee2eda1d23b8a14,
so let's effectively disable it for 3.1 as well.

see https://community.woltlab.com/thread/285573-wsc-3-1-17-smartphone-tastatur-schlie%C3%9Ft-sich-unbeabsichtigt/?postID=1816848#post1816848

Fix changing monolingual to multilingual package data via devtools

Close #3550

Fix typo in German language item

Merge branch '3.1' into 5.2

Incorrect calculation of the last 2 digits of the year

Fixes #3547

Merge pull request #3548 from WoltLab/acp-headContent

Add `$headContent` placeholder variable to header.tpl in ACP

Add `$headContent` placeholder variable to header.tpl in ACP

This allows the developer to easily add page specific `<style>` tags in
the appropriate location.

Release 5.2.9

Merge branch '3.1' into 5.2

Release 3.1.17

Revert "Fix deprecation warnings in form builder in PHP 8"

This reverts commit 5ae5d6c5621c8fe1f30d6a2c787024b9bcadee87.

As commented on the original commit this change is not compatible with PHP
7.0 which is the lowest PHP version we support as of right now. Revert
this commit for now to not break compatibility with existing installations
of 5.2 until a proper solution for 5.3 and later is developed.

See #3489.

`http_build_query()` is now allowed for calls from within a template

Fix notifications with are triggered by merged users
Objects of merged users are rewritten to the new user. This is currently done but not by the notification authors. There the author is simply deleted from the notification. This works for most notifications, but not for reactions, because they always expect a concrete author (a guest cannot assign reactions). If users are deleted, it works as intended, since the corresponding reaction object is then also deleted.

Release 5.2.9

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Updating minified JavaScript files

Merge branch '3.1'

Remove fullscreen mode in template diff
The fullscreen mode does not have any advantage at this point because the diff is also displayed with a height of 700px (just like without fullscreen mode).

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.