Merge branch '5.5'

Merge pull request #4799 from WoltLab/wcf-force-logout

Immediately destroy the session for banned users

Merge pull request #4795 from WoltLab/mb-strpos

Remove unneeded usage of `mb_strpos`

Merge pull request #4786 from WoltLab/remove-x-frame-options

Always send `x-frame-options`

Merge pull request #4793 from WoltLab/error-500

Use HTTP 500 in error conditions

Merge pull request #4794 from WoltLab/wcfacp-initPackage

Remove obsolete WCFACP::initPackage()

Merge pull request #4796 from WoltLab/canonical-url-s-parameter

Remove the obsolete stripping of the `s=<sessionID>` when handling the canonical URL

Merge pull request #4788 from WoltLab/template-modifier-allowlist

Always restrict valid template modifiers to an allow list

Immediately destroy the session for banned users

The forced logout for banned users was introduced in
ab84d9cab2f864c23f0b18dbeb67e7ea79b1fe9f and only destroyed the session during
shutdown.

At the point where this check runs the request effectively is fully booted up
and in any case the NamedUserException would abort any further booting, thus it
is safe to simply destroy the session immediately to keep all the necessary
logic in a single location.

Use `->prepare()` instead of `->prepareStatement()` in SessionHandler

Merge branch '5.4' into 5.5

Merge pull request #4798 from pehbeh/patch-1

Update URL to WoltLab Community and Plugin-Store in acp/pageHeaderUser.tpl

Merge pull request #4797 from WoltLab/mysql-requirements

Increase minimum MySQL version to 8.0.29 / MariaDB 10.5.12

Increase minimum MySQL version to 8.0.29 / MariaDB 10.5.12

Update URL to WoltLab Community and Plugin-Store in acp/pageHeaderUser.tpl

Remove the obsolete stripping of the `s=<sessionID>` when handling the canonical URL

Remove unneeded usage of `mb_strpos`

If the return value of `mb_strpos` is only compared to `false` then there is no
need to use the multibyte engine, instead `str_contains()` does example the
same and is clearer.

The same applies if the return value is compared to `0`, in this case the size
of multibyte characters cannot have affected the offset. `str_starts_with()`
can be used instead.

Merge branch '5.5'

Fix incorrect use of `mb_strpos` in MysqlSearchEngine

The return value of `mb_strpos` needs to be checked type-safely, as both `0`
and `false` are falsy. In this case this likely was safe, as the inner join may
not appear at the start of the query, it was a questionable nonetheless.

Fix this by using `str_contains()` which makes the intent even clearer.

Merge remote-tracking branch 'origin/master'

Merge branch '5.5'

Merge branch '5.4' into 5.5

Update URL to WoltLab Ticket Support in acp/pageHeaderUser.tpl

Merge pull request #4792 from WoltLab/styleID-parameter

Remove the `$_REQUEST['styleID']` handling

Remove obsolete WCFACP::initPackage()

Ever since WoltLab Suite Core got its own frontend the PACKAGE_ID constant
should be defined at all times, making this method obsolete.

Remove the `$_REQUEST['styleID']` handling

Resolves #4533

Use HTTP 500 in error conditions

Sending a 500 Internal Server Error for unplanned errors is more appropriate
than a 503 Service Unavailable, as the latter is defined:

RFC 7231#6.6.4:

> The 503 (Service Unavailable) status code indicates that the server
> is currently unable to handle the request due to a temporary overload
> or scheduled maintenance, which will likely be alleviated after some
> delay.

It's not likely that the exception will resolve itself after some delay.

Simplify generation of random cookie prefix in WCFSetup

Drop unused constant `URL_TO_LOWERCASE` from default options.inc.php

The option was removed in f3aa502157efa11ac3c76e8471d69d0ac9f69f1f.

Merge pull request #4791 from WoltLab/wcfsetup-n

Remove the WCF_N input in WCFSetup

Remove the WCF_N input in WCFSetup

Users should install separate instances into separate databases for security
reasons. This also avoids issues with users running a non-standard number and
any existing guides / queries not working, because the database tables have a
different name.

Merge branch 'header-normalize'

Use `Header::normalize()` in `ImageUtil::browserSupportsWebp()`

see #4534

Use `Header::normalize()` in `RequestHandler::sendPsr7Response()`

see #4534

Merge branch '5.5'

Merge pull request #4784 from WoltLab/require-x64

Require 64-bit support

Merge pull request #4783 from WoltLab/ts-target

Target es2019 in tsconfig.json

Merge pull request #4787 from WoltLab/user-online-list

User online statistics were invisible when all active users were invisible

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Add the `required` attribute to the recipientID select in contact.tpl

