Merge branch '5.2'

Merge branch '3.1' into 5.2

Incorrect calculation of the last 2 digits of the year

Fixes #3547

Merge pull request #3545 from WoltLab/styleLogo-download

Add DownloadStyleLogoJob

Merge branch '5.2' into master

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.

Merge pull request #3546 from WoltLab/styleUpdate-idempotent

Support running the style update script multiple times

Support running the style update script multiple times

Queue DownloadStyleLogoJob during style upgrade

Add DownloadStyleLogoJob

Merge branch 'master' of github.com:WoltLab/WCF

Report any duplicate usernames before applying the upgrade

Fixes #3544

Unnecessary description removed

Updating minified JavaScript files

Merge pull request #3539 from mutec/53upgrade

WIP: Prepare upgrade instructions for 5.2 to 5.3

Resolved some issues during the upgrade, first successful upgrade

Deploy some files ahead of the upgrade

Set up the proper default package servers

Merge branch 'master' into pr/3539

Added hover indicator in notification settings form

Merge branch '5.2'

Preserve the leading and trailing whitespace in `<kbd>` on output

Fixes #3536

Merge pull request #3495 from Krymonota/update-version-dev-config-file-name-5.3

Update version number in Devtools config file name to 5.3

prepare upgrade instructions for 5.2 to 5.3

We need to provide `/lib/core.functions.php` and `/lib/system/api/*` before executing the complete file-PIP; otherwise we're getting issues with Guzzle being not found and causing an abortion.
Put the style and email-confirmation scripts into standalone-instructions since they might cause long execution times. In my test-environment I updated without that parameter.

Merge pull request #3538 from mutec/53upgradeStyle

fix updating the styles to 5.3

fix updating the styles to 5.3

If `Style::$imagePath` is empty, it resulted in copying the shole WSC-folder into a style-asset-folder - and doing this until we reache the timeout limits.
Including `global.php` ended up in trying to define the option-constants (especially the fixed-value-options like `MODULE_ATTACHMENTS`) again while they are already defined - which causes a Fatal Error-Message.

Release 5.2.9

Merge branch '3.1' into 5.2

Release 3.1.17

Merge pull request #3524 from WoltLab/package-server-refactor

Clean up package update server management

Merge branch '5.2' into master

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

Merge pull request #3531 from WoltLab/jslang-template-plugin

Template plugin for the assignment of language variables in javascrip…

Added missing use of jslang

Added missing use of jslang

Merge branch '5.2'

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Updating minified JavaScript files

Unified usage of the elseif tag in template scripting

Fix selection of related articles for old MySQL versions

MySQL < 5.7 and apparently MariaDB up to at least 10.3 is unable to deduce
that there can only be exactly a single row from the `article` table, because
it is functionally dependent on `tag_to_object.objectID`.

Help them out by using `MAX` which effectively does nothing, because we're
taking the `MAX` of a single item.

Fixes #3528.

Fix Guzzle following redirects with a custom temporary sink

When following redirects Guzzle copies over the request options for the next
iteration internally, including the bare `resource` that was passed to it.

This bare resource will then be wrapped into the Stream wrapper. This stream
wrapper explicitely calls `close()` for the underlying resource when it is
being closed.

The created Streams for the subrequests will be closed before the `send`
function returns, thus ultimately closing the resource for the “parent”
request.

Fix this issue by passing a Stream, instead of a bare resource. This one
will be passed along as-is.

Fixes #3529.

Template plugin for the assignment of language variables in javascript code

Closes #3530

Improved the performance of reading tags without a specific language id

Merge branch '5.2'

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

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

Merge branch '5.2'

Duplicate `<br>` inserted on paste

Fixes #3454

Workaround for custom elements being chopped up by outdenting an empty `li`

Reproducible test case:
```
<woltlab-spoiler>
  <ul>
    <li>Test</li>
    <li>T</li> <!-- Backspace this item -->
  </ul>
</woltlab-spoiler>
```

Merge branch '5.2'

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

Merge branch '5.2'

Merge branch '3.1' into 5.2

Adjusted appearance of the "sign in with google" button

Merge branch '5.2'

Merge branch '3.1' into 5.2

Reopened dialogs were hidden behind new dialogs

Merge branch '5.2'

Incorrect blacklisting of `register_activation_method`

Updating minified JavaScript files

The implementation for JS bundles for other repositories was outdated

See #3520

Update the number of votes in a poll after voting

Merge branch '5.2'

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.

Updating minified JavaScript files

uglifyjs.minify expects the code and not a path

For whatever reason that this has worked in the past, in node 14+ the behavior appears to have been changed.

The `cascade` option is not mentioned in the list of valid options and since error message were implicitly suppressed, this went by unnoticed.

Fixes #3520

Ensure that the official package servers exist

Remove support for filtering the results of PackageUpdateServer::getActiveUpdateServers()

Dynamically generate package update server path in PackageUpdateServer::handleData()

Add `\wcf\getMinorVersion()`

Add `since` PHPDoc for cleaned package update server management

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

Merge branch '5.2'

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

Prevent rich metacode being dysfunctional in new messages

Merge branch '5.2'

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.

Add missing word in exception message

See 709b412bf72084e48d408787d8455f9876fcdaed

Use StringUtil::endsWith to validate package update server URL

Use dedicated column for loginUsername in packageUpdateServerList

Improve phrasing for cleaned package update server management

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

Add note to package.xml that obsolete update servers should be deleted

Force a proper URL for *.woltlab.com package update servers

Disallow deleting official update servers

Prevent adding *.woltlab.com update servers

Prevent adding duplicate package update servers

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

Disallow changing the package update server URL

Do not echo loginPassword in packageUpdateUnauthorized.tpl

Do not echo loginPassword in PackageUpdateServerEditForm

Show loginUsername in packageUpdateServerList

This allows the administrator to quickly see which update servers are missing
credentials.