see a8490749c3ba7014380d55462fc45dd635c1d71c

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Indicate that selecting a recipient is required in contact.tpl

Always restrict valid template modifiers to an allow list

Previously this allow list was only used in enterprise mode, but it is
generally a useful security feature. Thus the allow list previously known as
`$enterpriseFunctions` is applied in call cases.

This also makes it easier for developers, as there will be less differences
between the enterprise mode and the non-enterprise mode.

As before this allow list can easily be extended if a useful function is
missing from it.

Add 64-bit check to update_com.woltlab.wcf_5.5_checkSystemRequirements.php

Merge branch 'master' into require-x64

Add update_com.woltlab.wcf_5.5_checkSystemRequirements.php

see 48b47a4a8ba0260d52226c80063ebac081fa719b

Applied code suggestions

Show legend only when users are listed

Always send `x-frame-options`

Resolves #4484

User online statistics were invisible when all active users were invisible

Drop obsolete upgrade instructions from 5.4 to 5.5

Merge branch '5.5'

Merge branch '5.4' into 5.5

Drop obsolete update_com.woltlab.wcf_5.4.15_deleteDsStore.php

Drop obsolete fileDelete_5.5.xml

Check for 64-bit support in SystemCheckPage

Require 64-bit PHP in WCFSetup

see #4512

Merge branch '5.5'

Fix language items in recommended section of system requirements in WCFSetup

see 3445cbe2a005ead9843d9e17709a915631dd11b5
see e88d06dc88bc263b7424fbccfa47c13907413b8c

Target es2019 in tsconfig.json

Resolves #4595

Run php-cs-fixer using PHP 8.1

Merge pull request #4782 from WoltLab/system-requirements

Increase minimum PHP requirement

Release 5.5.0 Beta 2

Incorrect dialog position on smartphones

See https://www.woltlab.com/community/thread/295560-neues-thema-erstellen-au%C3%9Ferhalb-des-bildschirms/

Fix php.yml workflow

see 7cb8935ef0ef6707e0ed1cb8fe26ac17dd155400

Upgrade laminas/laminas-httphandlerrunner to 2.1.0

Tighten up version constraints in composer.json

Remove obsolete Symfony polyfills

Update PHP platform version in composer.json

Remove PHP < 8.1 from php.yml workflow

Increase minimum PHP version to 8.1.2

Add the 5.5 branch to GitHub workflows

Release 5.5.0 Beta 1

Removed the upgrade instructions from 5.4.*

Updating minified JavaScript files

Merge branch '5.4'

Fix English versions of `wcf.user.security.multifactor.backup.authenticationEmail.body.*`

The phrases contained broken template scripting, due to the use of the `'`
apostroph within a single quoted string.

Dialogs could become too wide with lots of text

Suppress the redundant loading indicator

See https://www.woltlab.com/community/thread/295539-doppelte-ladebalken/

Merge pull request #4781 from WoltLab/ci-check

Flip only the file name casing on AbstractFileDeletePackageInstallationPlugin::isFilesystemCaseSensitive()

Add fileDelete_5.5.xml

These files never were part of the git repository, but were accidentally
included in an upgrade package, because an unclean source directory was used
when building the upgrade.

Flip only the file name casing on AbstractFileDeletePackageInstallationPlugin::isFilesystemCaseSensitive()

Flipping the whole path does not provide any real benefit, because we only care
about whether the file system where WoltLab Suite resides is case sensitive or
not. It does however break when `open_basedir` is configured.

Merge branch '5.4'

Remove the codestyle workflow from branches that are not master

Lazy loading the user’s avatar causes flashes

Safari (macOS and iOS) does not handle lazy loaded images above the fold gracefully. This causes the user’s avatar to flicker noticeably on ever page navigation.

Merge pull request #4779 from WoltLab/search-result-pagination

Tracked the result page number in the url

Merge pull request #4777 from WoltLab/article-comment-like-notifications

Notifications about reactions to article comments

Merge pull request #4776 from WoltLab/scroll-to-comments

Scrolling to a comment not visible by default did not work

Proper prototype creation

Tracked the result page number in the url

Proper history navigation for searches

Track the page number of search result in the url

Visual separator for specialized search filters

See https://www.woltlab.com/community/thread/295502-kategorie-auswahl-ist-verrutscht/

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Revert "Show always an no selection option in custom select options build with the OptionHandler"

This reverts commit 6fef8b82e15794eee5317e6b15bb0670f137315c.

Incorrect margin of `.formSubmit` in dialogs

This was caused by the negative margin introduced in 1d224f10cdd05e7f5d09f869ee1fbbbdff025749 which caused the calculation to be off by 10px, causing a visible gap.

See https://www.woltlab.com/community/thread/295497-fehlerhafte-mobile-darstellung-von-dialogen/